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

⬆️ Bump Docusaurus to 2.0 beta 7 #4193

Merged
merged 1 commit into from
Oct 18, 2021

Conversation

Shrugsy
Copy link
Contributor

@Shrugsy Shrugsy commented Oct 18, 2021


name: 📝 Documentation Fix
about: Fixing a problem in an existing docs page

Checklist

  • Is there an existing issue for this PR?
    • N/A
  • Have the files been linted and formatted?
    • N/A

What docs page needs to be fixed?

  • Section: Sidebar
  • Page: N/A

What is the problem?

Certain sidebar items get obscured after expanding. Reproducibility depends on browser size.

Before (item obscured) After (item visible)
image image

What changes does this PR make to fix the problem?

Takes advantage of facebook/docusaurus#5694 which specifically referenced the redux docs when fixing the issue

Note also that this will address page shifting when toggling between TS/JS snippet tabs, which would occur if #4149 is merged.
i.e. @timdorr it would specifically address your concern in this comment: #4112 (comment)

See reduxjs/redux-toolkit#1620 (comment) for an example of how the page shifting is addressed.

@Shrugsy Shrugsy marked this pull request as draft October 18, 2021 17:42
@codesandbox-ci
Copy link

codesandbox-ci bot commented Oct 18, 2021

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit ea4055c:

Sandbox Source
Vanilla Configuration
Vanilla Typescript Configuration

@Shrugsy Shrugsy marked this pull request as ready for review October 18, 2021 17:44
@netlify
Copy link

netlify bot commented Oct 18, 2021

✔️ Deploy Preview for redux-docs ready!

🔨 Explore the source changes: a3ffa1d

🔍 Inspect the deploy log: https://app.netlify.com/sites/redux-docs/deploys/616db1ec1cb4020007f59731

😎 Browse the preview: https://deploy-preview-4193--redux-docs.netlify.app

@netlify
Copy link

netlify bot commented Oct 18, 2021

✔️ Deploy Preview for redux-docs ready!

🔨 Explore the source changes: ea4055c

🔍 Inspect the deploy log: https://app.netlify.com/sites/redux-docs/deploys/616dbb24d1bf34062a0ba22b

😎 Browse the preview: https://deploy-preview-4193--redux-docs.netlify.app

@timdorr
Copy link
Member

timdorr commented Oct 18, 2021

Yep, that looks to fix it. Thanks!

@timdorr timdorr merged commit 408ede0 into reduxjs:master Oct 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants