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.

Sign up for GitHub

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

Jump to bottom

... #15233

Closed
ghost opened this issue Apr 25, 2019 · 13 comments · Fixed by #15462
Closed

... #15233

ghost opened this issue Apr 25, 2019 · 13 comments · Fixed by #15462
Labels
3. to review Waiting for reviews bug feature: files high
Milestone
Nextcloud 17

Comments

@ghost
Copy link

ghost commented Apr 25, 2019 •
edited by ghost
Loading

-deleted-
@kesselb
Copy link
Contributor

kesselb commented Apr 25, 2019 •
edited
Loading

Looks interesting. I'm not sure about this error page? @rullzer @ChristophWurst

Do you have Opera Turbo enabled? If so could you try without?

@koarlmichael
Copy link

At first I thought it was just a bug in the gallery app, but it also happens in the webview.

nextcloud/gallery#515

@DaCryptor

This comment has been minimized.

Sign in to view
@rullzer
Copy link
Member

rullzer commented Apr 26, 2019

What phone are you actually using and what browser? All works as expected here in chromium and firefox mobile.

@skjnldsv @ChristophWurst @juliushaertl any of you in a position to test opera?

@koarlmichael
Copy link

koarlmichael commented Apr 26, 2019 •
edited
Loading

Can confirm the bug on:
OnePlus 3, Android 8.0.0, Chrome
Moto G5, Android 8.1.0, Chrome and Firefox
Huawei P9 lite, Android 7, Chrome and Firefox

Scroll only works in the yellow Area:

pic

@DaCryptor
Copy link

Bug confirmed on BQ Aquaris X Pro, Android 8.1, Firefox Mobile

@AndyFx12
Copy link

AndyFx12 commented Apr 28, 2019 •
edited
Loading

Bug confirmed on MotoG4 Play, Android 7.1 and Lenovo TAB2 A10-70F, Android 6, Chrome and Firefox
Edit: Some folders without write access work as expected.

@ChristophWurst
Copy link
Member

Bug confirmed on FF for Android

@ghost
Copy link

ghost commented May 4, 2019

I was able to reproduce the issue using chrome dev tools in mobile view.

The behavior is caused by the css class .ui-draggable-handle that has "tourch-action" set to "none".
If i remove it scrolling works as expected. Also "touch-action" is not disabled on the first column with the checkboxes. (see comment made by @Gatak )

If you remove this property scrolling works:
grafik

For a permanent solution further investigation is required to find out why "touch-action" has been added.

@skjnldsv
Copy link
Member

skjnldsv commented May 4, 2019

My uess is the move of our jquery library into our webpack config? @ChristophWurst
Maybe we added a new css change from one of our dependency? this seems related to jquery-ui (yeaaaah) :)

@skjnldsv skjnldsv added 1. to develop Accepted and waiting to be taken care of bug high feature: files labels May 4, 2019
@ChristophWurst
Copy link
Member

My uess is the move of our jquery library into our webpack config? @ChristophWurst

The jquery libs haven't changed much with the webpack bundling. But we updated jquery-ui a bit: #11718

@skjnldsv
Copy link
Member

skjnldsv commented May 6, 2019 •
edited
Loading

Yep, this is it: 1.12.1 - 1.11.0-beta.1
jquery/jquery-ui@28310ff#diff-e8b8fd48040d8abc0af402d340bb1b05R11

@skjnldsv skjnldsv mentioned this issue May 9, 2019
Merged
@skjnldsv skjnldsv added 3. to review Waiting for reviews and removed 1. to develop Accepted and waiting to be taken care of labels May 9, 2019
@skjnldsv
Copy link
Member

skjnldsv commented May 9, 2019

Fix in #15462

@MorrisJobke MorrisJobke added this to the Nextcloud 17 milestone May 9, 2019
@ghost ghost changed the title Scrolling doesn't work properly on android browsers in NC ... Jul 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
3. to review Waiting for reviews bug feature: files high
Projects
None yet
Milestone
Nextcloud 17
Development

Successfully merging a pull request may close this issue.

Fix mobile vertical scrolling nextcloud/server
8 participants
@rullzer @MorrisJobke @ChristophWurst @kesselb @DaCryptor @skjnldsv @koarlmichael @AndyFx12