From f0d641198c0be63b60a0db66faa50f3053a03641 Mon Sep 17 00:00:00 2001 From: NuklearRabbit <145918611+NuklearRabbit@users.noreply.github.com> Date: Tue, 4 Aug 2026 03:10:11 +0200 Subject: [PATCH] fix: serve the missing Fleet Ops favicon There was no favicon at all -- index.html never linked one, and the frontend Dockerfile's build stage never copied the public/ directory into the build context, so even after adding public/favicon.svg locally, the containerized build silently dropped it (nginx fell back to serving index.html for that path). Fixed both: index.html links /favicon.svg, and the Dockerfile now copies public/ alongside src/. The favicon reuses the existing BrandMark glyph (petrol background, teal accent) for visual consistency with the in-app brand mark. A regression test for this lives in the earlier translation-fix commit (frontend/e2e/fleet-ops-correction.spec.ts), added together with the fix at the time. --- frontend/Dockerfile | 1 + frontend/index.html | 1 + frontend/public/favicon.svg | 5 +++++ 3 files changed, 7 insertions(+) create mode 100644 frontend/public/favicon.svg diff --git a/frontend/Dockerfile b/frontend/Dockerfile index acd97ff..1ef139f 100644 --- a/frontend/Dockerfile +++ b/frontend/Dockerfile @@ -1,6 +1,7 @@ FROM node:22-alpine AS build WORKDIR /app COPY package.json package-lock.json tsconfig.json vite.config.ts index.html ./ +COPY public ./public COPY src ./src RUN npm ci && npm run build diff --git a/frontend/index.html b/frontend/index.html index 307c14e..373547b 100644 --- a/frontend/index.html +++ b/frontend/index.html @@ -4,6 +4,7 @@ +