Making WordPress.org

Opened 8 years ago

Closed 8 years ago

#2497 closed defect (bug) (fixed)

TITLE tag displaying site name instead of post title at make.wordpress.org

Reported by: laryswan's profile laryswan Owned by: ocean90's profile ocean90
Milestone: Priority: normal
Component: Make (Get Involved) / P2 Keywords: ux-feedback has-patch commit
Cc:

Description

Many/most make.wp.org pages have the same <TITLE> tag. That’s got big SEO implications, of course, and it also makes it tough to identify and revisit previously viewed pages (attaching a screen shot of my browser history). Quickest fix would probably be to populate the <TITLE> field with the page title instead of the site name.

Attachments (2)

Screen Shot 2017-02-14 at 9.37.19 AM.png (1017.5 KB) - added by laryswan 8 years ago.
screen shot of multiple identical page TITLEs
meta-2497.patch (604 bytes) - added by SergeyBiryukov 8 years ago.

Download all attachments as: .zip

Change History (8)

@laryswan
8 years ago

screen shot of multiple identical page TITLEs

#1 follow-ups: @Otto42
8 years ago

That's a good initial patch, but it produces single post titles like so:

"CampTix sales through PayPal Weekend Outage - Make WordPress Community"

Should we change the formatting of those any before making such a change across the whole make system? Prefix it with something perhaps?

#2 in reply to: ↑ 1 ; follow-up: @SergeyBiryukov
8 years ago

  • Keywords has-patch commit added

Replying to Otto42:

That's a good initial patch, but it produces single post titles like so:

"CampTix sales through PayPal Weekend Outage - Make WordPress Community"

That's what the old wporg-p2 theme was using before the switch to wporg-breathe was made, which uses get_bloginfo( 'name', 'display' ) for all titles for some reason. As far as I can see, it was not an intended change.

#3 in reply to: ↑ 1 @laryswan
8 years ago

Regarding both SEO and the UI, the best practice would be to add any info in addition to the page title as a suffix.

Replying to Otto42:

Should we change the formatting of those any before making such a change across the whole make system? Prefix it with something perhaps?

#4 in reply to: ↑ 2 @iandunn
8 years ago

Replying to SergeyBiryukov:

As far as I can see, it was not an intended change.

That was my guess too. @obenland, did you intend for this to happen in r3514?

#5 @obenland
8 years ago

I did not, it should work as in meta-2497.patch.
The global header should probably too fwiw

#6 @ocean90
8 years ago

  • Owner set to ocean90
  • Resolution set to fixed
  • Status changed from new to closed

In 4941:

Breathe: Use wp_get_document_title() for the title tag.

Props SergeyBiryukov.
Fixes #2497.

Note: See TracTickets for help on using tickets.