-
-
Notifications
You must be signed in to change notification settings - Fork 8.5k
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
feat(v2): add icon to generic sidebar link #4261
Conversation
Size Change: +489 B (0%) Total Size: 544 kB
ℹ️ View Unchanged
|
Deploy preview for docusaurus-2 ready! Built with commit 8591df9 |
⚡️ Lighthouse report for the changes in this PR:
Lighthouse ran on https://deploy-preview-4261--docusaurus-2.netlify.app/classic/ |
[V1] Deploy preview success Built with commit 8591df9 |
why not :) Not sure everybody will like that but it is easy to disable so let's merge |
@samj every single Docusaurus component can be ejected and overridden. You can Swizzle:
|
Motivation
I've noticed that many docs sites have external links in the sidebar that are visually different from the usual ones.
Typically an icon serves as a visual marker for external link, so I added one to the corresponding sidebar item type.
Have you read the Contributing Guidelines on pull requests?
Yes
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.)