Photos: NC 20.0.3: Scrolling not possible on big screen's

Created on 11 Dec 2020  路  6Comments  路  Source: nextcloud/photos

Describe the bug
When loading the photos app 35 pictures are shown. As I have a big screen, they do not fill the screen and no scrollbar is shown. Therefore scrolling down to load more pictures doesn't work. If I resize the browser window until the window is field scrolling down and loading more pictures is working nicely.

To Reproduce
Steps to reproduce the behavior:

  1. open Photos app
  2. wait
  3. wait longer
  4. Still only 35 pictures and no way to scroll.

Expected behavior
If the app should load pictures until the screen is full.

Screenshots

grafik

Desktop (please complete the following information):

  • OS: OpenSuse
  • Browser Firefox

Browser log

Open your console, reload your page and/or do the action leading to this issue and copy/paste the log in this thread.
grafik

Additional context
Add any other context about the problem here.

3. to review bug papercut regression

Most helpful comment

@skjnldsv probably yes, also we should load a number of photos that fillls at least twice the height of the view otherwise it doesn't feel smooth at all

35 per batch is not enough when you have a big screen

All 6 comments

cc @Mikescops we don't check for reached end on load I assume?

PS: @tacruc, nice dog! :heart_eyes:

@skjnldsv probably yes, also we should load a number of photos that fillls at least twice the height of the view otherwise it doesn't feel smooth at all

35 per batch is not enough when you have a big screen

We load xx lines, no?
I remember we talked about that. Don't recall if we did the change or not

We load 7 lines (x 5) but it shouldn't be that static

I'm fixing this in the grid lib

Was this page helpful?
0 / 5 - 0 ratings

Related issues

Skeebopstop picture Skeebopstop  路  9Comments

skjnldsv picture skjnldsv  路  3Comments

iwanttobefreak picture iwanttobefreak  路  7Comments

tillwf picture tillwf  路  14Comments

Mikescops picture Mikescops  路  5Comments