-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Note editor toolbar new item string as icon #10290
Conversation
AnkiDroid/src/main/java/com/ichi2/anki/noteeditor/CustomToolbarButton.kt
Outdated
Show resolved
Hide resolved
93190ef
to
47b8340
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! Main question is about the user losing data, which should be handled by
AnkiDroid/src/main/java/com/ichi2/anki/noteeditor/CustomToolbarButton.kt
Outdated
Show resolved
Hide resolved
AnkiDroid/src/main/java/com/ichi2/anki/noteeditor/CustomToolbarButton.kt
Show resolved
Hide resolved
8dc574f
to
a35fdf7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, thank you!
Pull Request template
Purpose / Description
The Note Editor toolbar add new items as number from 1...N, so I created this PR for adding string / single character as icon for the new items. It helps in recognize the button instead of memorizing button action with the number.
Fixes
Fixes #9077
Approach
icon
afterindex
.index, prefix and suffix
.Now when new buttons added it saved to SharedPrefs as
index, icon, prefix and suffix
How Has This Been Tested?
Test on Emulator on device
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration (SDK version(s), emulator or physical, etc)
Learning (optional, can help others)
Describe the research stage
Links to blog posts, patterns, libraries or addons used to solve this problem
Checklist
Please, go through these checks before submitting the PR.
if
statements)