NOTE: The current preferred location for bug reports is the GitHub issue tracker.
Bug 487 - add more obsolete elements to assertions checking and to legacy.rnc (and to driver files)
add more obsolete elements to assertions checking and to legacy.rnc (and to d...
Status: RESOLVED FIXED
Product: Validator.nu
Classification: Unclassified
Component: Schematron back end
HEAD
All All
: P2 enhancement
Assigned To: Michael[tm] Smith
Depends on:
Blocks: 481
  Show dependency treegraph
 
Reported: 2009-04-16 06:49 CEST by Michael[tm] Smith
Modified: 2009-06-26 08:07 CEST (History)
1 user (show)

See Also:


Attachments
patch with proposed changes (1.26 KB, patch)
2009-04-16 13:55 CEST, Michael[tm] Smith
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Michael[tm] Smith 2009-04-16 06:49:14 CEST
The assertions.sch and Assertions.java code is currently only reporting the following elements as obsolete: 

  big, center, font, s, strike, tt, u, acroynm, dir, applet

But the following elements are also obsoleted by the HTML5 draft:

 frame, frameset, noframes, basefont, isindex
Comment 1 Michael[tm] Smith 2009-04-16 10:12:22 CEST
Some relevant data:

http://dev.opera.com/articles/view/mama-phrase-block-list/#misc

... data on that page seems to indicate that <basefont> is used in more pages than <acronym>, <s> and <strike> are -- and in around the same order of magnitude as many pages as <tt> is. So it seems like we should be reporting <basefont>, if we are already reporting those.

But it definitely does not seem worth it to report <basefont>

http://dev.opera.com/articles/view/mama-head-structure/#head

"ISINDEX element was only found in 63 URLs" (out of 3,000,000+)
Comment 2 Michael[tm] Smith 2009-04-16 13:55:05 CEST
Created attachment 84 [details]
patch with proposed changes
Comment 3 Henri Sivonen 2009-04-16 16:02:09 CEST
Looks good. (Note that the isindex element will only end up in the infoset with the XML parser.)
Comment 4 Henri Sivonen 2009-04-16 16:02:41 CEST
BTW, the reason I didn't include basefont originally was that basefont is a void element.
Comment 5 Henri Sivonen 2009-05-20 10:46:26 CEST
For clarity, comment #3 counts as r=hsivonen