Skip to content
This repository has been archived by the owner on Nov 1, 2020. It is now read-only.

Bind scroll event to window #462

Merged
merged 1 commit into from
Aug 24, 2018
Merged

Bind scroll event to window #462

merged 1 commit into from
Aug 24, 2018

Conversation

chrisingenhaag
Copy link
Contributor

@chrisingenhaag chrisingenhaag commented Aug 19, 2018

Fixes: #459

Licence: MIT or AGPL

Description

Like in the filelist app binding the scroll event to $(window) works. This fixes #459 and enables app loading more than the initially loaded images.

Tests

Test plan

Open a folder in gallery with many pictures ( > 100). Try to scroll down and check if all images are loaded.

Tested on

  • Mac/Firefox
  • Mac/Chrome
  • Mac/Safari
  • Android 8/Chrome
  • iOS/Chrome

Like in the filelist app binding the scroll event to $(window) works. This fixes #459 and enables app loading more than the initially loaded images.

Signed-off-by: Christian Ingenhaag <[email protected]>
@skjnldsv skjnldsv added bug Something isn't working 3. to review Waiting for reviews high High priority labels Aug 24, 2018
@skjnldsv skjnldsv added this to the Nextcloud 14 milestone Aug 24, 2018
@MorrisJobke MorrisJobke merged commit 97e56fb into nextcloud:master Aug 24, 2018
@chrisingenhaag chrisingenhaag deleted the patch-1 branch August 26, 2018 08:01
@MorrisJobke MorrisJobke mentioned this pull request Aug 26, 2018
6 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
3. to review Waiting for reviews bug Something isn't working high High priority
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Nextcloud 14.0.0 Beta 3 - The gallery does not load the entire list of files
3 participants