Skip to content
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

Show a carousel view containing randomly picked archives from the current search #527

Closed
Difegue opened this issue Oct 20, 2021 · 3 comments · Fixed by #535
Closed

Show a carousel view containing randomly picked archives from the current search #527

Difegue opened this issue Oct 20, 2021 · 3 comments · Fixed by #535
Assignees
Labels

Comments

@Difegue
Copy link
Owner

Difegue commented Oct 20, 2021

Needs #515.

The goal here is to replace the "Random Archive button" by something that's more visible and gives a bit more choice to the user.

image
(dodgy html mockup, needs much smaller pictures in a real implementation)

  • The carousel should feature 5 to 10 archives from the current search/category
  • It should have a refresh button to get a new batch of random archives (not shown on mockup)
  • It should be collapsible in order to take up minimal space against the usual list (ideally, as much vertical space as the current Random Archive/Display Options buttons take)
  • Ideally, it should be extended to also feature stuff from the other filter buttons so those can be removed from the UI: New Archives, Untagged Archives, Currently Reading/On Deck (new feature!)
@Difegue Difegue added enhancement archive index user survey Issues matching common user requests. labels Oct 20, 2021
@Difegue Difegue added this to the 2021 User Survey Requests milestone Oct 20, 2021
@Difegue
Copy link
Owner Author

Difegue commented Oct 20, 2021

Putting up a bounty link for this due to a demand on the Discord server:

@rysolv-bot
Copy link

An anonymous user has contributed $100.00 to this issue on Rysolv.

The total bounty is now $100.00. Solve this issue on Rysolv to earn this bounty.

@Difegue Difegue self-assigned this Nov 10, 2021
@Difegue Difegue pinned this issue Nov 10, 2021
Difegue added a commit that referenced this issue Nov 10, 2021
@Difegue
Copy link
Owner Author

Difegue commented Nov 10, 2021

The base carousel has been implemented as of the latest commit(random only for now), alongside a bunch of index improvements :

image

I'm reusing the same components as the thumbnail view for now, but that's open for improvements later on if I feel like the carousel is too tall. (it seems fine as-is! might change my mind with further usage)

@Difegue Difegue mentioned this issue Nov 11, 2021
@Difegue Difegue linked a pull request Nov 11, 2021 that will close this issue
@Difegue Difegue closed this as completed Nov 22, 2021
@Difegue Difegue unpinned this issue Nov 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants