? Success

User tests: Successful: Unsuccessful:

avatar marcobiedermann
marcobiedermann
6 Sep 2015

Optimize images with ImageOptim.
Saves 92KB without losing any quality.

joomla-images

avatar marcobiedermann marcobiedermann - open - 6 Sep 2015
avatar marcobiedermann marcobiedermann - change - 6 Sep 2015
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 6 Sep 2015
Labels Added: ?
avatar zero-24 zero-24 - change - 6 Sep 2015
Category Templates (admin)
avatar zero-24 zero-24 - change - 6 Sep 2015
Easy No Yes
avatar designbengel
designbengel - comment - 24 Oct 2015

How to test in connection with Joomla ?


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

avatar balzercomp
balzercomp - comment - 24 Oct 2015

Nothing to test. Another tool for image compression.


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

avatar SniperSister
SniperSister - comment - 24 Oct 2015

@balzercomp @designbengel in this PR the images already have been optimized with imageoptim.

So, in order to test this, you should check a few of the mentioned images in multiple browsers (and maybe some older operating systems too) to make sure the optimization didn't break anything.


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

avatar designbengel
designbengel - comment - 24 Oct 2015

After @SniperSister cleared it up, i wanted to test this but i canĀ“t find the usages of the files.
For Example: administrator/templates/isis/images/admin/featured.png, administrator/templates/isis/images/admin/filesave.png, administrator/templates/isis/images/admin/publish_x.png, administrator/templates/isis/images/admin/icon-16-deny.png

I searched with phpStorm "find in path" option... seems for me there ist some absolutely unused stuff in there ?


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

avatar mbabker
mbabker - comment - 24 Oct 2015

I'm pretty sure in Isis most of the images are unused (1.5 and 2.5 admin templates were really image heavy, Isis is mostly font icons with fewer images).

avatar superknutsel superknutsel - test_item - 12 Dec 2015 - Tested successfully
avatar superknutsel
superknutsel - comment - 12 Dec 2015

I have tested this item :white_check_mark: successfully on 4803f76

@test Tested several images in both Firefox, Chrome and Internet Explorer without any issues.


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

avatar RickR2H RickR2H - test_item - 12 Dec 2015 - Tested successfully
avatar RickR2H
RickR2H - comment - 12 Dec 2015

I have tested this item :white_check_mark: successfully on 4803f76

Works! A nice example that all small things count! Thanks for the PR @marcobiedermann


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

avatar marcobiedermann
marcobiedermann - comment - 12 Dec 2015

@RickR2H You are very welcome :)

avatar brianteeman brianteeman - change - 14 Dec 2015
Status Pending Ready to Commit
avatar brianteeman
brianteeman - comment - 14 Dec 2015

Thanks for testing - setting RTC


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

avatar joomla-cms-bot joomla-cms-bot - change - 14 Dec 2015
Labels Added: ?
avatar rdeutz rdeutz - change - 6 Jan 2016
Milestone Added:
avatar photodude
photodude - comment - 15 Jan 2016

@rdeutz Is there really any reason to hold back this PR until 3.5.1? Seems reducing image file size is a good optimization that should go in 3.5.0.

avatar rdeutz
rdeutz - comment - 15 Jan 2016

Beta state means only bug fixes go in, that's the procedure, that's all.

avatar photodude
photodude - comment - 15 Jan 2016

I would consider this a SEO bug since Google flags sites that don't lossly compress images.

avatar wilsonge
wilsonge - comment - 26 Jan 2016

Can we solve the conflicts please?

avatar Hackwar
Hackwar - comment - 23 Feb 2016

@marcobiedermann Can you please fix the conflicts?

avatar joomla-cms-bot
joomla-cms-bot - comment - 23 Feb 2016

This PR has received new commits.

CC: @RickR2H, @superknutsel


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

avatar marcobiedermann
marcobiedermann - comment - 23 Feb 2016

@Hackwar Fixed merge conflicts

avatar joomla-cms-bot joomla-cms-bot - close - 23 Feb 2016
avatar wilsonge wilsonge - change - 23 Feb 2016
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2016-02-23 17:02:14
Closed_By wilsonge
avatar wilsonge wilsonge - close - 23 Feb 2016
avatar joomla-cms-bot joomla-cms-bot - change - 23 Feb 2016
Labels Removed: ?

Add a Comment

Login with GitHub to post a comment