reduxjs / redux

A JS library for predictable global state management
https://redux.js.org
MIT License
60.93k stars 15.27k forks source link

chore: updated docs to remove extra heading tag #4708

Closed Zain-Bin-Arshad closed 5 months ago

Zain-Bin-Arshad commented 5 months ago

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

Checklist

What docs page needs to be fixed?

What is the problem?

We are using docusaurus for the docs and it will automatically add a header tag from the title attribute. So there is no need to add a redundant h1 tag containing the same title. Currently in the docs here it will display 2 titles with same text which is not a good user experience.

image image

What changes does this PR make to fix the problem?

I have removed the extra h1 tag and now we leave it to docusaurus to render the page's title. This is how it has been done on the sections i.e. Redux Essentials

codesandbox-ci[bot] commented 5 months ago

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.

netlify[bot] commented 5 months ago

Deploy Preview for redux-docs ready!

Name Link
Latest commit 419d1d3e6d87dd5f0b3f079c6f0b90b8246ca486
Latest deploy log https://app.netlify.com/sites/redux-docs/deploys/6656fb813aca26000817c661
Deploy Preview https://deploy-preview-4708--redux-docs.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

EskiMojo14 commented 5 months ago

thanks!