#20047 closed enhancement (fixed)

get_the_author_link() Should default to rel="author"

Reported by: Ipstenu Owned by: nacin
Priority: normal Milestone: 3.4
Component: General Version: 3.3.1
Severity: normal Keywords: has-patch commit
Cc: sirzooro, ben@…

Description

Per this WP.org forum post, the linktype of author would comply more with w3c's wibbly wobbly guidelines.

Attachments (2)

20047.diff (725 bytes) - added by Ipstenu 15 months ago.
Changing external to author
20047-2.diff (734 bytes) - added by CoenJacobs 14 months ago.
Improved version of Ipstenu's patch, using 'author external' as rel

Download all attachments as: .zip

Change History (9)

Changing external to author

  • Milestone changed from Awaiting Review to 3.4
  • Cc sirzooro added

More appropriate would be rel="author external" - these links usually leads to external websites.

They usually lead to /author/username in my experience. When I click on the post author in a post, I usually go to the in-blog page for that author.

comment:4 follow-up: ↓ 5   husobj14 months ago

  • Cc ben@… added

I believe sirzooro is correct.

In this instance the get_the_author_link() gets the website URL that is entered into the text field on a user's profile which is usually an external URL (unless someone manually enters a link to the site site), not the users post archive page.

The link to the users post archive page can be retrieved using the_author_posts_link() which already adds a rel="author" attribute.

Version 0, edited 14 months ago by husobj (next)

Improved version of Ipstenu's patch, using 'author external' as rel

comment:5 in reply to: ↑ 4   CoenJacobs14 months ago

Replying to husobj:

I believe sirzooro is correct.

In this instance the get_the_author_link() gets the website URL that is entered into the text field on a user's profile which is usually an external URL (unless someone manually enters a link to the site), not the users post archive page.

I agree. If there was a easy way to check the url to be external, it would be nicer to check that and apply external if it really is an external link. Provided a new patch (based on Ipstenu's fix) to get this ready.

Last edited 14 months ago by CoenJacobs (previous) (diff)
  • Keywords commit added
  • Owner set to nacin
  • Resolution set to fixed
  • Status changed from new to closed

In [20615]:

Add 'author' to rel attribute for get_the_author_link(). props sirzooro, CoenJacobs, Ipstenu. fixes #20047.

Note: See TracTickets for help on using tickets.