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

[2.3] Use H2 for headings in Annex I (diff from prev editions) to fix nav bar issue #983

Merged
merged 1 commit into from
Aug 9, 2024

Conversation

bact
Copy link
Collaborator

@bact bact commented Jul 8, 2024

To allow readthedocs to generate proper heading items in the navigation bar.

Currently when a user clicked on the page https://spdx.github.io/spdx-spec/v2.3/diffs-from-previous-editions/ ,
the nav bar on the left will not expanding the subheadings. This is behavior is different from other chapters/annexes.

The reason is, in Markdown, H1 is reserved for the title.
mkdocs's readthedocs theme will read only H2 headings to generate expandable items in the nav bar.

This fix add one level to all headings in the doc (H1 -> H2), except the title.

There is no change in the content.

See screenshot of a similar fix (for 3.0) in #988

To allow readthedocs to generate proper heading items in the navigation bar.

In Markdown, H1 is reserved for the title.
readthedocs theme will read only H2 headings to generate expandable items in the nav bar.

There is no change in the content.

Signed-off-by: Arthit Suriyawongkul <[email protected]>
Copy link
Member

@goneall goneall left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@goneall goneall modified the milestones: 3.0.1, 2.3.1 Aug 9, 2024
@goneall goneall merged commit ed8c9b5 into spdx:development/v2.3 Aug 9, 2024
1 of 2 checks passed
@bact bact deleted the fix-h1-diff-versions-v2.3 branch August 9, 2024 19:00
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.

3 participants