Feature #75309

Epic #77810: Improve (responsive) image handling

Add image manipulation (crop) to filelist itself and edit original image

Added by Bernhard Eckl over 2 years ago. Updated 3 months ago.

Status:
Under Review
Priority:
Should have
Assignee:
-
Category:
Backend User Interface
Start date:
2016-03-30
Due date:
% Done:

10%

PHP Version:
5.6
Tags:
Complexity:
hard
Sprint Focus:

Description

With TYPO3 7.6 you can crop an image in sys_file_reference (e.g. in a content element), but you can not crop the image directly in the module file list. This would be a very useful feature.

Small summary:

Add a ImageManipulation field to sys_file and use this as default value in sys_file_reference


Related issues

Related to TYPO3 Core - Feature #65585: Add GUI to enable the editor to set the cropping area for an image Closed 2015-04-15

History

#1 Updated by Riccardo De Contardi over 2 years ago

  • Category set to Backend User Interface

#2 Updated by Xavier Perseguers over 2 years ago

You are thus thinking about adding a destructive operation?

#3 Updated by Bernhard Eckl over 2 years ago

I remember in the earlier days there was an extension called fdfx_be_image, you could check wether a new file will be generated or the existing will be overridden.

#4 Updated by Job Rutgers over 2 years ago

It's maybe also nice to have the option to rotate the images from the file list

#5 Updated by Frans Saris over 2 years ago

  • Parent task set to #77810

#6 Updated by Frans Saris over 2 years ago

  • Description updated (diff)

#7 Updated by Sascha Egerer over 2 years ago

  • Status changed from New to Accepted
  • Assignee set to Sascha Egerer

#8 Updated by Gerrit Code Review over 2 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/49818

#9 Updated by Gerrit Code Review over 2 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/49818

#10 Updated by Gerrit Code Review over 2 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/49818

#11 Updated by Gerrit Code Review over 2 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/49818

#12 Updated by Gerrit Code Review over 2 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/49818

#13 Updated by Gerrit Code Review over 2 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/49818

#14 Updated by Gerrit Code Review almost 2 years 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/49818

#15 Updated by Gerrit Code Review almost 2 years 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/49818

#16 Updated by Gerrit Code Review almost 2 years 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/49818

#17 Updated by Gerrit Code Review almost 2 years ago

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

#18 Updated by Gerrit Code Review almost 2 years ago

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

#19 Updated by Sascha Egerer over 1 year ago

  • Sprint Focus set to On Location Sprint

#20 Updated by Sascha Egerer over 1 year ago

  • Target version set to Candidate for Major Version

#21 Updated by Gerrit Code Review over 1 year ago

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

#22 Updated by Sascha Egerer over 1 year ago

My comment from Gerrit:


Just tried to rework this patch at the coding night but stumbled over multiple problems. I can remember that some of these problems where the main reason i stopped here.

There are some conceptional problems with this feature.

1. The file relation does always have the cropping information in the database which means that the cropping information from the file metadata is always overridden. Thats a thing that we may get fixed somehow.

2. The cropVariants can be overridden on a per field basis. That means that the cropVariants could be different on a field then the default ones defined for the file. Each crop variant can have multiple allowed aspect rartios. It is very hard to merge those settings and just allow the crop variants defined in the file if the are allowed by the configuration of the per field configuration.

Antoher idea could be to save the cropping information not in the relation and not in the file but in a seperate "cropping" table.

The cropping variants (sets) can be configured with a name. Than those croppings a created for a named set and saved in the cropping table.

On a per field basis you can then allow which cropping set you allow and not define the crop variants.

Then you have also the possibility to define the cropping in the file and they a "selectedable" in the file relation.

But i will stop working on this patch for now as it is a really big change that should be rethinked from a conceptional point of view.

#23 Updated by Sascha Egerer over 1 year ago

  • Assignee deleted (Sascha Egerer)
  • Target version changed from Candidate for Major Version to 9 LTS
  • % Done changed from 0 to 10
  • Complexity set to hard

#24 Updated by Anja Leichsenring over 1 year ago

  • Sprint Focus deleted (On Location Sprint)

#25 Updated by Susanne Moog 3 months ago

  • Target version changed from 9 LTS to Candidate for Major Version

Also available in: Atom PDF