Bug #49051

CSS Styled Content receives double stdWrap

Added by Francois Suter about 8 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Should have
Category:
Content Rendering
Target version:
Start date:
2013-06-12
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
6.0
PHP Version:
5.3
Tags:
Complexity:
easy
Is Regression:
Sprint Focus:

Description

Since TYPO3 CMS 6.0, USER objects have a stdWrap property. css_styled_content already supported stdWrap and applied it in each of its "render" methods. This results now in stdWrap being applied twice.

The solution is to simply remove the call to stdWrap inside css_styled_content.

#1

Updated by Francois Suter about 8 years ago

How to test. Add something like this to your template:

tt_content.image.20.stdWrap.wrap = <div style="padding: 4px; border: 1px solid red">|</div>

and observe that all image-type content elements get a double border, instead of a single one.

#2

Updated by Gerrit Code Review about 8 years ago

  • Status changed from Accepted to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/21358

#3

Updated by Gerrit Code Review about 8 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/21358

#4

Updated by Francois Suter about 8 years ago

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

Updated by Gerrit Code Review about 8 years ago

  • Status changed from Resolved to Under Review

Patch set 1 for branch TYPO3_6-1 has been pushed to the review server.
It is available at https://review.typo3.org/21362

#6

Updated by Gerrit Code Review about 8 years ago

Patch set 1 for branch TYPO3_6-0 has been pushed to the review server.
It is available at https://review.typo3.org/21364

#7

Updated by Gerrit Code Review about 8 years ago

Patch set 2 for branch TYPO3_6-0 has been pushed to the review server.
It is available at https://review.typo3.org/21364

#8

Updated by Francois Suter about 8 years ago

  • Status changed from Under Review to Resolved
#9

Updated by Benni Mack over 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF