Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings
This repository was archived by the owner on Apr 26, 2024. It is now read-only.

Conversation

@natandias
Copy link
Contributor

Fix sidebar background color when using darkmode

@benhalverson benhalverson added the create-preview Generate preview on staging.nodejs.dev label Oct 4, 2020
@github-actions github-actions bot removed the create-preview Generate preview on staging.nodejs.dev label Oct 4, 2020
@github-actions
Copy link

github-actions bot commented Oct 4, 2020

Please find a preview at: https://staging.nodejs.dev/955/

@benhalverson benhalverson linked an issue Oct 4, 2020 that may be closed by this pull request
Copy link
Member

@benhalverson benhalverson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

Copy link
Member

@manishprivet manishprivet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Apart from the CSS, there's also this cross which is present from the mobile menu

image

@natandias natandias force-pushed the fixCssOnSidebarDarkMode branch from cc8f375 to 85a9c6a Compare October 4, 2020 06:08
@benhalverson
Copy link
Member

@manishprivet I haven't seen that issue before how did you get that?

@manishprivet
Copy link
Member

@benhalverson The cross is there in the screenshots you posted at #953, you might have missed that

@benhalverson
Copy link
Member

You're right. I totally didn't see that part of the issue. 🤦

@marcustisater
Copy link
Contributor

LGTM

Thanks for your contribution @natandias

@marcustisater
Copy link
Contributor

/preview

@github-actions
Copy link

github-actions bot commented Oct 4, 2020

Please find a preview at: https://staging.nodejs.dev/955/

@marcustisater marcustisater merged commit 0d7f22b into nodejs:master Oct 4, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CSS issue on /learn page sidebar navigation

4 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.