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)
Change History (13)
Changed 9 years ago by
Attachment: | image styles.pdf added |
---|
comment:1 Changed 9 years ago by
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
15KB refers to style openscience_square_col2, which is custom made for us
comment:3 Changed 9 years ago by
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
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
Priority: | major → minor |
---|
comment:6 Changed 8 years ago by
Owner: | changed from Bas van der Veen to Sander Maijers |
---|---|
Status: | new → assigned |
comment:7 Changed 8 years ago by
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
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:10 Changed 8 years ago by
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
Owner: | changed from Bas van der Veen to Dieter Van Uytvanck |
---|
comment:12 Changed 6 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Closing this one, since we have not seen similar issues after the major clean-up and migration of the website.
image styles on clarin.eu