WordPress.org
Get WordPress

Make WordPress Core

Opened 21 years ago

Closed 20 years ago

Last modified 19 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: Focuses:

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 21 years ago.
Replacement of strike with del in allowed comment tags

Download all attachments as: .zip

Change History (9)

@Viper007Bond
21 years ago

Replacement of strike with del in allowed comment tags

#1 @davidhouse
21 years ago

  • Keywords bg|2nd-opinion added

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

#2 @markjaquith
21 years ago

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

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

+1

#3 @matt
20 years ago

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

#4 @markjaquith
20 years ago

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

#5 @foolswisdom
20 years ago

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

#6 @Nazgul
20 years ago

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

#7 @matt
20 years ago

  • Resolution → wontfix
  • Status assigned → closed

#8 @Nazgul
19 years ago

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