WordPress / twentytwentytwo

Twenty Twenty-Two, the default WordPress theme that will launch with WordPress 5.9.
404 stars 92 forks source link

Having h1 titles on pages #250

Closed SGr33n closed 2 years ago

SGr33n commented 2 years ago

The theme is using an H2 to display the page titles. That title should have an H1 tag.

kjellr commented 2 years ago

Thanks for the issue. I'm going to combine this with #249, since they're so related.

kjellr commented 2 years ago

Oh, sorry  — I didn't realize this was a PR. I'll leave it open for now while we discuss.

kjellr commented 2 years ago

As per the discussion in https://github.com/WordPress/twentytwentytwo/issues/233#issuecomment-985743555, we should be ok using more than one H1 per page. I've updated the "No Separators" template too, and will merge this in. Thanks for the help, @SGr33n!