Skip to content

Move service favicon to domain row - #293

Merged
arjunkomath merged 1 commit into
mainfrom
fix/service-card-favicon-domain
Aug 15, 2026
Merged

Move service favicon to domain row#293
arjunkomath merged 1 commit into
mainfrom
fix/service-card-favicon-domain

Conversation

@arjunkomath

Copy link
Copy Markdown
Member

Summary

  • remove the favicon from beside the service name
  • use the favicon instead of the globe icon in the public-domain row
  • preserve the existing favicon lookup, size, and error handling

Checks

  • pnpm exec oxfmt --check components/service/service-canvas.tsx
  • ./node_modules/.bin/tsc --noEmit
  • pnpm lint (passes with 7 existing warnings in unrelated files)

@github-actions

Copy link
Copy Markdown

React Doctor found no new issues. 🎉

Reviewed by React Doctor for commit eb9010a.

@arjunkomath
arjunkomath added this pull request to the merge queue Aug 15, 2026
Merged via the queue into main with commit cab4652 Aug 15, 2026
8 checks passed
@arjunkomath
arjunkomath deleted the fix/service-card-favicon-domain branch August 15, 2026 05:46
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