Skip to content

fix: Properly handle spacing for spinner checkmarks - #21469

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

fix: Properly handle spacing for spinner checkmarks#21469
david-allison wants to merge 1 commit into
ankidroid:mainfrom
david-allison:21030

Conversation

@david-allison

@david-allison david-allison commented Aug 11, 2026

Copy link
Copy Markdown
Member

Note

Assisted-by: Claude Fable 5 - all (comments modified)

Purpose / Description

Fixes a TODO

image

Fixes

Approach

  • use InsetDrawable (8dp) - using a ldrtl override for RTL handling

How Has This Been Tested?

Before

image

After

image

RTL
image

Learning (optional, can help others)

ldrtl = layour direction; right to left

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

* use `InsetDrawable` (8dp) - using a `ldrtl` override for RTL
 handling

Fixes 21030

Assisted-by: Claude Fable 5 - all
@github-actions

Copy link
Copy Markdown
Contributor

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

  • DeckPickerScreenshotTest: 2 changes
All 2 changed screenshots

DeckPickerScreenshotTest

  • hierarchy_lines_collapsed_compare.png
  • hierarchy_lines_compare.png

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.

Properly handle spacing for spinner checkmarks

1 participant