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

Sometimes scrolling (up) does not work #17593

Closed
HarHarLinks opened this issue Jun 8, 2021 · 20 comments
Closed

Sometimes scrolling (up) does not work #17593

HarHarLinks opened this issue Jun 8, 2021 · 20 comments
Assignees
Labels
A-Timeline O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect X-Cannot-Reproduce X-Regression

Comments

@HarHarLinks
Copy link
Contributor

HarHarLinks commented Jun 8, 2021

Description

Rolling my mouse wheel to go back in the timeline, but it doesn't work reliably?

Steps to reproduce

  • I'm visiting a couple public, federated rooms that were active over what is night for me
  • there's the jump to last unread button
  • try scrolling a bit
  • nothing
  • move mouse around for a bit, did element freeze?
  • try scrolling here and there
  • for some reason is scrolls when pointer is here, but not there
  • eventually it works normally
  • ???

could actually be a performance or backfilling issue?

Logs being sent: no

Version information

  • Platform: desktop

For the desktop app:

  • OS: Arch
  • Version: aur/element-desktop-nightly-bin 2021060802-1
@SimonBrandner SimonBrandner added A-Timeline S-Major Severely degrades major functionality or product features, with no satisfactory workaround X-Regression X-Release-Blocker labels Jun 8, 2021
@jryans jryans added the P1 label Jun 9, 2021
@germain-gg germain-gg self-assigned this Jun 9, 2021
@germain-gg
Copy link
Contributor

I have spent a bit of time trying to reproduce this issue without much luck so far.

If it does happen to you again, would you mind rageshaking as it might unveil some precious detail to debug

In the mean time a few questions to understand a bit more where that could come from

  • Did that happen right after you launched Element or did you left it on overnight?
  • Have you experienced it again after that time?

@HarHarLinks
Copy link
Contributor Author

  • it was shortly after launching
  • i'm keeping an eye on it, so far i don't think it happened again

my intuition is that it is a performance problem of element and/or my (own, somewhat slow) homeserver. it might have been backfilling history at the point, but unlucky content somehow that the spinner wasn't visible?

@rkfg
Copy link
Contributor

rkfg commented Jun 10, 2021

Happens to me as well on current develop. An easy way to reproduce when you have unread messages:

  • click "Jump to first unread message"
    2021-06-10_14-14-20
  • scroll works there
  • click "Scroll to most recent messages":
    2021-06-10_14-15-18
  • scroll locks up; move the mouse around while scrolling, after some time it randomly unlocks. Often locks up again when you scrolled to the last message.

Note: it only applies to scrolling using mouse wheel, it's always possible to drag the scroll bar handle but it's very inconvenient.

@germain-gg
Copy link
Contributor

Would you be able to share a bit more information regarding your spec (OS, Browser, version, ...)
And if possible a rageshake could potentially help to unveil some important bits to resolve this

Thank you

@rkfg
Copy link
Contributor

rkfg commented Jun 10, 2021

