Ticket #356 (closed defect: fixed)

Opened 8 years ago

Last modified 8 years ago

Patch: Dropping <strike> for semantically rich <del>

Reported by: roy@… Owned by: gogo
Priority: normal Milestone: Version 1.0
Component: Xinha Core Version: trunk
Severity: normal Keywords:
Cc:

Description

AFAIK, <strike> is in the same family of tags like <b> and <i>, while technically still valid, are being slowly falling out of favor for the more semantically rich <del>. I've fixed this in my local copy, but I'm not sure how to send it to you guys... line 4049 in htmlarea.js:

  html = html.replace(/<(\/?)strike(\s|>|\/)/ig, "<$1del$2");

line 4078 in htmlarea.js:

    html = html.replace(/<(\/?)del(\s|>|\/)/ig, "<$1strike$2");

line 4532 in htmlarea.js:

HTMLArea._closingTags = " head script style div span tr td tbody table em strong del b i strike code cite dfn abbr acronym font a title textarea select form ";

Attachments

htmlarea (1.5 kB) - added by roy@… 8 years ago.
modifying strike to del

Change History

Changed 8 years ago by roy@…

modifying strike to del

Changed 8 years ago by anonymous

  • severity changed from minor to normal

Changed 8 years ago by ursm@…

Please apply this patch, or ask for clarification, should be straight forward

Changed 8 years ago by gogo

  • summary changed from Dropping <strike> for semantically rich <del> to Patch: Dropping <strike> for semantically rich <del>
  • version set to trunk
  • milestone set to Version 1.0

Changed 8 years ago by gogo

  • status changed from new to closed
  • resolution set to fixed

Applied in changeset:275

Note: See TracTickets for help on using tickets.