Skip to content

feat(v2): add ability hide dark mode toggle#1956

Merged
yangshun merged 1 commit intofacebook:masterfrom
lex111:disable-dark-mode
Nov 11, 2019
Merged

feat(v2): add ability hide dark mode toggle#1956
yangshun merged 1 commit intofacebook:masterfrom
lex111:disable-dark-mode

Conversation

@lex111
Copy link
Copy Markdown
Contributor

@lex111 lex111 commented Nov 10, 2019

Motivation

See t-regx/t-regx.github.io#14 (comment)

Have you read the Contributing Guidelines on pull requests?

Yes

Test Plan

When themeConfig.disableDarkMode = false, dark mode toggle will not be rendered.

image

Related PRs

(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)

@facebook-github-bot facebook-github-bot added the CLA Signed Signed Facebook CLA label Nov 10, 2019
@endiliey endiliey added the pr: new feature This PR adds a new API or behavior. label Nov 10, 2019
Copy link
Copy Markdown
Contributor

@endiliey endiliey left a comment

Choose a reason for hiding this comment

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

@docusaurus-bot
Copy link
Copy Markdown
Contributor

Deploy preview for docusaurus-2 ready!

Built with commit 60e1989

https://deploy-preview-1956--docusaurus-2.netlify.com

@docusaurus-bot
Copy link
Copy Markdown
Contributor

Deploy preview for docusaurus-preview ready!

Built with commit 60e1989

https://deploy-preview-1956--docusaurus-preview.netlify.com

Copy link
Copy Markdown
Contributor

@yangshun yangshun left a comment

Choose a reason for hiding this comment

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

Thanks for this @lex111

@yangshun yangshun merged commit 9e290c9 into facebook:master Nov 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed Signed Facebook CLA pr: new feature This PR adds a new API or behavior.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants