Twenty Twenty-One: H1 is misplaced for a page set as blogpost container

[rianrietveld]

When a page is used to display the blogposts, the H1 isn't the page title but the site title in the header. This results in an H1 that doesn't represent the content of the page.

How to reproduce:
In the Admin go to Settings > Reading.
With "Your homepage displays" select "A static page (select below)" and for the Posts page select an existing page.

Expected result:
The page title is the (visual) H1, which is set above the listed posts.
The same way the posts of category are displayed.

[justinahinon]

This patch adds the page header to the blog posts page

[mukesh27]

i tested 52938.diff​ but it will not return any title and show an empty header tag. @justinahinon is the above patch working for you?

[justinahinon]

@mukesh27 yes, I do have the title on blog page.

How you can test:

• On a fresh install with Twenty Twenty-One, create a page (Blog for instance)
• Go to Settings > Reading and select the page created as the "Posts page"
• Go to the new created page on the frontend and you should see its title at the top of the page. See this screenshot ​https://cln.sh/0Bv0jw.

[mukesh27]

@justinahinon thanks for confirming.

If we select the Posts page as the existing page then it will show the title of that page.

But For Settings > Reading > Your latest posts option shows an empty title. ​https://tinyurl.com/yez25bbn

[justinahinon]

Replying to mukesh27:

But For Settings > Reading > Your latest posts option shows an empty title. ​https://tinyurl.com/yez25bbn

I noticed this. Thanks for flagging. Working on updating the patch.

[justinahinon]

@mukesh27 in 52938-1.diff​, I checked if the page is not the default front page and is is_home() is true.

But I'm still seeing an edge case where this will display an empty title. When in Settings > Readings, "A static page" is selected, a page is set as "Posts page" but no page is set as "Homepage". The site homepage will display an empty title.

[mukesh27]

Agreed with you. In one case it shows empty title :(

[sabernhardt]

adds heading to blog page only if title exists, removes the H1 from posts page site-title

[sabernhardt]

Thanks for the patches!

The site-branding template needed editing, too, so it would not have two H1 headings.

For edge cases, 52938.2.diff​ simply does not output the header and H1 tags if there is no title (without the prefix). This could result in no H1 for the page, but I think that's better than outputting an empty tag.

[mukesh27]

Patch 52938.2.diff​ working fine for me in latest version 5.8-alpha-50787

@justinahinon can you please test the patch so we can remove needs-testing keyword

[justinahinon]

Replying to mukesh27:

Tested with fresh WordPress 5.8-alpha-50427-src.

Works fine for me.

But I'd suggest we keep the need-testing tag for a bit, so that other people can test also.

[francina]

Tested on

• MacOS 11.3
• Safari 14.1
• WordPress 5.8-alpha-50427-src

Patch 52938.2.diff works fine for me

[francina]

Screenshot of blog archive page after patch

[justinahinon]

Removing the needs-testing feedback after @francina's test.

@audrasjb, at this point, can we add the commit tag to the ticket?

[audrasjb]

I tested the proposed patch and I can confirm it shows the single_post_title in a <h1> tag (which is "a Blog page" when using the Theme Unit Test dataset and when this page is defined as the blog page). Also, the site title uses a <p> tag.

Good to go!

[SergeyBiryukov]

In 50802:

Twenty Twenty-One: Display page title as the H1 heading when a static page is selected as the "Posts page".

This ensures that the heading represents the content of the page. Previously, the site title was displayed instead.

Props sabernhardt, justinahinon, rianrietveld, mukesh27, francina, audrasjb, Boniu91.
Fixes #52938.

[SergeyBiryukov]

In 50806:

Twenty Twenty-One: Fix "Opening PHP tag must be on a line by itself" WPCS issue.

Follow-up to [50802].

See #52938.