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

Scrollbar Behaviour #90

Closed
dfoxg opened this issue Oct 5, 2023 · 7 comments
Closed

Scrollbar Behaviour #90

dfoxg opened this issue Oct 5, 2023 · 7 comments
Labels
bug Something isn't working

Comments

@dfoxg
Copy link

dfoxg commented Oct 5, 2023

What is wrong?
The scroll bar has currently a weird behaviour, which shows in two use cases:

  • the bar isn't visible always, at least on my phone it's really tricky to access it
  • I got one picture from 2004 in my library, the other images are from 2015+. The scroll bar takes the half of my screen to navigate through my library, even there are no images:
  • when using the scroll bar to scrolling into a date, all images in the future relative to the scrolled date aren't visible. You have to do a new scroll to get to those images.

Screenshots
If applicable, add screenshots to help explain your problem.

Device (please complete the following information):

  • Model: Google Pixel 7
  • Android version: 14
@dfoxg dfoxg added the bug Something isn't working label Oct 5, 2023
@dfoxg
Copy link
Author

dfoxg commented Oct 5, 2023

photoprism2.mp4

@ippocratis
Copy link
Contributor

ippocratis commented Oct 5, 2023

when using the scroll bar to scrolling into a date, all images in the future relative to the scrolled date aren't visible. You have to do a new scroll to get to those images.

#53

#54

@Radiokot
Copy link
Owner

Radiokot commented Oct 6, 2023

Thanks, @ippocratis , I have nothing to add here 😌

@Radiokot Radiokot closed this as completed Oct 6, 2023
@dfoxg
Copy link
Author

dfoxg commented Oct 6, 2023

Sorry, i don't get why this got closed. I got it, that my third point is already answered as won't fix (thanks for linking that one @ippocratis ) but the other two points are still open, aren't they?

  • the bar isn't visible always, at least on my phone it's really tricky to access it
  • I got one picture from 2004 in my library, the other images are from 2015+. The scroll bar takes the half of my screen to navigate through my library, even there are no images

@ippocratis
Copy link
Contributor

ippocratis commented Oct 6, 2023

@dfoxg

  • the bar isn't visible always, at least on my phone it's really tricky to access it

True the bar is not always visible it only appears after you start sliding it , but it doesn't need to be visible to start sliding it, just touch the side of your screen and start gliding your finger down.

  • I got one picture from 2004 in my library, the other images are from 2015+. The scroll bar takes the half of my screen to navigate through my library, even there are no images

In my case I have pics all the way down to 2002 in my library some years have significantly more photos than others.
As far as I understant the app takes the more old photo and the more recent photo and seperates the side of the screen evenly not taking accound the number of pics each year contains.
I guess it has to do with how the kotlin module used for this function handles the timeline , idk Oleg can obviously explain how it works better and if it's doable to change it's behavior.

@Radiokot
Copy link
Owner

Radiokot commented Oct 6, 2023

@dfoxg The main thing to understand about the scrollbar on the gallery screen is that it is not really a scroll bar, but rather a control to quickly jump to a specific month. The gallery has no idea how many photos you have and how they are distributed in time. Only the newest and the oldest dates for the search query are loaded, and the scroll steps (month bubbles) are created for each month no matter how many photos there are or if there are any at all.
This is why the scroll handle can't follow you as you scroll, and so it looks really wrong when always kept visible. This is also the reason why in your case it has steps for all the months from 2004 to 2015.
It is also not possible to scroll up from a specific month, by design. It looks doable, but tricky. Michael, the PhotoPrism main developer, said they are going to add the proper timeline implementation, so I'd rather wait for them to do it than tweak the current gallery implementation.

@Radiokot Radiokot pinned this issue Oct 6, 2023
@dfoxg
Copy link
Author

dfoxg commented Oct 6, 2023

Now it makes sence, thanks for the deep dive!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants