Skip to content

Fix PWA icons: replace misformatted WebP portrait photos with proper circular PNG icons matching Friends app style#5

Merged
OrF8 merged 3 commits intomasterfrom
copilot/update-pwa-app-icons-seinfeld
Mar 24, 2026
Merged

Fix PWA icons: replace misformatted WebP portrait photos with proper circular PNG icons matching Friends app style#5
OrF8 merged 3 commits intomasterfrom
copilot/update-pwa-app-icons-seinfeld

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 24, 2026

The existing PWA icons were WebP files with a .png extension, sized at 1024×1536 (portrait photos) — completely wrong for PWA icon surfaces. This replaces them with proper square PNG icons styled consistently with RandomFriendsEpisode.

Icon changes

  • Format: WebP → actual PNG
  • Dimensions: 1024×1536 portrait → 192×192 / 512×512 square
  • Composition (matches Friends icon proportions exactly):
    • Solid dark background #0f2027 (from manifest) fills full square — no transparency
    • Original Seinfeld photo/logo content cropped into a centered circle at 75% of icon size (same ratio as Friends' circle)

No manifest changes needed

manifest.json already declares type: image/png and correct paths — the file content now matches the declared type.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI changed the title [WIP] Update PWA/app icons to match Friends style Fix PWA icons: replace misformatted WebP portrait photos with proper PNG icons matching Friends app style Mar 24, 2026
Copilot AI requested a review from OrF8 March 24, 2026 18:27
Copilot finished work on behalf of OrF8 March 24, 2026 18:27
Copy link
Copy Markdown
Owner

@OrF8 OrF8 left a comment

Choose a reason for hiding this comment

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

@copilot I want the style to remain the same (before the change), but the style (circular) to be like the other repo

…und, matching Friends style

Co-authored-by: OrF8 <162809292+OrF8@users.noreply.github.com>
Agent-Logs-Url: https://github.com/OrF8/RandomSeinfeldEpisode/sessions/d4031b77-d9e1-4002-bf28-e0b226755b9c
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Mar 24, 2026

@copilot I want the style to remain the same (before the change), but the style (circular) to be like the other repo

Done in commit 887d291. The icons now keep the original Seinfeld photo/logo content, but with the circular composition style from the Friends repo — the Seinfeld image is cropped into a circle centered on a dark #0f2027 background, at 75% of the icon size (same proportions as Friends).

Here's the comparison (Friends reference on the left, new Seinfeld on the right):

comparison

Both icons are proper 192×192 and 512×512 PNGs.

Copilot AI changed the title Fix PWA icons: replace misformatted WebP portrait photos with proper PNG icons matching Friends app style Fix PWA icons: replace misformatted WebP portrait photos with proper circular PNG icons matching Friends app style Mar 24, 2026
Copilot AI requested a review from OrF8 March 24, 2026 18:50
Copilot finished work on behalf of OrF8 March 24, 2026 18:50
@OrF8 OrF8 marked this pull request as ready for review March 24, 2026 20:19
Copilot AI review requested due to automatic review settings March 24, 2026 20:19
@OrF8 OrF8 merged commit 88779b9 into master Mar 24, 2026
3 of 4 checks passed
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@OrF8 OrF8 deleted the copilot/update-pwa-app-icons-seinfeld branch March 24, 2026 20:19
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.

3 participants