WordPress.org

Make WordPress Core

Opened 10 years ago

Closed 10 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 10 years ago.
Patch that adds rel="author" to core and 2011

Download all attachments as: .zip

Change History (15)

@joostdevalk
10 years ago

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

#1 @markjaquith
10 years ago

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

#2 follow-up: @aaroncampbell
10 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
10 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
10 years ago

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

#5 @aaroncampbell
10 years ago

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

#6 @iandstewart
10 years ago

+1 for this.

#7 @iandstewart
10 years ago

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

#8 @iandstewart
10 years ago

  • Component changed from Template to Bundled Theme

#9 @azaozz
10 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
10 years ago

  • Milestone changed from Awaiting Review to 3.2

#11 @joostdevalk
10 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
10 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
10 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
10 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.