Bug 6175 - images have needlessly high depth
Summary: images have needlessly high depth
Status: CLOSED FIXED
Alias: None
Product: ThinLinc
Classification: Unclassified
Component: Other (show other bugs)
Version: trunk
Hardware: PC Unknown
: P2 Normal
Target Milestone: 4.8.0
Assignee: Pierre Ossman
URL:
Keywords: hean01_tester, prosaic
Depends on:
Blocks: 6132
  Show dependency treegraph
 
Reported: 2017-02-23 17:01 CET by Pierre Ossman
Modified: 2017-03-20 16:54 CET (History)
2 users (show)

See Also:
Acceptance Criteria:


Attachments

Description Pierre Ossman cendio 2017-02-23 17:01:49 CET
For some reason most of our images have been created with a depth of 16 bits per channel:

> tlclient_192.png: PNG image data, 192 x 192, 16-bit/color RGBA, non-interlaced

The reason is probably because they are generated from a SVG and include gradients. ImageMagick then likely concludes it cannot reduce the depth without losing some information.

But this is overkill for us as these images will not be used in places where such precision is needed. We can save quite a bit of space by reducing things to the more standard 8 bits per channel. In the case of tlclient_192.png the difference is 59722 bytes vs 14886 bytes.
Comment 3 Pierre Ossman cendio 2017-02-23 17:29:13 CET
Tester should check that images still look okay:

 - tlclient icon on Linux
 - webaccess icon
 - tlwebadm icon
 - big logo in tlwebadm
 - logo in tl-xstartup-feedback
 - logo in profile selector
 - logo in rdesktop launcher
Comment 6 Henrik Andersson cendio 2017-02-28 14:48:31 CET
(In reply to comment #3)

Verified using lxde (OpenBox)

>  - tlclient icon on Linux

Looks good

>  - webaccess icon
>  - tlwebadm icon

Looks good

>  - big logo in tlwebadm

Looks good, except for the clipped antialiasing, however, this is not an regression.

>  - logo in tl-xstartup-feedback
>  - logo in profile selector
>  - logo in rdesktop launcher

Looks good

Note You need to log in before you can comment on or make changes to this bug.