NOTE: The current preferred location for bug reports is the GitHub issue tracker.
Bug 706 - Check text content of script/@src elements ("inline documentation for external scripts")
Check text content of script/@src elements ("inline documentation for externa...
Status: RESOLVED FIXED
Product: Validator.nu
Classification: Unclassified
Component: Non-schema checkers
HEAD
All All
: P2 normal
Assigned To: Michael[tm] Smith
http://www.whatwg.org/specs/web-apps/...
Depends on:
Blocks: 697
  Show dependency treegraph
 
Reported: 2010-01-05 12:00 CET by Michael[tm] Smith
Modified: 2011-01-13 11:12 CET (History)
2 users (show)

See Also:


Attachments
patch (7.35 KB, patch)
2010-05-28 12:31 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 2010-01-05 12:00:26 CET
+++ This bug was initially created as a clone of Bug #697 +++

HTML5 has the following requirements for "Inline documentation for external scripts":

http://www.whatwg.org/specs/web-apps/current-work/multipage/scripting-1.html#inline-documentation-for-external-scripts

I'd suggest having a warning for the other (r)cdata elements that don't match the ABNF for style. I'd also suggest having a warning for script that goes into double escaped but leaves it with "-->" rather than "</script>".
Comment 1 Michael[tm] Smith 2010-05-28 12:31:59 CEST
Created attachment 174 [details]
patch
Comment 2 Michael[tm] Smith 2010-12-28 12:12:05 CET
https://bitbucket.org/validator/syntax-patches/qseries?apply=t&qs_apply=script-checker
https://bitbucket.org/validator/syntax-patches/src/tip/script-checker

I implemented the core checks for "/*" without a later "/*" and "//", for the script/@src case, and "<!--" without a later "-->" for both the src-less script case and script/@src case

Up now at http://www.w3.org/html/check for testing
Comment 3 Michael[tm] Smith 2010-12-28 12:23:00 CET
For the record here, IRC discussion from 2010-01 about other cases that the ABNF in the spec is intended for:
http://krijnhoetmer.nl/irc-logs/whatwg/20100113#l-614 (and further down)