Skip to content

feat(web): add Stage favicon to the web UI#27

Merged
dastratakos merged 1 commit into
mainfrom
dastratakos/add-stage-favicon
May 4, 2026
Merged

feat(web): add Stage favicon to the web UI#27
dastratakos merged 1 commit into
mainfrom
dastratakos/add-stage-favicon

Conversation

@dastratakos

@dastratakos dastratakos commented May 4, 2026

Copy link
Copy Markdown
Contributor

Summary

Add the Stage favicon so the browser tab displays the Stage icon instead of the default blank icon. Assets copied from the hosted Stage monorepo.

Changes

  • Added favicon.ico, favicon-16.png, and favicon-32.png to packages/web/public/
  • Added <link> tags for all three favicon variants in packages/web/index.html

Testing

  • Ran pnpm typecheck and pnpm lint — both pass

Open in Stage

Copy favicon assets from the hosted Stage monorepo and add link tags
to index.html so the browser tab shows the Stage icon.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request adds standard favicon links to the index.html file. The review feedback suggests expanding the icon set to include higher-resolution versions, such as a 180x180 apple-touch-icon and a 192x192 PNG, to ensure a polished experience on high-DPI displays and mobile devices.

Comment thread packages/web/index.html
@dastratakos dastratakos marked this pull request as ready for review May 4, 2026 16:59
@dastratakos dastratakos merged commit 29b2069 into main May 4, 2026
5 checks passed
@dastratakos dastratakos deleted the dastratakos/add-stage-favicon branch May 4, 2026 16:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant