Repository navigation
Phase D1: design foundations (green scheme, record gold, shared formatter) - #22
Merged
Merged
Conversation
…tter) - Full light and dark colour schemes from #2E6B30 with every role set, surface containers included; dynamic colour is now off by default. - Record gold as an extended colour family (MaterialTheme.recordColors) beside the scheme; the logger's PR badge, History cards, exercise detail trophies and the finish summary's records card move off tertiary. - One formatter in ui/format: live timers keep the clock format, finished durations read "58 min" / "1 h 11 min", and every displayed weight and volume goes through it with its unit. - Tabular figures for timers, set fields, the Last column and totals. - Shared RecordChip, RecordDisc and StatTile components; History cards use the chip for their PR count. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014thT5DHYQ5GxporYx5N4XW
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014thT5DHYQ5GxporYx5N4XW
This was referenced Oct 31, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The first phase of the design pass (D1 in
docs/IMPLEMENTATION_PLAN.md): the whole app in one green scheme, records in gold, and numbers that read unambiguously.What changed
#2E6B30using the Theme Builder's tonal mapping, with primary pinned to the brand colour. Every role is set, surface containers included, so nothing falls back to Material's purple baseline.dynamicColornow defaults to off (F01). The window background matches the new scheme.MaterialTheme.recordColors: record, onRecord, recordContainer, onRecordContainer), provided besideMaterialThemeso it stays gold once dynamic colour can be turned back on. The logger's PR badge, History cards, exercise detail trophies and the finish summary's records card all moved off tertiary (F02). The calendar's selected-day ring was also tertiary, so it moved to a neutralonSurfacering.ui/format/Formats.kt) (F03):formatClockis used only for live timers and rest settings.weightText/volumeText/weightNumberText, with the unit in one string. Decimal marks follow the locale.ui/components/RecordMarks.kt):RecordChip,RecordDiscandStatTile. History cards already use the chip for their PR count.formatElapsedandformatVolumemoved out ofWorkoutViewModel.kt. Their tests moved toui/format, and new tests cover durations and weights.Testing
./gradlew testDebugUnitTest ktlintCheck lintDebug assembleDebugpasses locally.Note
app/build.gradle.ktssets lint'stextOutput = file("stdout"), which writes a file namedstdoutintoapp/on every lint run instead of printing to the console. One got committed here and is removed in the second commit. The setting is left as it was.Generated by Claude Code