Skip to content

Keep reading and sidebar: change heading level to 3 - #96

Merged
MaggieCabrera merged 1 commit into
trunkfrom
update-heading-level
Sep 30, 2026
Merged

MaggieCabrera merged 1 commit into
trunkfrom
update-heading-level

Conversation

@carolinan

@carolinan carolinan commented Sep 24, 2026 •

Copy link
Copy Markdown
Contributor

Description
Fixes the heading level skip in the "keep reading" pattern in the sidebar.
There was a skip from H2 to H5. Changing the post titles from H5 to H3 removes that gap.

Partial for #44, #48

Testing Instructions

Make sure your test installation has a a few posts.
Set one of the posts to use the template with the sidebar.
View the post on the front of the site.
Confirm that there are no visual changes with the PR applied.
Confirm that the heading levels are no longer skipped.

@github-actions github-actions Bot added the First-time Contributor Pull request opened by a first-time contributor to the repository label Sep 24, 2026
@github-actions

github-actions Bot commented Sep 24, 2026 •

Copy link
Copy Markdown

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: carolinan <poena@git.wordpress.org>
Co-authored-by: MaggieCabrera <onemaggie@git.wordpress.org>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@github-actions

Copy link
Copy Markdown

👋 Thanks for your first pull request, @carolinan! We'll take a look soon.

@github-actions github-actions Bot added [Area] Patterns Issues or PRs that touch the theme's block patterns. and removed First-time Contributor Pull request opened by a first-time contributor to the repository labels Sep 24, 2026
@MaggieCabrera
MaggieCabrera merged commit 0970d29 into trunk Sep 30, 2026
8 checks passed
@MaggieCabrera
MaggieCabrera deleted the update-heading-level branch September 30, 2026 08:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Area] Patterns Issues or PRs that touch the theme's block patterns.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants