Show the cloud as the app favicon #57

Closed
opened 2026-09-30 02:57:40 +00:00 by agent-pi · 1 comment
Collaborator

The page header shows a little cloud (the ☁️ emoji in the title), but the browser tab has no favicon — every page just gets the browser default icon.

The cloud should also appear as the favicon for the app.

The page header shows a little cloud (the ☁️ emoji in the title), but the browser tab has no favicon — every page just gets the browser default icon. The cloud should also appear as the favicon for the app.
Author
Collaborator

Plan of attack:

  • Add src/yunjin/web/static/favicon.svg — renders the same ☁️ the header uses (emoji-based SVG, adapts to light/dark browser chrome).
  • <link rel="icon" type="image/svg+xml" href="..."> in base.html <head>.
  • Small /favicon.ico redirect route on the app (browsers request it unprompted; without it every page view logs a 404).
  • Tests in tests/test_web.py: link tag present in rendered base page, SVG served from static, /favicon.ico resolves to the icon (TDD — failing tests first).

Branch: feat/container-version-and-favicon (shared with #56).

**Plan of attack**: - Add `src/yunjin/web/static/favicon.svg` — renders the same ☁️ the header uses (emoji-based SVG, adapts to light/dark browser chrome). - `<link rel="icon" type="image/svg+xml" href="...">` in `base.html` `<head>`. - Small `/favicon.ico` redirect route on the app (browsers request it unprompted; without it every page view logs a 404). - Tests in `tests/test_web.py`: link tag present in rendered base page, SVG served from static, `/favicon.ico` resolves to the icon (TDD — failing tests first). Branch: `feat/container-version-and-favicon` (shared with #56).
Sign in to join this conversation.
No milestone
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Reference
marvin8/yunjin#57
No description provided.