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

Fix news header levels #730

Merged
merged 6 commits into from
Jun 3, 2024
Merged

Fix news header levels #730

merged 6 commits into from
Jun 3, 2024

Conversation

bensmrs
Copy link
Contributor

@bensmrs bensmrs commented Jun 3, 2024

The header levels are not consistent across .yaml news files. This PR aligns them to the top (H1) level and gives a similar look to the Incus documentation.

Signed-off-by: Benjamin Somers <[email protected]>
Signed-off-by: Benjamin Somers <[email protected]>
Signed-off-by: Benjamin Somers <[email protected]>
Signed-off-by: Benjamin Somers <[email protected]>
Signed-off-by: Benjamin Somers <[email protected]>
@bensmrs
Copy link
Contributor Author

bensmrs commented Jun 3, 2024

The change from 2-2 to 2-3 in the TOC levels is a matter of taste ; I can revert everything to 2-2, but I felt like a TOC with only one level was a bit strange considering the news page only shows the latest 5 news entries…

@stgraber stgraber merged commit f8af563 into lxc:main Jun 3, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants