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

doc: fix color contrast on <kbd> elements #37185

Merged
merged 1 commit into from
Feb 2, 2021

Conversation

aduh95
Copy link
Contributor

@aduh95 aduh95 commented Feb 2, 2021

<kbd> elements are unreadable with dark mode on.

On master:
image

With this PR:
image

@nodejs-github-bot nodejs-github-bot added the doc Issues and PRs related to the documentations. label Feb 2, 2021
@aduh95 aduh95 added the fast-track PRs that do not need to wait for 48 hours to land. label Feb 2, 2021
@aduh95
Copy link
Contributor Author

aduh95 commented Feb 2, 2021

fast-track to add it to #37183?

@nodejs-github-bot
Copy link
Collaborator

PR-URL: nodejs#37185
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@aduh95 aduh95 merged commit b750f74 into nodejs:master Feb 2, 2021
@aduh95
Copy link
Contributor Author

aduh95 commented Feb 2, 2021

Landed in b750f74

@aduh95 aduh95 deleted the fix-doc-kbd-contrast branch February 2, 2021 13:44
@aduh95 aduh95 mentioned this pull request Feb 2, 2021
targos pushed a commit that referenced this pull request Feb 2, 2021
PR-URL: #37185
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
targos pushed a commit that referenced this pull request May 1, 2021
PR-URL: #37185
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@danielleadams danielleadams mentioned this pull request May 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc Issues and PRs related to the documentations. fast-track PRs that do not need to wait for 48 hours to land.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants