search

jgero / roesena-app

Cross platform Web-App with user-created content and event manager
https://rösena.de/
0 stars 1 forks source link

image carousel #308

Closed jgero closed 3 years ago

jgero commented 3 years ago

On image detail view add buttons for "next image" and "previous image". It would be fine to just enable navigating through the images on the current page like this. For going to further pages the buttons could navigate back to the overview page and open the next page there.

jgero commented 3 years ago

Maybe a complete revamp of the image cards would be appropriate.

An idea would be to not use cards at all, but to only show the image. Then when touching on or hovering over the image some kind of overlay menu could appear with buttons to open the detail view or the carousel and show the tags. This overlay should disappear when the element is not hovered anymore or loses focus.

The carousel would then just be the higher quality image as popup in the front with faded out background and the back and forth buttons.

jgero commented 3 years ago

Depends on #310, should not be implemented on old store architecture.

jgero commented 3 years ago

Add a "has next event" and "has previous event" selector to know when to grey out the previous and next button. When there is no previous or next event on the page for the carousel to display show a "go to next/previous page" button, maybe also by selector.

When clicking the carousel button the clicked image should be loaded into the active image object of the store and some "is carousel open" value could be toggled. Then actions with "go next/prev in carousel" and "go next page" would be very easy to implement. "go next in carousel" doesn't even need to fetch anything from firestore, just take the id and load the full image.

jgero commented 3 years ago

It is actually way easier, will create PR with it shortly.