Bug #25083

Content rendering: Definition lists inserted in RTE are wrapped in paragraphs

Added by Christian Hennecke over 11 years ago. Updated about 10 years ago.

Status:
Closed
Priority:
Should have
Category:
Content Rendering
Target version:
Start date:
2011-02-18
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
4.5
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

By default, definition lists entered in the RTE are wrapped in paragraph tags when rendered in the frontend. This is not valid HTML.

Currently, the solution is to add something like

lib.parseFunc_RTE.nonTypoTagStdWrap.encapsLines.encapsTagList := addToList(dl, dt, dd)

to the TS setup of the template. However, standard HTML elements should work out of the box.
(issue imported from #M17648)


Files

typo3core_bugfix_17648_trunk.patch (1.81 KB) typo3core_bugfix_17648_trunk.patch Administrator Admin, 2011-02-21 04:05
typo3core_bugfix_17648_typo3_4-4.patch (983 Bytes) typo3core_bugfix_17648_typo3_4-4.patch Administrator Admin, 2011-02-21 04:23
#1

Updated by Stanislas Rolland over 11 years ago

I would not add the dd element to this list but treat it rather like an external block because it may contain block-level elements.

#2

Updated by Stanislas Rolland over 11 years ago

Please test attached patch.

#3

Updated by Susanne Moog over 11 years ago

  • Target version deleted (4.4.7)
#4

Updated by Stanislas Rolland over 11 years ago

  • Category set to RTE (rtehtmlarea + ckeditor)
  • TYPO3 Version set to 4.5
#5

Updated by Stanislas Rolland over 11 years ago

  • TYPO3 Version changed from 4.5 to 4.4
#6

Updated by Mr. Hudson about 11 years ago

Patch set 1 of change Ia73d01105beeaba59f91dd87754c7528796d4d07 has been pushed to the review server.
It is available at http://review.typo3.org/4181

#7

Updated by Mr. Hudson about 11 years ago

Patch set 1 of change I7465c48d53d255985a1638a425020392022ffdb7 has been pushed to the review server.
It is available at http://review.typo3.org/4182

#8

Updated by Mr. Hudson about 11 years ago

Patch set 1 of change I48229b7835a67fc690e810eaf5a54df506fb2add has been pushed to the review server.
It is available at http://review.typo3.org/4184

#9

Updated by Stanislas Rolland almost 11 years ago

  • Status changed from New to Under Review
#10

Updated by Stanislas Rolland almost 11 years ago

  • Category changed from RTE (rtehtmlarea + ckeditor) to Content Rendering
  • Assignee set to Stanislas Rolland
  • Target version set to 1341
#11

Updated by Gerrit Code Review over 10 years ago

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

#12

Updated by Gerrit Code Review over 10 years ago

Patch set 2 for branch TYPO3_4-5 has been pushed to the review server.
It is available at http://review.typo3.org/4182

#13

Updated by Gerrit Code Review over 10 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/4181

#14

Updated by Gerrit Code Review over 10 years ago

Patch set 4 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/4181

#15

Updated by Gerrit Code Review over 10 years ago

Patch set 5 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/4181

#16

Updated by Gerrit Code Review over 10 years ago

Patch set 6 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/4181

#17

Updated by Gerrit Code Review over 10 years ago

Patch set 7 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/4181

#18

Updated by Stanislas Rolland over 10 years ago

  • Target version changed from 1341 to 4.5.12
#19

Updated by Gerrit Code Review over 10 years ago

Patch set 1 for branch TYPO3_4-6 has been pushed to the review server.
It is available at http://review.typo3.org/9077

#20

Updated by Gerrit Code Review over 10 years ago

Patch set 1 for branch TYPO3_4-5 has been pushed to the review server.
It is available at http://review.typo3.org/9078

#21

Updated by Stanislas Rolland over 10 years ago

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

Updated by Gerrit Code Review over 10 years ago

  • Status changed from Resolved to Under Review

Patch set 1 for branch TYPO3_4-6 has been pushed to the review server.
It is available at http://review.typo3.org/9229

#23

Updated by Gerrit Code Review over 10 years ago

Patch set 1 for branch TYPO3_4-5 has been pushed to the review server.
It is available at http://review.typo3.org/9236

#24

Updated by Wouter Wolters about 10 years ago

Can be closed. The patches were for the backported version of 4.5 and 4.6. AFAIK they will never be released

#25

Updated by Ernesto Baschny about 10 years ago

  • Status changed from Under Review to Closed
  • TYPO3 Version changed from 4.4 to 4.5

Nevertheless the bugfix was merged in 4.5.12 and 4.6.5. Case closed!

Also available in: Atom PDF