? Success

User tests: Successful: Unsuccessful:

avatar andrepereiradasilva
andrepereiradasilva
3 Nov 2016

Pull Request for Issue #9638 (css part).

Summary of Changes

This does for css the same that was done in #12373 for js.

So, this PR replaces all deprecated stylesheets methods signature usage across the CMS.

Also:

  • adds stylesheet version to all js files to solve #9638
  • use the api in conditional ie statements

Testing Instructions

  1. Code review.
  2. Use latest staging
  3. Apply #12375 and them this patch
  4. Navigate in the frontend/backend/installation (several template). Confirm all is working fine and check all external css files are loaded and with media version

Documentation Changes Required

None.

Notes

Unit tests will fail until #12375 is merged.

avatar andrepereiradasilva andrepereiradasilva - open - 3 Nov 2016
avatar andrepereiradasilva andrepereiradasilva - change - 3 Nov 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 3 Nov 2016
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - change - 3 Nov 2016
Category Administration Components com_media Templates (admin) Front End Installation Layout Libraries
avatar andrepereiradasilva andrepereiradasilva - change - 3 Nov 2016
The description was changed
avatar andrepereiradasilva andrepereiradasilva - edited - 3 Nov 2016
avatar roland-d
roland-d - comment - 3 Nov 2016

@andrepereiradasilva Travis still doesn't like you ;)

avatar andrepereiradasilva
andrepereiradasilva - comment - 3 Nov 2016

Notes

Unit tests will fail until #12375 is merged.

?

avatar brianteeman brianteeman - test_item - 3 Nov 2016 - Tested successfully
avatar brianteeman
brianteeman - comment - 3 Nov 2016

I have tested this item successfully on 519d0ce


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

avatar andrepereiradasilva
andrepereiradasilva - comment - 4 Nov 2016

@brianteeman just updated branch since the other pr is now merged to fix the unit tests. Please Mark again as tested with success. No changes.

avatar jeckodevelopment jeckodevelopment - test_item - 4 Nov 2016 - Tested successfully
avatar jeckodevelopment
jeckodevelopment - comment - 4 Nov 2016

I have tested this item successfully on ae520ae


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

avatar brianteeman brianteeman - test_item - 4 Nov 2016 - Tested successfully
avatar brianteeman
brianteeman - comment - 4 Nov 2016

I have tested this item successfully on ae520ae


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

avatar jeckodevelopment jeckodevelopment - change - 4 Nov 2016
Status Pending Ready to Commit
avatar jeckodevelopment
jeckodevelopment - comment - 4 Nov 2016

RTC


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

avatar jeckodevelopment jeckodevelopment - change - 4 Nov 2016
Labels Added: ?
avatar jeckodevelopment jeckodevelopment - change - 4 Nov 2016
Milestone Added:
avatar rdeutz rdeutz - change - 4 Nov 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-11-04 10:07:02
Closed_By rdeutz
avatar rdeutz rdeutz - close - 4 Nov 2016
avatar rdeutz rdeutz - merge - 4 Nov 2016
avatar rdeutz rdeutz - reference | babf501 - 4 Nov 16
avatar rdeutz rdeutz - merge - 4 Nov 2016
avatar rdeutz rdeutz - close - 4 Nov 2016

Add a Comment

Login with GitHub to post a comment