Twenty Twenty: h1 when is_front_page() is a static page

[SGr33n]

Hi,
I see that template-tags.php doesn't assign the h1 to the site title correctly when you assign a static page to the front page.

It's due to the code
'condition' => ( is_front_page() || is_home() ) && ! is_page()

is_page(), infact, returns true on is_front_page() in that case.

I suppose it should be:

'condition' => is_front_page() || ( is_home() && ! is_page() )

Diff patch attached.

[ianbelanger]

I tested Twenty Twenty with a static page set as the homepage and I am seeing the page title as an H1. @SGr33n Could you please confirm that you are still seeing an issue?

[SGr33n]

Hi @ianbelanger,
It's not the page title. On the home page, according to SEO, the correct H1 should be the site title, or the site logo. It's the same behavior we had on earlier themes.

[poena]

That the page title is the H1 is intentional. When it is possible, the H1 should be the main heading of the page, describing the content below it. It is important for making the content accessible.
It does not damage SEO.

The headings are not comparable with older default themes because SEO best practices has changed and the accessibility awareness in the WordPress project has increased over time.

[SGr33n]

Replying to poena:

The headings are not comparable with older default themes because SEO best practices has changed and the accessibility awareness in the WordPress project has increased over time.

It was a 4 years old ticket 😋 It isn't actual anymore. Everything changed in 4 years 😅