-
Notifications
You must be signed in to change notification settings - Fork 186
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
Scrolling buggy, jumping around #2498
Comments
This is not enough info to debug the issue. |
When you scroll down in the "unread articles" the page reloads and then jumps to a new location. |
And what is the actual version of news? Because "newest" means nothing without knowing what release channel you are on. And what is in the browser error log? You know, the questions in the issue template... |
News 25.0.0 alpha2 There is no error log. |
Since we haven't seen any other reports so far and also during development we didn't see that, I would suggest that you check if maybe some addon is causing this. I think you can start chrome in safe mode. |
Same here, tested with the mobile versions of Chrome based Kiwi, Samsung Internet, and Firefox. Kiwi lets me modify the concerning HTML elements: Removing 'class="virtual-scroll"' from the additional div between '.feed-item-display-container' and the '.container-window', which holds the feed-item-rows, changes the behavior back to smooth scrolling. |
It's not the browser or operating system. Emails in Nextcloud can be scrolled through very cleanly, but not news //edit |
The mail app doesn't use the VirtualScroll.vue component which seems to be problematic. Bookmarks does and there are problems with scrolling, too, although the structure of the items is simplier. Maybe there is a miscalculation in the height of the items or the list. |
I'm also experiencing this issue both when scrolling via mousewheel (hope that can be seen as well) and also when trying to drag the scrollbar: https://github.com/nextcloud/news/assets/414984/8e8f1694-00ca-444b-999d-46af8a8b22ad Edit: Adding another this time more obvious example you experience when using the mouse scroll wheel, slowly scrolling always just one tick, you suddenly see a jump in the whole list which makes it extremly disturbing to use/read: https://github.com/nextcloud/news/assets/414984/5c898d02-419d-4a13-83f1-06d8c84cc83c |
The issue is browser dependent. I have the jumping issue with FF 121.0 on Windows 10/11, but Opera 105 (Chromium 119) on Windows 10, FF 115.6esr on Linux work fine. |
This comment was marked as duplicate.
This comment was marked as duplicate.
This comment has been minimized.
This comment has been minimized.
No need for further confirmations, someone needs to analyze the code, fix it or change the component. The code was borrowed from the bookmarks app. |
While I haven't looked into the Vue code, I did check the resulting DOM and the changes to it. <div data-v-1a16f31a="" data-v-bd60c07c="" class="virtual-scroll">
<div data-v-1a16f31a="" class="upper-padding" style="height: 1426px;"></div>
<div data-v-1a16f31a="" class="container-window" style="height: 966px;"><!-- actual data here --></div>
<div data-v-1a16f31a="" class="lower-padding" style="height: 3542px;"></div>
</div> While scrolling, the I don't know how browsers calculate their scrolling animations specifically, but changing the geometry during the animation sounds like it's going to trip them up if they don't add extra safe guards against that. Case in point: When I'm at the very top of the list, a single click of the wheel doesn't trigger a change in those |
I tried to integrate https://github.com/Akryum/vue-virtual-scroller/tree/v1/packages/vue-virtual-scroller, but I wasn't able to make it work yet. |
IMPORTANT
Read and tick the following checkbox after you have created the issue or place an x inside the brackets ;)
Explain the Problem
The scrolling behavior is not manageable.
Neither with a scroll bar nor with a mouse wheel
In addition, automatic reading when scrolling is missing.
Currently unusable.
System Information
The text was updated successfully, but these errors were encountered: