Skip to content

Enable edge to edge for FilteredDeckOptionsFragment - #21597

Open
lukstbit wants to merge 1 commit into
ankidroid:mainfrom
lukstbit:feat-edge-create-filtered-deck
Open

Enable edge to edge for FilteredDeckOptionsFragment#21597
lukstbit wants to merge 1 commit into
ankidroid:mainfrom
lukstbit:feat-edge-create-filtered-deck

Conversation

@lukstbit

Copy link
Copy Markdown
Member

Purpose / Description

Enable edge to edge for FilteredDeckOptionsFragment. I had this implemented manually but waited after seeing criticalAY's latest PR.

The changed activity, ConfigAwareSingleFragmentActivity is used both by FilteredDeckOptionsFragment and by ScheduleRemindersFragment(in DeckPicker). With these changes ScheduleRemindersFragment doesn't respect edge to edge but I didn't do anything about this yet to wait for #21519.

Some images:

API 28:

Screenshot_20260824_184141 Screenshot_20260824_184151 Screenshot_20260824_184200

API 36:

Screenshot_20260824_183950 Screenshot_20260824_184006 Screenshot_20260824_184021

Fixes

How Has This Been Tested?

Manually on two devices: API 28(big notch + system bars) and API 36(punch cutout + system bars).

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

Edge to edge was enabled for ConfigAwareSingleFragmentActivity which
is used as a container for FilteredDeckOptionsFragment and
ScheduleRemindersFragment.

@criticalAY criticalAY left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! thankyou

@criticalAY criticalAY added Needs Second Approval Has one approval, one more approval to merge and removed Needs Review labels Aug 24, 2026

@david-allison david-allison left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, merge at will

Optional suggestion to make the navigation bar match the background color

@david-allison david-allison added Needs Author Reply Waiting for a reply from the original author Pending Merge Things with approval that are waiting future merge (e.g. targets a future release, CI wait, etc) and removed Needs Second Approval Has one approval, one more approval to merge labels Aug 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs Author Reply Waiting for a reply from the original author Pending Merge Things with approval that are waiting future merge (e.g. targets a future release, CI wait, etc)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants