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

Documentation is flickering on light theme #49648

Closed
pravic opened this issue Apr 4, 2018 · 7 comments
Closed

Documentation is flickering on light theme #49648

pravic opened this issue Apr 4, 2018 · 7 comments
Labels
T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue.

Comments

@pravic
Copy link
Contributor

pravic commented Apr 4, 2018

I have noticed it with nightly and now it is in stable documentation.

It looks like a dark theme has become the default now, but users of the light one now are experiencing some very annoying flickering on each page load: it starts to load in dark theme and then switches to the light one.

Funny enough, it only happens with Chrome-based browsers (the old good Opera does not have this issue).

P.S. Firefox (Windows, Ubuntu) does not have such issue, too.

Is there a way to reduce such behavior? May be switch to the preferred theme a bit earlier?

@rust-lang/docs

@steveklabnik
Copy link
Member

It looks like a dark theme has become the default now,

This should not be true. That said, what you're describing sounds bad... @GuillaumeGomez ?

@GuillaumeGomez
Copy link
Member

It's not supposed to. By default, it is the light one which is selected.

@pravic
Copy link
Contributor Author

pravic commented Apr 5, 2018

Am I the only one who can see it? I tried different browsers and VMs, same here.

I'll check with devtools, what's going on there.

@pravic pravic changed the title Documentation flickering on light theme Documentation is flickering on light theme Apr 5, 2018
@pravic
Copy link
Contributor Author

pravic commented Apr 5, 2018

Well. The problem was with an unconditional theme switch, which was fixed in #48831. The current nightly works fine.

I doubt that that fix is possible to backport to the stable documentation, but may be at least in beta?

@steveklabnik
Copy link
Member

Yeah, stable is impossible, but beta seems good.

@QuietMisdreavus QuietMisdreavus added the T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. label May 15, 2018
@QuietMisdreavus
Copy link
Member

As the fix PR was merged and backported (and a release has happened as well), i'm calling this issue fixed. Feel free to reopen if you disagree!

@pravic
Copy link
Contributor Author

pravic commented May 15, 2018

Yes, stable works good. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue.
Projects
None yet
Development

No branches or pull requests

4 participants