Using Brave 1.24.86 Chromium: 90.0.4430.212 (Official Build) (64-bit) on Debian testing. Reproducible on develop (https://develop.element.io/#/room/#matrix:matrix.org). Rageshake logs sent.

@HarHarLinks
Copy link
Contributor Author

thanks for helping track this down, rkfg

it happened again to me just now, rageshake submitted. notice I did not use the go to unread nor go to most recent buttons, just load the room. element has been running for several hours before it.

@rkfg
Copy link
Contributor

rkfg commented Jun 10, 2021

Yes, it happens sometimes even after switching between rooms. Then when I scroll it eventually start scrolling as if I'm trying to scroll through some invisible space (?) below the page. I just literally scroll for a long time, not even moving the mouse and the timeline starts moving after a while. Looks very weird.

@HarHarLinks
Copy link
Contributor Author

Currently happening to me, again in TWIM room. Clicked "scroll up to last unread" and it's stuck (without going there). Clicked submit feedback then debug log, but that isn't popping up either now i can't even click submit feedback.

It "unlocked", I scrolled up slightly and back down, stuck again.

@jryans
Copy link
Collaborator

jryans commented Jun 14, 2021

What about keyboard scrolling actions like page up / page down, do they still work in the locked case?

@rkfg
Copy link
Contributor

rkfg commented Jun 14, 2021

PageUp/PageDown do work while mouse wheel doesn't.

@jryans
Copy link
Collaborator

jryans commented Jun 15, 2021

Hmm, I wonder if this was triggered by the recent scrolling change in matrix-org/matrix-react-sdk#5823...

@jryans
Copy link
Collaborator

jryans commented Jun 15, 2021

I think I can replicate this when testing in Chrome Canary on macOS with a mouse wheel, though it's a bit of a subtle issue.

@HarHarLinks
Copy link
Contributor Author

Have you tested using a federated room? It happens in particular almost every time I open https://matrix.to/#/#thisweekinmatrix:matrix.org from my somewhat underpowered private homeserver (there is a backlog of unread messages that start out of screen for me 99% of cases).

I can confirm PageUp/PageDown do work while mouse wheel doesn't. Additionally, after using PageUp once, then using the wheel works. It is only when going back to the very bottom while there are still unread messages that it sticks there again. Scrolling back down too quickly will skip marking some unread messages as read by/for me (green line is supposedly where other participants would see my avatar icon?), and it stays sticky.

@jryans
Copy link
Collaborator

jryans commented Jun 15, 2021

Hmm, still having trouble reproducing this consistently... 😕 I don't think it would be related to federated rooms, server performance, or anything like that...

@HarHarLinks
Copy link
Contributor Author

HarHarLinks commented Jun 16, 2021

It has now for the first time happened on a room internal to my hs. As before, I need only open the room which has had enough activity while I was away to offer the jump to first unread button and scrolling is stuck.

BTW, the "ignore and just mark everything as read" button also works to un-stick things.

My platform as mentioned above is (Arch) Linux, but I will add that I use the desktop app nightly through xwayland on wayland (swaywm) and pipewire, perhaps this is some weird edge case with this setup.

@rkfg
Copy link
Contributor

rkfg commented Jun 16, 2021

I don't think it's platform-dependent. I'm on Debian with just plain Xorg and AwesomeWM and it still happens.

@mserajnik
Copy link

mserajnik commented Jun 21, 2021

This happens to me with my self-built Element Desktop (Electron v13.1.2) as well after updating to v1.7.31 (matrix-react-sdk v3.24.0). It did not happen with v1.7.29 (matrix-react-sdk v3.22.0), which I used previously. I'm using Arch Linux with herbstluftwm, though I also think it's platform-independent, giving the different reports here.

I can't reliably reproduce it, it seems to happen randomly after new messages have been posted. What I can say is that dragging the scrollbar still works when this happens and doing that usually makes scrolling with the mouse wheel work again too.
Actually, never mind, it behaves exactly as described here for me as well.

Hmm, I wonder if this was triggered by the recent scrolling change in matrix-org/matrix-react-sdk#5823...

Giving that this issue definitely didn't exist with Element v1.7.29 (or at least, never happened to me with that or any prior version, and I use Element extensively 8+ hours daily), that sounds like a reasonable guess to me, because this was merged after that release.

@novocaine
Copy link
Contributor

@HarHarLinks is this still happening?

@novocaine novocaine added S-Minor Impairs non-critical functionality or suitable workarounds exist X-Cannot-Reproduce O-Uncommon Most users are unlikely to come across this or unexpected workflow and removed P1 S-Major Severely degrades major functionality or product features, with no satisfactory workaround labels Nov 9, 2021
@rkfg
Copy link
Contributor

rkfg commented Nov 9, 2021

Can't reproduce here, at least with the steps I described above. And I don't remember it happening for a while already. Thanks!

@novocaine
Copy link
Contributor

Hrm okay, if anyone still sees it we will reopen. Closing for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Timeline O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect X-Cannot-Reproduce X-Regression
Projects
None yet
Development

No branches or pull requests

7 participants