Skip to content

Fix heading levels on the download pages - #2228

Merged
cwickham merged 1 commit into
mainfrom
fix/download-headings
Sep 29, 2026
Merged

cwickham merged 1 commit into
mainfrom
fix/download-headings

Conversation

@cwickham

@cwickham cwickham commented Sep 28, 2026 •

Copy link
Copy Markdown
Member

The download pages skip heading levels, and axe reports heading-order on each page. The outline goes from the h1 title to h3 "Highlights" and "Release Notes", then to h6 changelog headings.

This PR:

  • Makes "Highlights" and "Release Notes" h2.
  • Changes the tab headings in docs/download/index.qmd from ## to #. As a result, the h2 sections stay inside their tab.
  • Changes populateNews() in _download.html. It moved every changelog heading to h6. Now it moves each heading down one level (## to h3, ### to h4), so the changelog keeps its structure.

The headings are now larger than before.

Checked with an axe scan before and after on docs/download/{index,release,prerelease}.html: the four heading-order findings are gone, and nothing is new or worse. The tabs on the index page are unchanged.

Check

Highlights and Release Notes become h2, the index tabsets split on h1,
and populateNews() shifts changelog headings down one level instead of
flattening them all to h6.
@github-actions

Copy link
Copy Markdown
Contributor

📝 Preview Deployment

🔍 Full site preview: https://deploy-preview-2228.quarto.org

🔄 Modified Documents

@cwickham
cwickham marked this pull request as ready for review September 29, 2026 15:02
@cwickham
cwickham merged commit 474c282 into main Sep 29, 2026
3 checks passed
@cwickham
cwickham deleted the fix/download-headings branch September 29, 2026 15:05
@github-actions

Copy link
Copy Markdown
Contributor

Successfully created backport PR for prerelease:

cwickham added a commit that referenced this pull request Sep 29, 2026
Highlights and Release Notes become h2, the index tabsets split on h1,
and populateNews() shifts changelog headings down one level instead of
flattening them all to h6.

(cherry picked from commit 474c282)

Co-authored-by: Charlotte Wickham <charlotte.wickham@posit.co>
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