Bug #89377
closedImagedescription visible in Backend, even if the image is set to hidden
0%
Description
Steps to reproduce:
1. Go to page module and create a new "Text & Media" Content Element.
2. Fill out the Fields "Bodytext", "Media Elements" and the Description of the Media element
3. Make sure to disable the Media Element
4. Save and Close the record
Expected behavior:
As a backend editor i don't want to see the description of a hidden image in the Page module
Files
Updated by Gerrit Code Review about 5 years ago
- Status changed from New 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/c/Packages/TYPO3.CMS/+/61961
Updated by Georg Ringer about 5 years ago
- Status changed from Under Review to Needs Feedback
I am not so sure about this change. IMO we don't have anywhere in the backend conditions based on hidden if something should be shown or not. therefore I like to avoid having now a different behaviour for this one field.
IMO it would be more important to have a good concept for the complete page module instead of fiddling around with one setting here and one setting there. do you agree?
Updated by Dennis Römmich almost 5 years ago
Georg Ringer wrote:
IMO we don't have anywhere in the backend conditions based on hidden if something should be shown or not. therefore I like to avoid having now a different behaviour for this one field.
There is actually a condition based on hidden in https://github.com/TYPO3/TYPO3.CMS/blob/TYPO3_8-7/typo3/sysext/backend/Classes/Utility/BackendUtility.php#L1706
This condition is the reason why we don't see the image in page module.
Georg Ringer wrote:
IMO it would be more important to have a good concept for the complete page module instead of fiddling around with one setting here and one setting there. do you agree?
I totally agree, "good concept" always sounds great :) But until we have a good concept, we should merge this, imo.
Updated by Benni Mack over 4 years ago
- Sprint Focus changed from PRC to Needs Decision
Updated by Benni Mack 4 months ago
- Status changed from Needs Feedback to Closed
Hey,
I could not reproduce this in all supported versions, so finally this problem has been solved!