Opened 9 years ago

Closed 6 years ago

#824 closed defect (fixed)

overactive compression on JPEGs

Reported by: Dieter Van Uytvanck Owned by: Dieter Van Uytvanck
Priority: minor Milestone:
Component: Website Version:
Keywords: Cc:

Description

The overall JPEG picture quality of the pictures on the site is suboptimal. In the process of uploading pictures there seems to be an overactive compression.

I increased the JPEG quality parameter to 100%. Now the quality is better, but still additional compression takes place. E.g. uploading a JPEG of 26 KB (perfectly acceptable, good quality) gets scaled down to 15 KB.

Especially when the uploaded JPEG matches the size requirement, an additional compression step is senseless.

Attachments (1)

image styles.pdf (563.3 KB) - added by Bas van der Veen 9 years ago.
image styles on clarin.eu

Download all attachments as: .zip

Change History (13)

Changed 9 years ago by Bas van der Veen

Attachment: image styles.pdf added

image styles on clarin.eu

comment:1 Changed 9 years ago by Bas van der Veen

Can you send me an url, where this occurs? Because we use almost 20 image styles on the website the problem is hard to track

comment:2 Changed 9 years ago by Bas van der Veen

15KB refers to style openscience_square_col2, which is custom made for us

comment:3 Changed 9 years ago by Bas van der Veen

Increasing JPEG quality to 100% on [admin/config/media/image-toolkit] is only part of the solution. All image styles have been made on purpose and contain many (undocumented) settings, so this needs further analysis. A good start will be to list all styles and where these are used on the website.

comment:4 Changed 9 years ago by Dieter Van Uytvanck

dvuytvanck@vz07-clarin-eu:/hum/web/www.clarin.eu/htdocs/sites/default/files/styles$ find . -iname \*.jpg

indicates the following styles are used with pictures:

  • openscience_wide_8col
  • openscience_2_col
  • openscience_9col
  • openscience_square_2col
  • openscience_12col
  • openscience_square_1col
  • openscience_5col
  • openscience_4col
  • openscience_wide_3col
  • openscience_3col
  • openscience_2col

comment:5 Changed 8 years ago by Sander Maijers

Priority: majorminor

comment:6 Changed 8 years ago by Sander Maijers

Owner: changed from Bas van der Veen to Sander Maijers
Status: newassigned

comment:7 Changed 8 years ago by Sander Maijers

Owner: changed from Sander Maijers to Bas van der Veen

Bas will handle the old tickets. Sorry, forgot.

comment:8 Changed 8 years ago by Bas van der Veen

The logic about image styles is kept hidden in code (see ../admin/structure/features/openscience_ui) and shown through definitions in views (for ex. see ../admin/structure/views/view/openscience_popular_content/edit), where configuration defines image content with style 'openscience_wide_3col'. The file openscience_ui.views_default.inc contains image style definitions. For example 'https://www.clarin.eu/admin/config/media/image-styles/edit/openscience_wide_3col' shows Scale and crop 255x140, which is defined in:
..
$stylesopenscience_wide_3col? = array(

'name' => 'openscience_wide_3col',

..
To solve this issue it should be made clear which images styles pose a problem and should be changed. To help here's a list where styles are applied in views (=human_name) on clarin.eu:

openscience_wide_8col OpenScience? UI
openscience_2_col CLARIN ERIC Portal Views
openscience_square_1col Featured LRT Showcase
openscience_4col CLARIN ERIC Portal Views
openscience_wide_3col OpenScience? Popular
openscience_2col CLARIN ERIC Portal Views


not found: openscience_12col
not used in views: openscience_9col + openscience_square_2col + openscience_5col + openscience_3col

comment:9 Changed 8 years ago by Sander Maijers

Status? Can we get rid of the old image styles?

comment:10 Changed 8 years ago by Dieter Van Uytvanck

I have the impression this has improved with the new setup (especially when the style of images in content types is 'original' in stead of 'rendered', but needs to be double checked.

As to the old image styles, we should inventorize where they are still used, change that, and then remove them.

comment:11 Changed 6 years ago by Dieter Van Uytvanck

Owner: changed from Bas van der Veen to Dieter Van Uytvanck

comment:12 Changed 6 years ago by Dieter Van Uytvanck

Resolution: fixed
Status: assignedclosed

Closing this one, since we have not seen similar issues after the major clean-up and migration of the website.

Note: See TracTickets for help on using tickets.