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

Jumpiness from Twitter trying to reload trends widget #3

Open
dhasial opened this issue Aug 4, 2022 · 1 comment
Open

Jumpiness from Twitter trying to reload trends widget #3

dhasial opened this issue Aug 4, 2022 · 1 comment

Comments

@dhasial
Copy link

dhasial commented Aug 4, 2022

This is definitely an improvement over the current state of things, though there is still definitely some strange jumpy artefacts from Twitter repeatedly trying to force the widget to load in-between other tweets.

@lucaswerkmeister
Copy link
Owner

Hm, I didn’t realize how bad the jumpiness was in that use case. (I use another app, Cawbird, for reading the timeline, and Twitter Web mainly for notifications and DMs.)

I’m not sure how to solve it at the moment, though. Using CSS instead of JS would be more immediate, but as far as I can tell, the top-level trends element doesn’t have any classes that other elements don’t have, even if we assume that these classes are stable:

five sibling elements in the dev tools inspector, with the third one highlighted; all five elements have the same five classes, and the fourth element has all the classes of the third one (plus one extra class)

Once Firefox has proper :has() support, it might be possible to translate the current JS logic into CSS, I’ll have to look into that later. (Very recent versions have experimental support if you enable layout.css.has-selector.enabled, but it’s so incomplete that it’s not usable yet, I think.)

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

No branches or pull requests

2 participants