? Pending

User tests: Successful: Unsuccessful:

avatar dgrammatiko
dgrammatiko
29 Jul 2018

Pull Request for Issue # .

Summary of Changes

Reduces the impact of js uglification.
Some of the vendor files are not minified. We were running js uglification in every subdirectory of the media folder. This has a huge impact on performance.

Bench the npm i on a clean installation
Before: 80,69 sec
After: 59,24 sec

Testing Instructions

Expected result

Actual result

Documentation Changes Required

avatar dgrammatiko dgrammatiko - open - 29 Jul 2018
avatar dgrammatiko dgrammatiko - change - 29 Jul 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 29 Jul 2018
Category JavaScript Repository
avatar dgrammatiko
dgrammatiko - comment - 29 Jul 2018

Another way to benchmark the changes here are our tests:
Before:
screenshot 2018-07-29 at 19 43 50

After:
screenshot 2018-07-29 at 19 44 22

That's almost 2 minutes difference or 25% improvement!!!

avatar ggppdk ggppdk - test_item - 29 Jul 2018 - Tested successfully
avatar ggppdk
ggppdk - comment - 29 Jul 2018

I have tested this item successfully on 060ddf5

Works,
does not seem to cause problems
i can not code review this


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

avatar laoneo
laoneo - comment - 30 Jul 2018

Conflicts

avatar wilsonge wilsonge - change - 31 Jul 2018
Labels Added: ?
avatar wilsonge wilsonge - change - 31 Jul 2018
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-07-31 09:14:30
Closed_By wilsonge
avatar wilsonge wilsonge - close - 31 Jul 2018
avatar wilsonge wilsonge - merge - 31 Jul 2018

Add a Comment

Login with GitHub to post a comment