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

Navbar hideOnScroll: true shows light mode logo in dark mode #2430

Closed
nicholas-ochoa opened this issue Mar 20, 2020 · 1 comment · Fixed by #2434
Closed

Navbar hideOnScroll: true shows light mode logo in dark mode #2430

nicholas-ochoa opened this issue Mar 20, 2020 · 1 comment · Fixed by #2434
Labels
bug An error in the Docusaurus core causing instability or issues with its execution

Comments

@nicholas-ochoa
Copy link

🐛 Bug Report

I have the following settings configured:

themeConfig.navbar.hideOnScroll = true
themeConfig.navbar.logo.src = 'logo.png'
themeConfig.navbar.logo.srcDark = 'logo_dark.png'

When I scroll down the page in dark mode, the nav bar disappears as expected but the logo above the sidebar is the normal logo and not the logo specified in srcDark.

Have you read the Contributing Guidelines on issues?

Yes

To Reproduce

(Write your steps here:)

  1. Configure a dark mode logo that differs from the normal logo
  2. Set hideOnScroll to true
  3. Scroll a page in dark mode

Expected behavior

Dark mode logo should be displayed above side bar.

Actual Behavior

Default (light mode) logo is displayed incorrectly above sidebar when scrolling. Navbar has the correct logo.

Your Environment

  • Docusaurus version used: 2.0.0-alpha.48
  • Environment name and version (e.g. Chrome 78.0.3904.108, Node.js 10.17.0): Node 12.13.0
  • Operating system and version (desktop or mobile): Windows 10
@nicholas-ochoa nicholas-ochoa 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 Mar 20, 2020
@lex111
Copy link
Contributor

lex111 commented Mar 20, 2020

Oops, good catch, thanks, this will be fixed in the next release (end of March)!

@lex111 lex111 added v2 and removed status: needs triage This issue has not been triaged by maintainers labels Mar 20, 2020
@lex111 lex111 added this to the v2.0.0-alpha.49 milestone Mar 20, 2020
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
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants