Skip to content

hoverover on the new roomlist is very sluggish and lags the cursor by about 50-100ms #14327

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

Closed
ara4n opened this issue Jul 4, 2020 · 3 comments

Comments

@ara4n
Copy link
Member

ara4n commented Jul 4, 2020

there's also a 1px gap between the roomtiles which might accentuate this effect, given the list cycles through three different states (top highlight, no highlight, bottom highlight) as you move from top to bottom rather than going straight from top highlight to bottom highlight for adjacent roomtiles.

I'm assuming we're rerendering the whole control to show the highlight rather than just the roomtile in question?

https://matrix.to/#/!NTUKVQdPRwdQhaunBn:matrix.org/$osH6nyWINQfcwA80kColxW_EN1zQJP38_WacMsNVXxo?via=matrix.org&via=t2l.io&via=riot.ovh "sluggish hoverover.mov" shows this in action on my roomlist.

@ara4n ara4n changed the title there's a 1px gap for hoverover between new roomlist room tiles hoverover on the new roomlist is very sluggish and lags the cursor by about 50-100ms Jul 4, 2020
@turt2live
Copy link
Member

The gap feels a bit required as without it (either through elimination or smarter padding) the list feels a bit broken. The effect you're seeing is likely performance related where the browser is dropping fps (and therefore layout updates) due to the thing screaming in the background.

@turt2live
Copy link
Member

It looks like this is basically #14035 causing fps drops rather than anything spectacular. Will leave this open to double check though.

@turt2live turt2live self-assigned this Jul 10, 2020
@turt2live
Copy link
Member

This looks fixed now. If it continues to be a problem, please re-report.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants