? Success

User tests: Successful: Unsuccessful:

avatar Dade88
Dade88
15 Jan 2015

Lossless optimization of ALL Joomla PNG files. 301 files out of 541 were
definately changed to a smaller version.

66,28 KB Saved.

avatar Dade88 Dade88 - open - 15 Jan 2015
avatar jissues-bot jissues-bot - change - 15 Jan 2015
Labels Added: ?
avatar zero-24 zero-24 - change - 16 Jan 2015
Category Templates (admin) UI/UX
avatar zero-24
zero-24 - comment - 24 Jan 2015

@Bakual @rdeutz @roland-d @Kubik-Rubik @wilsonge can you do a PLT review here?

avatar rdeutz
rdeutz - comment - 24 Jan 2015

@Dade88 could you explain a bit how you did it, my guess is you used a tool to optimise the pictures. I don't think we can compare 300 pictures, so knowing how you did it would give me the confidence to support the merge.

avatar Dade88
Dade88 - comment - 24 Jan 2015

@rdeutz I have used a tool called PNGGauntlet, which make use of PNGOUT, OptiPNG and DeflOpt, compare the results and serves the best one. All tools have been configured for lossless compression. More info at https://en.wikipedia.org/wiki/Portable_Network_Graphics#Optimizing_tools

On a first try, I also tried to do the job via GIMP 'Save for Web' Plugin, however the results where not good as compressing images to PNG-8 results in poor results (most times, visible squared edges).

I might add I always used either this tools or GIMP without problems on my projects.

avatar AlexdeBorba
AlexdeBorba - comment - 26 Jan 2015

This seems to be broken. I tried to apply the Patch and it returns a notice saying the patch was applied however while scrolling down on Joomla! Patch Tester, the patch appears "Not Applied".

avatar brianteeman
brianteeman - comment - 26 Jan 2015

If I remember correctly the patchtester can't be used for binary files
@mbabker
On 26 Jan 2015 22:07, "Alex de Borba" notifications@github.com wrote:

This seems to be broken. I tried to apply the Patch and it returns a
notice saying the patch was applied however while scrolling down on Joomla!
Patch Tester, the patch appears "Not Applied".


Reply to this email directly or view it on GitHub
#5743 (comment).

avatar mbabker
mbabker - comment - 26 Jan 2015

Yep, about 15 months behind on the backlog :wink: joomla-extensions/patchtester#8

avatar Dade88
Dade88 - comment - 28 Jan 2015

Is there anything I can do to solve this or it's some sort of dev team problem?

avatar zero-24
zero-24 - comment - 28 Jan 2015

Is there anything I can do to solve this or it's some sort of dev team problem?

@Dade88 this is a know issue on the patchtester component your PR is fine here.

I hope @rdeutz can have a look into your comment and merge if it is ok for him: #5743 (comment)

Thanks :+1:

avatar wilsonge wilsonge - close - 28 Jan 2015
avatar wilsonge wilsonge - change - 28 Jan 2015
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2015-01-28 16:39:54

Add a Comment

Login with GitHub to post a comment