Skip to content

fix(settings): edge to edge toolbar color - #22096

Open
david-allison wants to merge 1 commit into
ankidroid:mainfrom
david-allison:22087
Open

david-allison wants to merge 1 commit into
ankidroid:mainfrom
david-allison:22087

Conversation

@david-allison

Copy link
Copy Markdown
Member

Note

Assisted-by: GPT-6

Fixes

How Has This Been Tested?

⚠️ Trusting screenshot tests on this one

Checklist

  • You have a descriptive commit message with a short title (first line, max 50 chars).
  • You have commented your code, particularly in hard-to-understand areas
  • You have performed a self-review of your own code
  • UI changes: include screenshots of all affected screens (in particular showing any new or changed strings)
  • UI Changes: You have tested your change using the Google Accessibility Scanner

@github-actions

github-actions Bot commented Sep 25, 2026 •

Copy link
Copy Markdown
Contributor

Snapshot diff report vs main. Open screenshot-diff for diffs.

  • PreferencesScreenshotTest: 10 changes
All 10 changed screenshots

PreferencesScreenshotTest

  • GeneralSettingsFragment_sdk36_collapsed_compare.png
  • GeneralSettingsFragment_sdk36_fullscreen_collapsed_compare.png
  • GeneralSettingsFragment_sdk36_fullscreen_cutout_collapsed_compare.png
  • GeneralSettingsFragment_sdk36_fullscreen_cutout_compare.png
  • GeneralSettingsFragment_sdk36_fullscreen_expanded_compare.png
  • GeneralSettingsFragment_sdk36_system_bars_restored_compare.png
  • HeaderFragment_gesture_bar_hidden_scrolled_to_bottom_compare.png
  • HeaderFragment_gesture_bar_scrolled_to_bottom_compare.png
  • HeaderFragment_toolbar_collapsed_compare.png
  • HeaderFragment_toolbar_expanded_compare.png

@david-allison
david-allison marked this pull request as draft September 26, 2026 15:57
Keep the top inset inside the toolbar so the collapsing scrim covers the
status bar. Increase both toolbar heights to preserve the scroll range
and touch targets.

Adjust the scrim visibility threshold by the same inset. Otherwise, a
48dp cutout prevents the scrim from appearing even when fully collapsed.
Reproduced this in settingsPanelSystemBarsChange's fullscreen cutout
screenshot.

Validated eight screenshot cases on phones and tablets in light and dark
themes, plus Kotlin formatting and Android lint.

GPT-6 assisted with implementation, screenshot tests, and self-review.

Fixes 22087

Assisted-by: GPT-6
@david-allison
david-allison marked this pull request as ready for review September 26, 2026 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Settings screen: better handle background scrolling color

1 participant