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

Theme toggle :hover contrast issue while navbar style: 'dark' #8587

Closed
3 of 7 tasks
dewanshDT opened this issue Jan 26, 2023 · 4 comments · Fixed by #8615
Closed
3 of 7 tasks

Theme toggle :hover contrast issue while navbar style: 'dark' #8587

dewanshDT opened this issue Jan 26, 2023 · 4 comments · Fixed by #8615
Labels
bug An error in the Docusaurus core causing instability or issues with its execution domain: theme Related to the default theme components

Comments

@dewanshDT
Copy link
Contributor

Have you read the Contributing Guidelines on issues?

Prerequisites

  • I'm using the latest version of Docusaurus.
  • I have tried the npm run clear or yarn clear command.
  • I have tried rm -rf node_modules yarn.lock package-lock.json and re-installing packages.
  • I have tried creating a repro with https://new.docusaurus.io.
  • I have read the console error message carefully (if applicable).

Description

while hovering on the theme toggle it still shows light background with light foreground color. this is a design flaw and leads to poor accessibility of the application.
fixing this will improve the viewability of the button by more users and improve the accessibility of the application.

check the expected and current behaviors of the button below:

Reproducible demo

No response

Steps to reproduce

Step 1

set the navbar style to 'dark' in docusaurus.config.js file

navbar: {
        title: "My Site",
        style: "dark",

Step 2

while in light mode hover on the theme toggle

Expected behavior

docusaurauscorrect

Actual behavior

docusauraus

Your environment

  • Docusaurus version used: latest
  • Environment name and version (e.g. Chrome 89, Node.js 16.4): Chorme 89
  • Operating system and version (e.g. Ubuntu 20.04.2 LTS): Windows

Self-service

  • I'd be willing to fix this bug myself.
@dewanshDT dewanshDT added bug An error in the Docusaurus core causing instability or issues with its execution status: needs triage This issue has not been triaged by maintainers labels Jan 26, 2023
@Josh-Cena
Copy link
Collaborator

Josh-Cena commented Jan 27, 2023

Can't reproduce this on https://docusaurus.io/ using Windows 7 and Chrome 89. Can you see the same behavior on our site, or does it only happen with your site?

while navbar style: 'dark'

Oh I see. Thanks for reporting!

@Josh-Cena Josh-Cena added domain: theme Related to the default theme components and removed status: needs triage This issue has not been triaged by maintainers labels Jan 27, 2023
@dewanshDT
Copy link
Contributor Author

hey can I work on this?

@slorber
Copy link
Collaborator

slorber commented Feb 1, 2023

Yes, you don't need to ask for such issues, first good PR will be merged @dewanshDT

@dewanshDT
Copy link
Contributor Author

hey @slorber @Josh-Cena I've fixed this problem and created a PR 😁

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug An error in the Docusaurus core causing instability or issues with its execution domain: theme Related to the default theme components
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants