Skip to content

Center training widget tiles and fit workout icons - #13

Closed
AlexSchmidt1999 wants to merge 2 commits into
codex/today-sync-layoutfrom
codex/training-widget-alignment
Closed

AlexSchmidt1999 wants to merge 2 commits into
codex/today-sync-layoutfrom
codex/training-widget-alignment

Conversation

@AlexSchmidt1999

Copy link
Copy Markdown
Owner

Training favorite tiles now center their existing workout icons and names. Idle tiles reserve two centered text lines so single-line and wrapped names align; compact buttons keep natural height. The existing SF Symbols fit their square icon bounds without distortion, including the strength-training dumbbell. Sport glyph identities, user-defined names and training actions are unchanged. UI uses existing design tokens and native image scaling.

Validation: eight existing WorkoutTypeIconTests passed; macOS app and iOS Release app/widgets/watch targets compiled. Translation, doc-comment, source policy and four-bundle unsigned binary checks passed. Local SwiftUI previews cover German and longer English labels at small widget sizes, plus active/paused/end states; Live Activities retain their 160-point height checks. Actual iPhone tinted/Always-On presentation remains to be reviewed on the device.

NOOP 12.0.0 build 439 is verified installed and launched on the owner’s iPhone over Wi-Fi with the existing bundle ID. It combines the previous fork fixes with this Apple-only UI follow-up. The separate baseline commit changes only the reviewed build-counter project hash. Stacked on the Today sync-layout branch; no upstream PR or merge is requested and the local monitor remains paused.

@AlexSchmidt1999

Copy link
Copy Markdown
Owner Author

Closing this older fork test PR. Its full head (b13d23414307) is included in the combined test branch. New test-version work continues in #28. The branch and commits are kept.

The relevant changes are now submitted for upstream review in #2761.

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.

2 participants