Opened 15 months ago

Closed 13 months ago

Last modified 4 months ago

#20149 closed defect (bug) (invalid)

Del button should work the same in Visual & HTML Editors

Reported by: aaroncampbell Owned by:
Priority: normal Milestone:
Component: Editor Version: 3.3
Severity: minor Keywords:
Cc: stas@…, aaroncampbell, mark@…

Description

If you highlight something in the visual editor and click strikethrough (del) it wraps the content in <del> tags. If you do the same in the HTML editor it gets wrapped in <del> tags with the datetime parameter filled with what looks like an ISO 8601 date.

Example:

<del>Deleted from Visual</del>

<del datetime="2012-03-01T13:56:01+00:00">Deleted from HTML</del>

We should standardize these.

One of the side effects of this is that the del button in the HTML editor can't be used in captions because it messes up the quotes (#18311).

Attachments (1)

20149.diff (1.6 KB) - added by sushkov 15 months ago.

Download all attachments as: .zip

Change History (12)

The same is true for the <ins> tag. The datetime attribute for <del> and <ins> doesn't seem to be displayed or used by the browsers so removing them wouldn't affect anything.

  • Cc stas@… added
  • Keywords has-patch added
  • Cc aaroncampbell added

Fine with removing the timestamp if it makes things easier. Easy enough to add datetime with a plugin if someone really wants that attribute.

  • Milestone changed from Awaiting Review to 3.4
  • Keywords commit added; 2nd-opinion removed

This patch looks good to go.

I think removing the timestamp here is a retrograde move and would rather we didn't regress this.

If captions are breaking we need to fix captions, not this.

  • Keywords has-patch commit removed
  • Resolution set to invalid
  • Status changed from new to closed

We're going to fix captions instead of regressing here

  • Milestone 3.4 deleted
  • Cc mark@… added
  • Severity changed from normal to minor

IMO this is wrong. Semantically (http://www.whatwg.org/specs/web-apps/current-work/multipage/edits.html) the del tag have less meaning without the deletion time. I guess it is OK if WordPress do not make it easy to provide the info at all, but to provide it in the less used editor component while ignoring the more used one is strange.

related #23321

Note: See TracTickets for help on using tickets.