Skip to content

feat(ios): add the Luke icon to the Apple Watch app - #657

Merged
dastratakos merged 1 commit into
mainfrom
dastratakos/belo-horizonte-v2
Sep 3, 2026
Merged

feat(ios): add the Luke icon to the Apple Watch app#657
dastratakos merged 1 commit into
mainfrom
dastratakos/belo-horizonte-v2

Conversation

@dastratakos

@dastratakos dastratakos commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Summary

  • The LukeWatch target shipped with no app icon, so the watch home screen drew a blank circle beside the iPhone app's Luke face. This gives the watch app the same icon, using the iPhone appiconset as the reference.
  • watchOS masks every icon to a circle and takes a single appearance (no dark or tinted variants), so the icon is one 1024×1024 PNG in a new apps/ios/LukeWatch/Assets.xcassets/AppIcon.appiconset/, cut from the existing full-bleed design/brand/icon/luke-icon-ios-dark.svg rather than a second drawing. Space black is the tile that reads on the watch's black home screen, the same call the packaged .icns makes for macOS. The glyph keeps its 58% share of the visible tile, which for a circular mask is the diameter, and its bounding box sits well inside the circle.
  • Luke.xcodeproj: the two LukeWatch build configurations now set ASSETCATALOG_COMPILER_APPICON_NAME = AppIcon, as the iPhone target's do. No other project change.
  • design/brand/README.md gains the rsvg-convert line that cuts the watch PNG beside the iPhone ones and a note in the files table; the generator's iOS icon comment records that the dark cut also serves the watch, since changing that drawing now changes two icons. No SVG changed, so generate-brand-assets.mjs --check is unaffected.

Evidence

  • Platform-independent checks: ./scripts/check.sh passed (repository contract checks, Biome, typecheck, tests, build).
  • macOS Electron verification (./scripts/verify.sh): not run — no desktop change; nothing outside apps/ios/ and design/ is touched.

Mac-certified results (053d28e, watchOS 26.5 / Xcode 26.6)

xcodebuild LukeWatch scheme (Apple Watch Series 11 (46mm) simulator):  BUILD SUCCEEDED
LukeWatch.app/Info.plist:  CFBundleIcons.CFBundlePrimaryIcon.CFBundleIconName = "AppIcon"
LukeWatch.app/Assets.car:  AppIcon, Idiom watch, 1024×1024 Icon Image

Automated visual evidence

Download the deterministic macOS evidence · workflow run

  • Commit: 053d28ebc63ba38a1172d2e1dae5220aaead5823
  • Scenario: smoke
  • Physical-notch check: not performed by CI

Physical-device evidence

  • Screenshot or screen recording: not attached — the compiled asset catalog was inspected with assetutil instead; the PNG itself is in the diff.
  • Physical-notch check: not performed — not applicable, watchOS change only
  • Device/display configuration: Apple Watch Series 11 (46mm) simulator, watchOS 26.5, Xcode 26.6

Notes

  • Blockers or follow-up verification: None

Open workspace in Conductor

Open in Alchemize

The LukeWatch target shipped with no app icon, so the watch home screen drew
a blank circle. watchOS masks every icon to a circle and takes a single
appearance, so the icon is a second 1024 PNG cut from the full-bleed dark
iOS drawing rather than a second drawing: space black is the tile that reads
on the watch's black home screen, the same call the packaged .icns makes.
The watch target's build settings now name the appiconset, as the iPhone
target's do, and the brand README's recipe cuts the watch PNG beside the
iPhone's.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@vercel

vercel Bot commented Sep 3, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
luke-web Ready Ready Preview Sep 3, 2026 3:22am UTC

Request Review

@dastratakos
dastratakos added this pull request to the merge queue Sep 3, 2026
Merged via the queue into main with commit 6b00b23 Sep 3, 2026
10 checks passed
@dastratakos
dastratakos deleted the dastratakos/belo-horizonte-v2 branch September 3, 2026 03: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