Closed
Bug 736818
Opened 13 years ago
Closed 13 years ago
Port |Bug 653038 - Replacement image for sync-bg.png smaller optimized PNG|
Categories
(SeaMonkey :: Sync UI, defect)
SeaMonkey
Sync UI
Tracking
(Not tracked)
RESOLVED
FIXED
seamonkey2.11
People
(Reporter: InvisibleSmiley, Assigned: InvisibleSmiley)
Details
Attachments
(1 file)
53.48 KB,
patch
|
neil
:
review+
|
Details | Diff | Splinter Review |
sync-bg.png (used for Tabs From Other Computers = about:sync-tabs) was introduced with Sync UI Bug 576970 and last modified through PNG opt Bug 593840. Seems the improvement done through the latter was suboptimal, since the FF guys were able to squeeze it even more in Bug 653038:
Old size FF: 26756 bytes
Old size SM: 26754 bytes
New size: 21309 bytes (~20% off)
Attachment #606953 -
Flags: review?(neil)
![]() |
||
Comment 1•13 years ago
|
||
I ran the following on the sync-bg.png from Bug 593840:
optipng.exe -zc1-9 -zm1-9 -zs0-3 -f0-5 -nx -o7 sync-bg.png
And the smallest I could get was 24,629 bytes.
(pngcrush.exe -rem gAMA -rem cHRM -rem iCCP -rem sRGB did not make any difference)
I wonder if they removed more chunks than we did? So I reran with
pngcrush.exe -rem alla
optipng.exe -zc1-9 -zm1-9 -zs0-3 -f0-5 -nx -o7
And I got it down to 23,710 so I wonder what we're doing wrong.
Comment 2•13 years ago
|
||
Comment on attachment 606953 [details] [diff] [review]
patch [Checkin: Comment 3]
I'm assuming here that this just copies the image.
Attachment #606953 -
Flags: review?(neil) → review+
Assignee | ||
Comment 3•13 years ago
|
||
Comment on attachment 606953 [details] [diff] [review]
patch [Checkin: Comment 3]
http://hg.mozilla.org/comm-central/rev/1c51aa83002e
Attachment #606953 -
Attachment description: patch → patch [Checkin: Comment 3]
Assignee | ||
Updated•13 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.11
You need to log in
before you can comment on or make changes to this bug.
Description
•