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

docs bug fix #351

Closed
wants to merge 1 commit into from
Closed

docs bug fix #351

wants to merge 1 commit into from

Conversation

MitchellShiell
Copy link
Contributor

@MitchellShiell MitchellShiell commented Nov 23, 2023

Documentation page loading bug #349

This PR addresses the React hooks error narrowed down to the sectionTableOfContents component

Issue

Some documentation pages fail to load when clicking between them from the documentation dropdown.

When running locally, you get this error:

15048 | if (!!didRenderTooFewHooks) {
15049 | {

15050 | throw Error( "Rendered fewer hooks than expected. This may be caused by an accidental early return statement." );
15051 | }
15052 | }
15053 | (edited)
Specifically when clicking between updated docs pages (arranger, DMS-UI and Song)
The issue does not occur when moving between older docs pages (Score, Maestro and Ego) or accessing any pages from the URL.
The error only occurs form the megaNav, mobile menu works as expected

@MitchellShiell MitchellShiell deleted the overture-docs-megamenu-fix branch May 1, 2024 16:16
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.

1 participant