-
-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
refactor(theme-classic): fix a few margin inconsistencies #7129
Conversation
@@ -60,7 +60,7 @@ export default function LocaleDropdownNavbarItem({ | |||
label={ | |||
<span> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TODO: remember to remove the parent span
during next updating of Infima.
@@ -29,6 +29,7 @@ | |||
|
|||
.sidebarItemLink { | |||
color: var(--ifm-font-color-base); | |||
display: block; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just for the sake of uniformity -- since all the links in such blocks (sidebars, toc) are full-width.
✅ [V2]
To edit notification comments on pull requests, go to your Netlify site settings. |
⚡️ Lighthouse report for the changes in this PR:
Lighthouse ran on https://deploy-preview-7129--docusaurus-2.netlify.app/ |
Size Change: +385 B (0%) Total Size: 806 kB
ℹ️ View Unchanged
|
0f98c0d
to
a6acab3
Compare
a6acab3
to
dd65ca6
Compare
packages/docusaurus-theme-classic/src/theme/DocCardList/index.tsx
Outdated
Show resolved
Hide resolved
packages/docusaurus-theme-classic/src/theme/DocCardList/index.tsx
Outdated
Show resolved
Hide resolved
It's all fixed, should be good this time. |
I see 🙈 All good now
Should we wait for another release? |
Probably shouldn't. I put it down in my notes what changes need to be made when updating Infima. I want to do few things in Infima and release its new version soon. |
👍 |
Motivation
Yes
Have you read the Contributing Guidelines on pull requests?
These are mainly fixes to ensure consistency in design, such as removing extra spacing, eg:
Test Plan
Preview, nothing critical.
Related PRs