? Pending

User tests: Successful: Unsuccessful:

avatar mbabker
mbabker
16 Sep 2016

Summary of Changes

In a complete OCD move, this changes the order of the <link> attributes for stylesheets to match the order used for links in the section above this. There is seriously no other benefit than it being consistent for both types now and calming the OCD of people like me who notice these things.

Testing Instructions

Pre-patch, you'll have something like this:

<link href="/templates/joomla/favicon.ico" rel="shortcut icon" type="image/vnd.microsoft.icon" />
<link rel="stylesheet" href="https://cdn.joomla.org/template/css/template_2.1.0.min.css" />

Post-patch, it'll be like this:

<link href="/templates/joomla/favicon.ico" rel="shortcut icon" type="image/vnd.microsoft.icon" />
<link href="https://cdn.joomla.org/template/css/template_2.1.0.min.css" rel="stylesheet" />

Documentation Changes Required

N/A

avatar mbabker mbabker - open - 16 Sep 2016
avatar mbabker mbabker - change - 16 Sep 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 16 Sep 2016
Category Libraries
avatar joomla-cms-bot joomla-cms-bot - change - 16 Sep 2016
Labels Added: ?
avatar dgt41
dgt41 - comment - 16 Sep 2016

I have not tested this item.


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

avatar dgt41 dgt41 - test_item - 16 Sep 2016 - Not tested
avatar jeckodevelopment jeckodevelopment - test_item - 16 Sep 2016 - Tested successfully
avatar jeckodevelopment
jeckodevelopment - comment - 16 Sep 2016

I have tested this item successfully on 6a433b7


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

avatar Bakual Bakual - change - 16 Sep 2016
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-09-16 18:31:15
Closed_By Bakual
avatar Bakual Bakual - close - 16 Sep 2016
avatar Bakual Bakual - merge - 16 Sep 2016
avatar Bakual
Bakual - comment - 16 Sep 2016

Had to look up OCD ?

avatar zero-24
zero-24 - comment - 16 Sep 2016

Any reason to not have a milestone here? @bakual

Add a Comment

Login with GitHub to post a comment