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

Render declared sidebar order #1139

Merged
merged 1 commit into from
Dec 8, 2024
Merged

Render declared sidebar order #1139

merged 1 commit into from
Dec 8, 2024

Conversation

tom-blake
Copy link
Contributor

Rendering the sidebar sections in the order that they're declared prevents any unnecessary churn tweaking order numbers whenever the sidebar is added to or restructured.

Test plan:

  • Confirm sidebar order remains the same

@tom-blake tom-blake requested a review from a team as a code owner December 8, 2024 20:24
@tom-blake tom-blake force-pushed the default-sidebar-order branch from cadadca to 5abd47d Compare December 8, 2024 20:25
Base automatically changed from missing-api-docs to master December 8, 2024 22:43
Rendering the sidebar sections in the order that they're declared
prevents any unnecessary churn tweaking order numbers whenever the
sidebar is added to or restructured.

Test plan:
- Confirm sidebar order remains the same
@tom-blake tom-blake force-pushed the default-sidebar-order branch from 5abd47d to 7af9dd2 Compare December 8, 2024 22:44
@tom-blake tom-blake merged commit 168381f into master Dec 8, 2024
8 checks passed
@tom-blake tom-blake deleted the default-sidebar-order branch December 8, 2024 22:48
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.

2 participants