Skip to content

fix(v2): use proper element for pagination nav label#3895

Merged
lex111 merged 1 commit intomasterfrom
lex111/proper-el-pag-nav-item
Dec 8, 2020
Merged

fix(v2): use proper element for pagination nav label#3895
lex111 merged 1 commit intomasterfrom
lex111/proper-el-pag-nav-item

Conversation

@lex111
Copy link
Copy Markdown
Contributor

@lex111 lex111 commented Dec 8, 2020

Motivation

The pagination-nav__label CSS class must be used with a div, not an h4. Otherwise, it leads to the adding of extra margins (see Test plan).

Have you read the Contributing Guidelines on pull requests?

Yes

Test Plan

Before:

image

After:

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

@lex111 lex111 added the pr: bug fix This PR fixes a bug in a past release. label Dec 8, 2020
@lex111 lex111 requested a review from slorber as a code owner December 8, 2020 15:11
@facebook-github-bot facebook-github-bot added the CLA Signed Signed Facebook CLA label Dec 8, 2020
@github-actions
Copy link
Copy Markdown

github-actions bot commented Dec 8, 2020

Size Change: +15 B (0%)

Total Size: 154 kB

ℹ️ View Unchanged
Filename Size Change
website/build/blog/2017/12/14/introducing-docusaurus/index.html 20.7 kB -2 B (0%)
website/build/docs/introduction/index.html 180 B 0 B
website/build/index.html 5.82 kB -2 B (0%)
website/build/main.********.js 109 kB +19 B (0%)
website/build/styles.********.css 17.5 kB 0 B

compressed-size-action

@netlify
Copy link
Copy Markdown

netlify bot commented Dec 8, 2020

✔️ Deploy preview for docusaurus-2 ready!

🔨 Explore the source changes: 5dcec60

🔍 Inspect the deploy logs: https://app.netlify.com/sites/docusaurus-2/deploys/5fcf97b1df08c300072b6981

😎 Browse the preview: https://deploy-preview-3895--docusaurus-2.netlify.app

@github-actions
Copy link
Copy Markdown

github-actions bot commented Dec 8, 2020

⚡️ Lighthouse report for the changes in this PR:

Category Score
🟠 Performance 71
🟢 Accessibility 99
🟢 Best practices 100
🟢 SEO 100
🟢 PWA 95

Lighthouse ran on https://deploy-preview-3895--docusaurus-2.netlify.app/classic/

@lex111 lex111 merged commit 3e12e95 into master Dec 8, 2020
@lex111 lex111 deleted the lex111/proper-el-pag-nav-item branch December 8, 2020 15:34
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: bug fix This PR fixes a bug in a past release.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants