fix(air-quality): use rounded chips in the legend - #1107
Merged
PranshulGG merged 1 commit intoAug 28, 2026
Merged
Conversation
Co-authored-by: OpenAI GPT-5.6 Sol <noreply@openai.com>
Owner
|
Pretty small UI change. Wouldn't call it a bug, but okay, I guess. Shouldn't cause any issues, even if ai did it |
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.
Closes #1106.
What's happening
The category legend at the top of the detailed Air Quality screen renders as touching rectangular color blocks. Once the labels wrap, the connected strip breaks across two lines and no longer reads as a continuous AQI scale. It also looks unrelated to the rounded label treatment used elsewhere in WeatherMaster.
The legend uses the default rectangular shape from
Surface, has no spacing between entries, and relies on the default text style with broad horizontal padding.Implementation
labelLargewith the alert chip's padding instead of the default text treatment.The AQI category order, semantic colors, contrast colors, and chart data remain unchanged. The labels are informational, so they remain plain
Surfaces rather than interactive Material filter or assist chips. The change is limited to the legend on the detailed Air Quality screen.Testing
./gradlew clean assembleDebugwith JDK 21 successfully.I did not verify every translated label. Languages with substantially longer category names may wrap differently. This change preserves the existing AQI foreground and background colors; the existing Moderate and Poor color pairs remain below the WCAG 4.5:1 contrast target for normal-sized text.
Screenshots
AI disclosure
OpenAI GPT-5.6 Sol through the Codex harness in T3 Code was used to inspect the existing UI patterns, implement the Compose styling change, run the build and emulator checks, and draft this description. I reviewed the diff and rendered result and take responsibility for the submitted change.