Skip to content

Added few lines for logo in dark theme#3797

Closed
yahu1031 wants to merge 1 commit intofacebook:masterfrom
yahu1031:patch-1
Closed

Added few lines for logo in dark theme#3797
yahu1031 wants to merge 1 commit intofacebook:masterfrom
yahu1031:patch-1

Conversation

@yahu1031
Copy link

There is no description about dark theme logo. I have added 2 to 3 lines which guide user to add srcDark in navbar.

Motivation

(Write your motivation here.)

Have you read the Contributing Guidelines on pull requests?

yes

Test Plan

As they are just documentation changes, I hope no need of test plan.

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.)

There is no description about dark theme logo. I added 2 to 3 lines which guide user to add `srcDark` in navbar.
@facebook-github-bot
Copy link
Contributor

Hi @yahu1031!

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file.

In order for us to review and merge your code, please sign at https://code.facebook.com/cla. If you are contributing on behalf of someone else (eg your employer), the individual CLA may not be sufficient and your employer may need to sign the corporate CLA.

If you have received this in error or have any questions, please contact us at cla@fb.com. Thanks!

@netlify
Copy link

netlify bot commented Nov 21, 2020

Deploy preview for docusaurus-2 ready!

Built without sensitive environment variables with commit d0e447c

https://deploy-preview-3797--docusaurus-2.netlify.app

@facebook-github-bot facebook-github-bot added the CLA Signed Signed Facebook CLA label Nov 21, 2020
@facebook-github-bot
Copy link
Contributor

Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

@slorber slorber added the pr: documentation This PR works on the website or other text documents in the repo. label Nov 23, 2020
@slorber
Copy link
Collaborator

slorber commented Nov 23, 2020

thanks ;)

@slorber
Copy link
Collaborator

slorber commented Nov 23, 2020

You documented the bootstrap theme, not the classic theme. Is it what you really wanted to do? I don't think the bootstrap theme is currently used by anyone as it's not production ready.

@yahu1031
Copy link
Author

You documented the bootstrap theme, not the classic theme. Is it what you really wanted to do? I don't think the bootstrap theme is currently used by anyone as it's not production ready.

uh oh! okie i will give it a try too. cause it is in development. 😉

@Simek
Copy link
Contributor

Simek commented Nov 25, 2020

@slorber This note is not true for theme-classic after the #3784 fix has been merged. There was no intention to introduce breaking change and require darkSrc instead of using src as a fallback. It was a bug.

@yahu1031 yahu1031 closed this Nov 28, 2020
@yahu1031
Copy link
Author

Fixed the issue in #3784

@yahu1031 yahu1031 deleted the patch-1 branch November 28, 2020 16:26
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: documentation This PR works on the website or other text documents in the repo.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants