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

[Docs]: Documentation Light/Dark Theme #79

Open
majamarijan opened this issue Oct 27, 2023 · 4 comments
Open

[Docs]: Documentation Light/Dark Theme #79

majamarijan opened this issue Oct 27, 2023 · 4 comments

Comments

@majamarijan
Copy link

Describe what's incorrect/missing in the documentation

I noticed some misbehavior in the functionality when changing themes in React Router Docs.
The theme is changing back or back and forth between dark/light when you:

  1. click on the theme icon and change the theme and then immediately click on the h2 (for example in the Tutorial page heading with the id="data-writes--html-forms") or
  2. clicking on the empty space between the main text and the sidebar with the headings

This is checked on both Chrome (Version 118.0.5993.117 ) and Firefox (Version 118.0.5993.117 )

changeThemeProblem

@timdorr timdorr transferred this issue from remix-run/react-router Oct 27, 2023
@brookslybrand
Copy link
Contributor

@majamarijan sorry no one ever got back to you on this

I've been spending some time cleaning up issues and updating the website. Is this still happening for you? I was unable to reproduce

@majamarijan
Copy link
Author

Hi @brookslybrand! Yes, it is better (Chrome and Firefox), but the issue still exists => when change the theme and then click on the bookmark.

Clicking on the blank space on the page doesn't affect the theme anymore. That's corrected well!

themeChangingProblem

@brookslybrand
Copy link
Contributor

@majamarijan, are you toggling your network speed or anything? I'm wondering if the action is getting cancelled before the cookie has a chance to update?

@majamarijan
Copy link
Author

majamarijan commented Apr 1, 2024

@brookslybrand , no toggling. Yeah, I think that's the problem. If the theme is changed, and I click on the bookmark in a half a second or less after changing the theme, then this happens. Or, I change the Theme and then double click on the bookmark (like # Tutorial). I do the same on the MDN website and this is not happening (they have the same theme button that opens a dialog).

Yet, it is now much better than before.

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