WordPress.org

Make WordPress Core

Opened 6 years ago

Closed 6 years ago

#17722 closed enhancement (fixed)

Implement rel="author" in Twenty Eleven, Twenty Ten and core

Reported by: joostdevalk Owned by: joostdevalk
Milestone: 3.2 Priority: normal
Severity: normal Version: 3.2
Component: Bundled Theme Keywords:
Focuses: Cc:

Description

Google today announced support for the new rel="author". Of course, as WordPress has always been about semantic publishing, we like and support that, so I thought I'd quickly come up with a patch and see if we can push this into 3.2 :)

Attachments (1)

rel-author.diff (11.4 KB) - added by joostdevalk 6 years ago.
Patch that adds rel="author" to core and 2011

Download all attachments as: .zip

Change History (15)

@joostdevalk
6 years ago

Patch that adds rel="author" to core and 2011

#1 @markjaquith
6 years ago

Seems like an easy win. Good for semantics, nice PR.

#2 follow-up: @aaroncampbell
6 years ago

+1 - It looks like the patch affects Twenty Ten as well as Twenty Eleven. How are we handling stuff like this for Twenty Ten?

#3 in reply to: ↑ 2 @joostdevalk
6 years ago

Replying to aaroncampbell:

+1 - It looks like the patch affects Twenty Ten as well as Twenty Eleven. How are we handling stuff like this for Twenty Ten?

Yeah I sort of did that on purpose to be honest, if people are using Twenty Ten this is a nice upgrade, right?

#4 @joostdevalk
6 years ago

Although it might be wise to note it in the ticket description as well then ;)

#5 @aaroncampbell
6 years ago

Personally I'd love to see Twenty Ten get the update as well.

#6 @iandstewart
6 years ago

+1 for this.

#7 @iandstewart
6 years ago

  • Summary changed from Implement rel="author" to Implement rel="author" in Twenty Eleven and Twenty Ten

#8 @iandstewart
6 years ago

  • Component changed from Template to Bundled Theme

#9 @azaozz
6 years ago

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

In [18189]:

Implement rel="author" in Twenty Eleven and Twenty Ten, props joostdevalk, fixes #17722

#10 @ryan
6 years ago

  • Milestone changed from Awaiting Review to 3.2

#11 @joostdevalk
6 years ago

  • Summary changed from Implement rel="author" in Twenty Eleven and Twenty Ten to Implement rel="author" in Twenty Eleven, Twenty Ten and core

For the record, the ticket now says in Twenty Eleven and Twenty Ten, the patch does fix it in core too.

#12 @wet
6 years ago

  • Keywords has-patch removed
  • Resolution fixed deleted
  • Status changed from closed to reopened

I fail to grasp how this patch fulfills Google's definition of how this attribute is supposed to add semantic value:

An author page is a page about a specific author. [...] 
The rel="author" link must point to an author page
on the same site as the content page.

IMHO, all we get is a filtered list of an author's content items. This is by no means an author page (bio, contact info, author profile) AFAICS.

Can anyone shed some light on this, besides the obvious PR value?

#13 @nacin
6 years ago

In some themes, the author page will show specific information about an author. Often it might actually be their bio, link to their website, their photo/gravatar, a link to their twitter handle, what have you, followed by just a bulleted list of recent stories or something.

Many themes won't have that. In that case, this would not be beneficial, but it won't hurt, either. But on sites where the author page *does* mean something, then this definitely provides a value to search engines.

#14 @nacin
6 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

Discussion in a ticket can still occur even when it is closed.

Note: See TracTickets for help on using tickets.