Skip to content
This repository has been archived by the owner on Feb 14, 2025. It is now read-only.

Jumpiness from Twitter trying to reload trends widget #3

Closed
dhasial opened this issue Aug 4, 2022 · 2 comments
Closed

Jumpiness from Twitter trying to reload trends widget #3

dhasial opened this issue Aug 4, 2022 · 2 comments

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.)

@lucaswerkmeister
Copy link
Owner

Gonna close this issue now per #5 (comment).

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

No branches or pull requests

2 participants