Opened 7 years ago

Closed 6 years ago

Last modified 6 years ago

#2470 closed defect (bug) (wontfix)

<strike> sucks, <del> does not

Reported by: Viper007Bond Owned by: markjaquith
Priority: low Milestone:
Component: Administration Version: 2.0.1
Severity: minor Keywords: has-patch
Cc:

Description

<strike> was deprecated in HTML 4.01 and is not supported XHTML 1.0 Strict DTD or XHTML 1.1, so it shouldn't be allowed in comments, at least by default.

Attachments (1)

no_more_strike.diff​ (390 bytes) - added by Viper007Bond 7 years ago.
Replacement of strike with del in allowed comment tags

Download all attachments as: .zip

Change History (9)

Viper007Bond — 7 years ago

Replacement of strike with del in allowed comment tags

comment:1   davidhouse — 7 years ago

  • Keywords bg|2nd-opinion added

+1, but I'm marking with 2nd-opinion.

comment:2   markjaquith — 7 years ago

  • Keywords bg|commit added; bg|2nd-opinion removed
  • Milestone set to 2.1
  • Owner changed from anonymous to markjaquith
  • Priority changed from normal to low
  • Severity changed from normal to minor
  • Status changed from new to assigned

If it breaks XHTML 1.0 strict validation, we should axe it.

+1

comment:3   matt — 7 years ago

The problem is del is both an inline and block level tag, and can seriously mess up page layouts if allowed.

comment:4   markjaquith — 7 years ago

Same problem with &lt;ins&gt; ... currently, you can't use &lt;ins&gt; as block level in WP.

comment:5   foolswisdom — 7 years ago

Is the appropriate thing just to remove <strike> and not replace it with <del>?

comment:6   Nazgul — 7 years ago

  • Keywords has-patch added; bg|has-patch bg|commit removed

comment:7   matt — 6 years ago

  • Resolution set to wontfix
  • Status changed from assigned to closed

comment:8   Nazgul — 6 years ago

  • Milestone 2.1 deleted
Note: See TracTickets for help on using tickets.