Opened 14 years ago

Closed 14 years ago

#356 closed defect (fixed)

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 (1)

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

Download all attachments as: .zip

Change History (5)

Changed 14 years ago by roy@…

modifying strike to del

comment:1 Changed 14 years ago by anonymous

  • Severity changed from minor to normal

comment:2 Changed 14 years ago by ursm@…

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

comment:3 Changed 14 years ago by gogo

  • Milestone set to Version 1.0
  • Summary changed from Dropping <strike> for semantically rich <del> to Patch: Dropping <strike> for semantically rich <del>
  • Version set to trunk

comment:4 Changed 14 years ago by gogo

  • Resolution set to fixed
  • Status changed from new to closed

Applied in changeset:275

Note: See TracTickets for help on using tickets.