Skip to content

fix(change-note-type): stop mappings being clipped - #22286

Open
david-allison wants to merge 3 commits into
ankidroid:mainfrom
david-allison:air/19443-2-13eebe8e-b0f-5222de1e-a6e
Open

david-allison wants to merge 3 commits into
ankidroid:mainfrom
david-allison:air/19443-2-13eebe8e-b0f-5222de1e-a6e

Conversation

@david-allison

@david-allison david-allison commented Oct 3, 2026 •

Copy link
Copy Markdown
Member

Note

Assisted-by: GPT-6

Purpose / Description

See screenshots 😆

Fixes

Approach

Add regression tests, then:

  • Fix scrolling spacing so bottom rows are reachable
  • Wrapped field/template don't overlap vertically
  • Truncate note-type names
  • Tabs handle large text

How Has This Been Tested?

desktop_fields_fit_viewport_compare desktop_gesture_navigation_compare desktop_landscape_compare desktop_long_note_type_names_large_text_landscape_compare desktop_long_note_type_names_large_text_narrow_portrait_compare desktop_many_fields_large_text_landscape_compare desktop_many_fields_scrolled_to_bottom_compare desktop_many_templates_large_text_landscape_compare desktop_many_templates_scrolled_to_bottom_compare desktop_portrait_compare desktop_portrait_cutout_compare desktop_templates_fit_viewport_compare desktop_templates_tab_large_text_compare fields_fit_viewport_compare gesture_navigation_compare landscape_compare long_note_type_names_large_text_landscape_compare long_note_type_names_large_text_narrow_portrait_compare many_fields_large_text_landscape_compare many_fields_scrolled_to_bottom_compare many_templates_large_text_landscape_compare many_templates_scrolled_to_bottom_compare portrait_compare portrait_cutout_compare templates_fit_viewport_compare templates_tab_large_text_compare

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

@david-allison

david-allison commented Oct 3, 2026 •

Copy link
Copy Markdown
Member Author

@ZornHadNoChoice FYI

@david-allison david-allison added Needs Author Reply Waiting for a reply from the original author and removed Needs Review labels Oct 3, 2026
@github-actions

github-actions Bot commented Oct 3, 2026 •

Copy link
Copy Markdown
Contributor

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

  • ChangeNoteTypeDialogScreenshotTest: 26 changes
All 26 changed screenshots

ChangeNoteTypeDialogScreenshotTest

  • desktop_fields_fit_viewport_compare.png
  • desktop_gesture_navigation_compare.png
  • desktop_landscape_compare.png
  • desktop_long_note_type_names_large_text_landscape_compare.png
  • desktop_long_note_type_names_large_text_narrow_portrait_compare.png
  • desktop_many_fields_large_text_landscape_compare.png
  • desktop_many_fields_scrolled_to_bottom_compare.png
  • desktop_many_templates_large_text_landscape_compare.png
  • desktop_many_templates_scrolled_to_bottom_compare.png
  • desktop_portrait_compare.png
  • desktop_portrait_cutout_compare.png
  • desktop_templates_fit_viewport_compare.png
  • desktop_templates_tab_large_text_compare.png
  • fields_fit_viewport_compare.png
  • gesture_navigation_compare.png
  • landscape_compare.png
  • long_note_type_names_large_text_landscape_compare.png
  • long_note_type_names_large_text_narrow_portrait_compare.png
  • many_fields_large_text_landscape_compare.png
  • many_fields_scrolled_to_bottom_compare.png
  • many_templates_large_text_landscape_compare.png
  • many_templates_scrolled_to_bottom_compare.png
  • portrait_compare.png
  • portrait_cutout_compare.png
  • templates_fit_viewport_compare.png
  • templates_tab_large_text_compare.png

@david-allison
david-allison force-pushed the air/19443-2-13eebe8e-b0f-5222de1e-a6e branch from a5e4930 to 9dd21fe Compare October 3, 2026 21:20
* Fix scrolling spacing so bottom rows are reachable
* Wrapped field/template don't overlap vertically
* Truncate note-type names
* Tabs handle large text

Fixes 21008

Assisted-by: GPT-6
@david-allison
david-allison force-pushed the air/19443-2-13eebe8e-b0f-5222de1e-a6e branch from 9dd21fe to 0b125c8 Compare October 3, 2026 21:32
@david-allison david-allison added Needs Review and removed Needs Author Reply Waiting for a reply from the original author labels Oct 3, 2026
@ZornHadNoChoice

Copy link
Copy Markdown
Collaborator
Some ideas:
  • The little downwards arrow takes up a lot of space.
  • In practice, it's more common to have just one of the two note types with a long name. (e.g. Basic → Basic (and reversed card)). Maybe we can make the boxes fit to the text. The only downside is losing symmetry.
  • We have both text "From"/"To" and an arrow. We can also sacrifice one of them to save space.

@david-allison

Copy link
Copy Markdown
Member Author

@ZornHadNoChoice thanks! Do you feel this can be merged as an incremental improvement?

The little downwards arrow takes up a lot of space.

Absolutely!


I think we should move the others to an issue.

  • Hiding From/to seems sensible if size is limited.

Removing symmetry: I think this would work, but it feels like overloading this with complexity

@david-allison david-allison added the Needs Author Reply Waiting for a reply from the original author label Oct 4, 2026
@ZornHadNoChoice

Copy link
Copy Markdown
Collaborator

Do you feel this can be merged as an incremental improvement?

Of course!

The design of 'To' already indicates it's clickable
and we could do with the space to render the note type
name

From a review comment on 21008

Assisted-by: GPT-6
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 Needs Review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Change note type screen doesn't have enough padding at the bottom

2 participants