Skip to content
This repository was archived by the owner on Apr 26, 2024. It is now read-only.

Learn-page: fix sidebar background on darkmode#955

Merged
marcustisater merged 3 commits intonodejs:masterfrom
natandias:fixCssOnSidebarDarkMode
Oct 4, 2020
Merged

Learn-page: fix sidebar background on darkmode#955
marcustisater merged 3 commits intonodejs:masterfrom
natandias:fixCssOnSidebarDarkMode

Conversation

@natandias
Copy link
Copy Markdown
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
Copy Markdown

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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Member

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

@manishprivet
Copy link
Copy Markdown
Member

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

@benhalverson
Copy link
Copy Markdown
Member

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

@marcustisater
Copy link
Copy Markdown
Contributor

LGTM

Thanks for your contribution @natandias

@marcustisater
Copy link
Copy Markdown
Contributor

/preview

@github-actions
Copy link
Copy Markdown

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