Fix documentation subsection links - #110
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This change waits for the
zero-mdcomponents on a documentation page to finish rendering before reapplying a subsection fragment from the URL. Without this step, content loaded above the target can move it after the browser’s initial fragment navigation.I reproduced the reported problem locally using
multi_physics.html#equation_section, then verified that this change opens the page at the Equation Section. Because the fix is in the shared documentation JavaScript, it also supports direct subsection links on other pages using the samezero-mdstructure.OpenAI’s code-analysis and browser-testing tools helped identify the rendering-order problem and test the fix.
Closes #108.