Bug 397990 - Carousel images too large
Summary: Carousel images too large
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Website (show other bugs)
Version: unspecified
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-08-28 14:59 UTC by Pat David
Modified: 2018-08-28 15:42 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 6.0.0
Sentry Crash Report:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Pat David 2018-08-28 14:59:47 UTC 
Gilles,

The carousel images are all .PNG files.  This is awesome for quality.  Not so awesome for file size... :(

Loading the front page requires 13.2MB of download (almost 2 minutes to load over fast 3G)!

I would like to:

1. Compress these to JPG (trying to keep as much visual quality as possible while keeping the file sizes smaller).
2. Consider reducing the number of images in the carousel to cut down on total download size.

Can I go ahead and compress the carousel images?
Comment 1 caulier.gilles 2018-08-28 15:15:44 UTC 
You are right.

Using JPEG will reduce bandwidth. Let's go to convert PNG to JPEG with imagemagick...

Gilles
Comment 2 Pat David 2018-08-28 15:16:42 UTC 
(In reply to caulier.gilles from comment #1)
> You are right.
> 
> Using JPEG will reduce bandwidth. Let's go to convert PNG to JPEG with
> imagemagick...
> 
> Gilles

Excellent!  I'll convert them now and push the change.
Comment 3 Pat David 2018-08-28 15:40:41 UTC 
commit 8ed4b3279c6a946a857e12077cf7c3d32a6c407c (HEAD -> dev, origin/dev)
Author: Pat David <patdavid@gmail.com>
Date:   Tue Aug 28 10:36:19 2018 -0500

    Compress carousel images from PNG to JPG

    This also removes the /static/img/carousel/info.txt file as it is
    no longer needed (use /data/carousel.yml intead).

 data/carousel.yml                   |  28 ++++++++++++++--------------
 static/img/carousel/carousel-01.jpg | Bin 0 -> 195135 bytes
 static/img/carousel/carousel-02.jpg | Bin 0 -> 134058 bytes
 static/img/carousel/carousel-03.jpg | Bin 0 -> 148632 bytes
 static/img/carousel/carousel-04.jpg | Bin 0 -> 118731 bytes
 static/img/carousel/carousel-05.jpg | Bin 0 -> 113304 bytes
 static/img/carousel/carousel-06.jpg | Bin 0 -> 85932 bytes
 static/img/carousel/carousel-07.jpg | Bin 0 -> 131056 bytes
 static/img/carousel/carousel-08.jpg | Bin 0 -> 85323 bytes
 static/img/carousel/carousel-09.jpg | Bin 0 -> 137596 bytes
 static/img/carousel/carousel-10.jpg | Bin 0 -> 111399 bytes
 static/img/carousel/carousel-11.jpg | Bin 0 -> 114148 bytes
 static/img/carousel/carousel-12.jpg | Bin 0 -> 97692 bytes
 static/img/carousel/carousel-13.jpg | Bin 0 -> 114397 bytes
 static/img/carousel/carousel-14.jpg | Bin 0 -> 79370 bytes
 static/img/carousel/info.txt        |  14 --------------
 16 files changed, 14 insertions(+), 28 deletions(-)