Skip to content

Comments

chore: convert NoteTypesAdapter to ViewBinding#20076

Merged
BrayanDSO merged 1 commit intoankidroid:mainfrom
david-allison:vb-NotetypeAdapter
Jan 17, 2026
Merged

chore: convert NoteTypesAdapter to ViewBinding#20076
BrayanDSO merged 1 commit intoankidroid:mainfrom
david-allison:vb-NotetypeAdapter

Conversation

@david-allison
Copy link
Member

I did this one from scratch due to 7c11d01
With a few renames from 771bee1

How Has This Been Tested?

Screen opens and is usable on my Pixel 9 Pro

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

@lukstbit lukstbit added Needs Author Reply Waiting for a reply from the original author Needs Second Approval Has one approval, one more approval to merge and removed Needs Review labels Jan 16, 2026
@david-allison david-allison removed the Needs Author Reply Waiting for a reply from the original author label Jan 16, 2026
@BrayanDSO BrayanDSO added this pull request to the merge queue Jan 17, 2026
@BrayanDSO BrayanDSO added 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 Jan 17, 2026
Merged via the queue into ankidroid:main with commit 2bfe77f Jan 17, 2026
15 checks passed
@github-actions github-actions bot added this to the 2.24 release milestone Jan 17, 2026
@github-actions github-actions bot removed the Pending Merge Things with approval that are waiting future merge (e.g. targets a future release, CI wait, etc) label Jan 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants