In issue summaries you are presented with a variety of formatting options, but the del (strike through) option doesn't do anything if it contains a heading.

Examples of all styles:

Start strike through (del)
Bold (strong)

Heading 2

End of strike through

Comments

ianthomas_uk’s picture

ianthomas_uk
he/him
Primary language English
Location Brighton, UK
CreditAttribution: ianthomas_uk commented
Title: Del (strike through) style no longer works in issue summaries » Del (strike through) style no longer works when it hits a heading
Priority: Normal » Minor
Issue summary: View changes

My diagnosis was slightly wrong, correcting issue summary

TR’s picture

TR CreditAttribution: TR commented

Yes, but perhaps this is an issue for the WYSIWYG editor being used, rather than for the D7 upgrade queue?

tvn’s picture

tvn CreditAttribution: tvn commented
Issue tags: +Drupal.org 7.1
kalman.hosszu’s picture

kalman.hosszu CreditAttribution: kalman.hosszu commented
Status: Active » Needs review

I checked it in a new installed D7 site and I expert the same when "Convert line breaks into HTML (i.e.
and

)" filter is enabled. Otherwise if I use valid html tags (use the del tag inside the header tag) it works well with the filter, so I think it's not a system but a content editor bug.

drumm’s picture

drumm
he/him
Location NY, US
CreditAttribution: drumm commented
Status: Needs review » Active

I expect this is the "Correct faulty and chopped off HTML" text filter that is new in D7. This is implemented with PHP's DOMDocument.

HTML5 allows headings to be nested inside inline elements, like del, but this wasn't previously true. filter_dom_load() uses an XHTML 1.0 doctype. I think this should be fixed in core. As always, it needs to be fixed in D8 and backported.

sun’s picture

sun
Primary language German
Location Karlsruhe
CreditAttribution: sun commented
Status: Active » Closed (won't fix)
  1. filter_dom_load() uses DOMDocument, and it uses loadXML(), because D7 still uses XHTML1-Strict, which essentially means XML, not HTML. This won't change for D7.
  2. Even if core was hacked to use loadHtml() (NOT recommended), PHP's DOMDocument still implements DOM Level 2, even in its latest version.