-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Feature: Bookmark Categories #6035
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
Feature: Bookmark Categories #6035
Conversation
Hi, I have few questions,
Obviously that depends on the size of data but do we expect the bookmarks to be in large amounts?
Thanks :) |
Show when logged out: yes Paging, sorting, searching: Great idea, though the added complexity would currently not be justified. Would you mind creating a new issue and labeling it as enhancement, low priority? Thanks a lot! 🙂 |
Great! I also kept it that way.
I will shortly, outside right now (commenting from my mobile). :) |
app/src/main/java/fr/free/nrw/commons/category/CategoryDetailsViewModel.kt
Show resolved
Hide resolved
Signed-off-by: parneet-guraya <gurayaparneet@gmail.com>
Signed-off-by: parneet-guraya <gurayaparneet@gmail.com>
Signed-off-by: parneet-guraya <gurayaparneet@gmail.com>
Signed-off-by: parneet-guraya <gurayaparneet@gmail.com>
af95293
to
d1b6dc2
Compare
Docs added :) |
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.
Working great, thanks a lot!
@nicolas-raoul This closes this #5318 issue as well. Take a look :) |
Fixes #5922
What changes did you make and why?
Tests performed (required)
Tested betaDebug on Oneplus 9RT 5G (Android 14)
Screenshots (for UI changes only)
Record_2024-12-16-01-38-04.mp4