Make WordPress Core

Opened 21 months ago

Last modified 20 months ago

#49438 new defect (bug)

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

Reported by: SGr33n Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version: 5.3
Component: Bundled Theme Keywords: has-patch reporter-feedback
Focuses: Cc:

Description (last modified by SergeyBiryukov)

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.

Attachments (1)

h1-is_home.patch (511 bytes) - added by SGr33n 21 months ago.

Download all attachments as: .zip

Change History (4)

21 months ago

#1 @SergeyBiryukov
21 months ago

  • Description modified (diff)

#2 @ianbelanger
20 months ago

  • Keywords reporter-feedback added
  • Version changed from trunk to 5.3

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?

#3 @SGr33n
20 months ago

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.

Note: See TracTickets for help on using tickets.