After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 700351 - wasteful images
wasteful images
Status: RESOLVED OBSOLETE
Product: empathy
Classification: Core
Component: User Guide
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Sindhu S
empathy-maint
Depends on:
Blocks:
 
 
Reported: 2013-05-15 01:04 UTC by Matthias Clasen
Modified: 2018-05-22 16:09 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Fix image:conf_overview in overview.page (809.90 KB, patch)
2013-05-15 03:02 UTC, Sindhu S
none Details | Review
Fix image:croom_overview in overview.page (547.36 KB, patch)
2013-05-15 03:03 UTC, Sindhu S
none Details | Review
Fix image:video_overview in overview.page (312.04 KB, patch)
2013-05-15 03:03 UTC, Sindhu S
none Details | Review

Description Matthias Clasen 2013-05-15 01:04:30 UTC
The empathy help includes several big pngs: 

figures/video_overview.png
figures/conf_overview.png
figures/croom_overview.png

These are over 500K each, but are loaded at a small size of 150x150 in the docs.

Please run 

convert conf_overview.png -resize 150x150 out.png; mv out.png conf_overview.png

and so on, on these; it'll save quite a bit of space on space-constrained live images.
Comment 1 Sindhu S 2013-05-15 03:02:35 UTC
Created attachment 244267 [details] [review]
Fix image:conf_overview in overview.page
Comment 2 Sindhu S 2013-05-15 03:03:21 UTC
Created attachment 244268 [details] [review]
Fix image:croom_overview in overview.page
Comment 3 Sindhu S 2013-05-15 03:03:52 UTC
Created attachment 244269 [details] [review]
Fix image:video_overview in overview.page
Comment 4 Sindhu S 2013-05-15 03:05:17 UTC
I have split the patch into 3 and uploaded here because bugzilla wouldn't take patches bigger than 1600KB.

For the full single patch of all the above three changes see: http://paste.ubuntu.com/5666392/plain/

I locally have the full single patch versions so I can push to master when confirmed. Thank you.
Comment 5 Kat 2013-05-15 08:56:58 UTC
Looking at the help page, those images are currently quite small[1] which makes them actually very difficult to see, so they should be made larger on the help page.

Instead of scaling, the correct fix here should be to use thumbnails as these are already small images.

There is already an art bug filed for these, once that is fulfilled, the file sizes should be smaller than a scanned drawing.

[1] https://help.gnome.org/users/empathy/3.9/overview.html.en
Comment 6 Sindhu S 2013-05-15 09:03:35 UTC
Why s(In reply to comment #5)
> Looking at the help page, those images are currently quite small[1] which makes
> them actually very difficult to see, so they should be made larger on the help
> page.

> Instead of scaling, the correct fix here should be to use thumbnails as these
> are already small images.

Yes, can one link to the larger versionn of the image from the mallard based help page? If so this can be done by us.

In that case the 150x150 can be retained as thumbnails (renaming the files to reflect this status) and original sizes can be kept for the bigger view (via the link which we will include in the help page).
Comment 7 Matthias Clasen 2013-05-16 01:02:26 UTC
That doesn't really satisfy my original intention of saving space.
If you really want to keep the big images, please consider converting them to jpegs - that saves  quite a bit of space too.
Comment 8 Guillaume Desmottes 2013-12-04 16:32:32 UTC
Could someone from the doc team review those patches? thanks!
Comment 9 André Klapper 2015-01-13 18:05:58 UTC
Still valid: https://git.gnome.org/browse/empathy/tree/help/C/figures
Comment 10 GNOME Infrastructure Team 2018-05-22 16:09:24 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to GNOME's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.gnome.org/GNOME/empathy/issues/689.