-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Menu handled by DeckPicker instead of StudyOptionsFragment #17099
Conversation
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.
The following should not be visible if the collection is empty [fragment is invisible]:
- Rename Deck
- Delete Deck
- Custom Study
- Deck Options (I think)
- Export (unsure, but seems sensible)
- Search Decks (🪲 I believe this should only be shown if 10 decks are in the collection)
val searchIcon = decks.count() >= 10
A unit test, or @NeedsTest
should be added for these cases
Hello 👋, this PR has had no activity for more than 2 weeks and needs a reply from the author. If you think this is a mistake please comment and ping a maintainer to get this merged ASAP! Thanks for contributing! You have 7 days until this gets closed automatically |
Hello 👋, this PR has had no activity for more than 2 weeks and needs a reply from the author. If you think this is a mistake please comment and ping a maintainer to get this merged ASAP! Thanks for contributing! You have 7 days until this gets closed automatically |
|
Purpose / Description
DeckPicker should handle menu responsibility instead of StudyOptionsFramgment (Tablet)
I reverted some changes from PR #16425
Fixes
How Has This Been Tested?
Emulator
deckpicker_menu.mp4
Checklist
Please, go through these checks before submitting the PR.