Refresh Anycast color, typography, and spacing - #5
Conversation
|
Follow-up visual-state audit is now pushed in This pass covers the gaps from the simulator review: player control colors, blurred/darkened artwork background, settings text and both sliders, selected/unselected and disabled switches, compact episode/mini-player progress, synchronized remaining-time text, and non-seekable loading/unknown progress states. Light and dark endpoints were checked on an iPhone 17 Pro simulator, and the expanded suite passes all 14 tests. @sljeff this is ready for another review. I have not merged it; the new CI run is still in progress. |
|
Additional audit requested for transcript and subscriptions is pushed in Transcript now has explicit unavailable, prompt, processing, failed, loading, ready, translation-loading/failure, artwork fallback, and action states; the generate CTA and iOS status bar were corrected for contrast. Subscriptions now distinguish loading/error/empty/content and provide artwork loading/error fallbacks instead of briefly presenting loading as empty. Local simulator data had no transcript, translation, or subscription rows, so I did not trigger AI generation or fabricate user data. The non-present states are covered by deterministic state/component tests; the suite now passes 19 tests. @sljeff please review this latest commit after the restarted CI run completes. I have not merged the PR. |
|
Final CI for |
What
State audit
Impact
Presentation and visual-state handling only: colors, text styling, spacing, surfaces, progress presentation, async presentation flags, and control states. No data schema, persistence format, API, release version, signing, or store behavior changes.
Validation
dart analyzeon all follow-up changed source/test files — no issuesflutter test --no-pub— 19 tests passedgit diff --checkflutter build ios --simulator --debug --no-pubflutter pub get --enforce-lockfile— Flutter 3.44.6 would update five SDK transitive dependencies (characters,matcher,material_color_utilities,meta,test_api);pubspec.lockwas left unchangedflutter analyze --no-fatal-infos --no-pubexits nonzero on the existing experimentalprofilesSampleRatewarning atlib/main.dart:62; CI uses the repository’s fixed toolchain and is authoritativeAll required CI checks pass. Ready for owner review; not merged.