Bug 397990

Summary: Carousel images too large
Product: [Applications] digikam Reporter: Pat David <patdavid>
Component: WebsiteAssignee: Digikam Developers <digikam-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: caulier.gilles, patdavid
Priority: NOR    
Version First Reported In: unspecified   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed/Implemented In: 6.0.0
Sentry Crash Report:

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(-)