fix(theme): ignored className attribute on lazy loaded TabItem

facebook / docusaurus

Easy to maintain open source documentation websites.

https://docusaurus.io

MIT License

54.07k stars 8.1k forks source link

fix(theme): ignored className attribute on lazy loaded TabItem #10219

Closed lebalz closed 2 weeks ago

lebalz commented 2 weeks ago

Fixes #10218

Pre-flight checklist

[ ] I have read the Contributing Guidelines on pull requests.
[ ] If this is a code change: I have written unit tests and/or added dogfooding pages to fully verify the new behavior.
[ ] If this is a new API or substantial change: the PR has an accompanying issue (closes #0000) and the maintainers have approved on my working plan.

Motivation

Test Plan

Test links

Deploy preview: https://deploy-preview-_____--docusaurus-2.netlify.app/

Related issues/PRs

netlify[bot] commented 2 weeks ago

[V2]

Built without sensitive environment variables

Name	Link
Latest commit	b013ea90d428e27c44683d0de6be03f8ecfd27b0
Latest deploy log	https://app.netlify.com/sites/docusaurus-2/deploys/6670900c64f27e0008bd7409
Deploy Preview	https://deploy-preview-10219--docusaurus-2.netlify.app
Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

github-actions[bot] commented 2 weeks ago

⚡️ Lighthouse report for the deploy preview of this PR

URL	Performance	Accessibility	Best Practices	SEO	PWA	Report
/	🟠 61	🟢 98	🟢 96	🟢 100	🟠 88	Report
/docs/installation	🟠 61	🟢 96	🟢 100	🟢 100	🟠 88	Report
/docs/category/getting-started	🟠 75	🟢 100	🟢 100	🟢 90	🟠 88	Report
/blog	🟠 70	🟢 100	🟢 100	🟢 90	🟠 88	Report
/blog/preparing-your-site-for-docusaurus-v3	🟠 62	🟢 96	🟢 100	🟢 100	🟠 88	Report
/blog/tags/release	🟠 70	🟢 100	🟢 100	🟢 90	🟠 88	Report
/blog/tags	🟠 74	🟢 100	🟢 100	🟢 90	🟠 88	Report

slorber commented 2 weeks ago

Thanks 👍