Project

General

Profile

Actions

Feature #75309

closed

Epic #77810: Improve (responsive) image handling

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

Added by Bernhard Eckl over 8 years ago. Updated almost 3 years ago.

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

10%

Estimated time:
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 1 (0 open1 closed)

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

Actions
Actions #1

Updated by Riccardo De Contardi over 8 years ago

  • Category set to Backend User Interface
Actions #2

Updated by Xavier Perseguers over 8 years ago

You are thus thinking about adding a destructive operation?

Actions #3

Updated by Bernhard Eckl over 8 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.

Actions #4

Updated by Job Rutgers over 8 years ago

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

Actions #5

Updated by Frans Saris about 8 years ago

  • Parent task set to #77810
Actions #6

Updated by Frans Saris about 8 years ago

  • Description updated (diff)
Actions #7

Updated by Sascha Egerer about 8 years ago

  • Status changed from New to Accepted
  • Assignee set to Sascha Egerer
Actions #8

Updated by Gerrit Code Review about 8 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

Actions #9

Updated by Gerrit Code Review about 8 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

Actions #10

Updated by Gerrit Code Review about 8 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

Actions #11

Updated by Gerrit Code Review about 8 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

Actions #12

Updated by Gerrit Code Review about 8 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

Actions #13

Updated by Gerrit Code Review about 8 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

Actions #14

Updated by Gerrit Code Review almost 8 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

Actions #15

Updated by Gerrit Code Review almost 8 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

Actions #16

Updated by Gerrit Code Review almost 8 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

Actions #17

Updated by Gerrit Code Review almost 8 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

Actions #18

Updated by Gerrit Code Review almost 8 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

Actions #19

Updated by Sascha Egerer over 7 years ago

  • Sprint Focus set to On Location Sprint
Actions #20

Updated by Sascha Egerer over 7 years ago

  • Target version set to Candidate for Major Version
Actions #21

Updated by Gerrit Code Review over 7 years 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

Actions #22

Updated by Sascha Egerer over 7 years 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.

Actions #23

Updated by Sascha Egerer over 7 years 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
Actions #24

Updated by Anja Leichsenring about 7 years ago

  • Sprint Focus deleted (On Location Sprint)
Actions #25

Updated by Susanne Moog about 6 years ago

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

Updated by Susanne Moog over 5 years ago

  • Status changed from Under Review to New

Review has been abandoned.

Actions #27

Updated by Christian Kuhn almost 3 years ago

  • Status changed from New to Closed

Hey. I'm going to close this feature request for now since a PoC implementation failed due to various conceptual issues (for instance that cropping is usually bound to the consuming element).

This issue could be picked up again, but I'm unsure if the list module is actually the right place to do these things.

Actions

Also available in: Atom PDF