? Pending

User tests: Successful: Unsuccessful:

avatar wilsonge
wilsonge
26 Mar 2017

Pull Request for Issue #14888 .

Summary of Changes

Removes nofollow attributes in select places

Testing Instructions

Check com_content and com_fields url type no longer have a nofollow attribute

Expected result

Links do not have nofollow

Actual result

Links do have nofollow

Documentation Changes Required

None

avatar wilsonge wilsonge - open - 26 Mar 2017
avatar wilsonge wilsonge - change - 26 Mar 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 26 Mar 2017
Category Front End com_content Plugins Templates (site)
avatar brianteeman
brianteeman - comment - 26 Mar 2017

Simply removing them is not a good idea.

You are changing the behaviour of an existing site. If you really believe
that they are wrong (that's a different debate) then they should be
replaced with an option that the site owner can choose (thats what we do
everywhere else) and for backwards compatibility then they should have a
default value of follow in order not to break an existing site

http://www.avg.com/email-signature?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail
Virus-free.
www.avg.com
http://www.avg.com/email-signature?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail
<#DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>

On 26 March 2017 at 17:30, George Wilson notifications@github.com wrote:

Pull Request for Issue #14888
#14888 .
Summary of Changes

Removes nofollow attributes in select places
Testing Instructions

Check com_content and com_fields url type no longer have a nofollow
attribute
Expected result

Links do not have nofollow
Actual result

Links do have nofollow
Documentation Changes Required

None

You can view, comment on, or merge this pull request online at:

#14908
Commit Summary

  • Remove some no-follow to improve SEO. Fixes #14888

File Changes

Patch Links:


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#14908, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPH8WhIB7XSnehCJ0qotdwFR_BcP-Umks5rppK_gaJpZM4MpevC
.

--
Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
https://brian.teeman.net/ http://brian.teeman.net/

avatar brianteeman
brianteeman - comment - 26 Mar 2017

Simply blindly making this change can in fact harm seo and not improve it

avatar laoneo
laoneo - comment - 11 Apr 2017

For com_fields, we can add a "rel" attribute. I would also not just remove them.

avatar brianteeman
brianteeman - comment - 19 May 2017

I have tested this item ? unsuccessfully on c7ba42b

just recording an unsuccessful test on this PR on the issue tracker for the reasons stated earlier


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/14908.

avatar brianteeman brianteeman - test_item - 19 May 2017 - Tested unsuccessfully
avatar brianteeman
brianteeman - comment - 4 Jan 2018

I am closing this. Two of us have stated that its not a good idea and not b/c, it has conflicts and the OP hasnt responded

avatar brianteeman brianteeman - close - 4 Jan 2018
avatar brianteeman brianteeman - change - 4 Jan 2018
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2018-01-04 19:23:34
Closed_By brianteeman

Add a Comment

Login with GitHub to post a comment