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
feat(web): ascending order for slideshow #7502
Conversation
4aebbd9
to
9dce3f6
Compare
Looks good and works as expected. I'm only not so sure about how the order options are presented to the user. Maybe three buttons or a radio input with |
I agree. What do you think about using a dropdown menu (like the one in the album list to select the order) ? |
A dropdown menu would be an improvement, but since there are only three options a radio input has my preference. But I'm fine with both. |
9764203
to
373a16f
Compare
Yeah, I like dropdown more, it's easier to adapt if we add new navigation mechanisms in the future |
web/src/lib/components/shared-components/settings/setting-dropdown.svelte
Outdated
Show resolved
Hide resolved
Is hiding the dropdown text on small screens intentional? I'd say just show the text, there's enough space. |
Nope, it should be fixed |
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.
LGTM
…ng-order-slideshow
What's changed
This PR adds an ascending / descending mode for the slideshow
Screenshots
2024-02-28.18-47-37-2.mp4