Feature #48013

Add support for progressive jpg files

Added by most wanted over 6 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Should have
Category:
Image Generation / GIFBUILDER
Target version:
-
Start date:
2013-05-07
Due date:
% Done:

100%

PHP Version:
Tags:
Complexity:
medium
Sprint Focus:

Description

Would be great if processed jpg files would be progressive instead of baseline jpg files.

Associated revisions

Revision f7b516d0 (diff)
Added by Richard Haeser over 1 year ago

[FEATURE] Add interlace option for image conversion

When the option $GLOBALS['TYPO3_CONF_VARS'][GFX][processor_interlace]
is set, all images will be generated with the given interlace scheme.

Resolves: #48013
Releases: master
Change-Id: I9b78f8f27dd5223ac8e96ff33d1c26b5d1d6e1c9
Reviewed-on: https://review.typo3.org/47753
Tested-by: TYPO3com <>
Reviewed-by: Mathias Schreiber <>
Tested-by: Mathias Schreiber <>
Reviewed-by: Frank Naegler <>
Tested-by: Frank Naegler <>

History

#1 Updated by Thorsten Kahler over 6 years ago

  • Tracker changed from Bug to Feature
  • Status changed from New to Accepted
  • Complexity set to medium

#2 Updated by Ernesto Baschny almost 6 years ago

  • Target version deleted (6.2.0)

#3 Updated by Stefan Neufeind over 5 years ago

For ImageMagick the parameter "-interlace" needs to be passed to "convert". One possible addition would be to also use "-strip" since that will remove "profiles and comments".

How do we want to implement it? Additional setting (unchecked by default) in the GFX-options in installtool?

Or how about we string-settings for "convert-options" (and possibly combine-options etc. as well) for IM/GM so users could add whatever params they'd like to use? We could hint to -interlace and -strip in the helptext in installtool.

As a rough overview: convert is called in these places, which would need to get the additional parameter(s) passed.
typo3/sysext/core/Classes/Imaging/GraphicalFunctions.php
typo3/sysext/core/Classes/Resource/Processing/LocalPreviewHelper.php
typo3/sysext/core/Classes/Utility/GeneralUtility.php
typo3/sysext/core/Classes/Utility/GeneralUtility.php
typo3/sysext/core/Classes/Utility/GeneralUtility.php
typo3/sysext/backend/Classes/View/ThumbnailView.php

#4 Updated by Robert Rentz about 4 years ago

Is there any progress hot to implement it? I think, it would be a great feature.

#5 Updated by Jorgen van der Kroon about 4 years ago

Would be a nice feature.

#7 Updated by Maximilian Anton Grimm almost 4 years ago

+1 but Target version should also be 6.2.x

#8 Updated by Stefan Neufeind almost 4 years ago

It's a new feature, so can't go into older releases. Especially not 6.2 since it's not really a "priority bugfix".

#9 Updated by Richard Haeser over 3 years ago

  • Assignee set to Richard Haeser

#10 Updated by Gerrit Code Review over 3 years ago

  • Status changed from Accepted to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/47753

#11 Updated by Gerrit Code Review over 3 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/47753

#12 Updated by Gerrit Code Review over 3 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/47753

#13 Updated by Gerrit Code Review over 3 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/47753

#14 Updated by Gerrit Code Review over 3 years ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/47753

#15 Updated by Gerrit Code Review over 3 years ago

Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/47753

#16 Updated by Michael Fruehmann about 3 years ago

Any updates on this? is there a target version in which we can expect this feature to work?

#17 Updated by Susanne Moog almost 2 years ago

  • Category changed from Image Cropping to Image Generation / GIFBUILDER

#18 Updated by Gerrit Code Review over 1 year ago

Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/47753

#19 Updated by Gerrit Code Review over 1 year ago

Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/47753

#20 Updated by Gerrit Code Review over 1 year ago

Patch set 9 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/47753

#21 Updated by Richard Haeser over 1 year ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#22 Updated by Benni Mack about 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF