Bug 794 – investigate ways to improve reporting for script elements with inline scripts

NOTE: The current preferred location for bug reports is the GitHub issue tracker.

Bug 794 - investigate ways to improve reporting for script elements with inline scripts


Summary:	investigate ways to improve reporting for script elements with inline scripts

Status:	RESOLVED FIXED

Product:	Validator.nu
Classification:	Unclassified
Component:	General
Version:	HEAD
Hardware:	All All

Importance:	P2 enhancement
Assigned To:	Nobody

URL:

Depends on:
Blocks:	452 766
	Show dependency tree / graph

Reported:	2010-11-30 00:14 CET by Michael[tm] Smith
Modified:	2010-12-17 12:44 CET (History)
CC List:	1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Michael[tm] Smith 2010-11-30 00:14:12 CET

http://krijnhoetmer.nl/irc-logs/whatwg/20101129#l-793


# [20:49] <paulschreiber> i just found this transcript: http://krijnhoetmer.nl/irc-logs/whatwg/20100321  
# [20:49] <paulschreiber> 	.	http://krijnhoetmer.nl/irc-logs/whatwg/20100321#l-656 [16:48] <MikeSmith> Error: Element script is missing required attribute src.  
# [20:49] <paulschreiber> 	.	http://krijnhoetmer.nl/irc-logs/whatwg/20100321#l-657 [16:48] <MikeSmith> I really need to get that fix committed  
# [20:49] <paulschreiber> 	.	http://krijnhoetmer.nl/irc-logs/whatwg/20100321#l-658 [16:49] <Philip`> That's still a far less helpful error than saying it's required because of the charset  
# [20:49] <paulschreiber> 	.	http://krijnhoetmer.nl/irc-logs/whatwg/20100321#l-660 [16:49] <Philip`> People will think they have to add a src, they won't think they have to remove the charset  
# [20:49] <paulschreiber> that was the same error i hit with the validator  
# [20:50] <paulschreiber> what's the best way to get the error message fixed to help folks like myself?  

see also bug 766 and bug 452

Comment 1 Michael[tm] Smith 2010-12-01 16:14:53 CET

Paul,


I've checked in a fix for this bug that causes the validator to emit the
following message instead:

"Error: Element script must not have attribute charset unless attribute src is
also specified."

You can test it yourself here:

http://www.w3.org/html/check

...and it will get deployed at http://validator.nu/ and
http://html5.validator.nu/ soonish

Comment 2 Paul Schreiber 2010-12-01 17:28:42 CET

That looks good to me. Will this get deployed to validator.w3.org as well?

Comment 3 Michael[tm] Smith 2010-12-02 03:49:33 CET

(In reply to comment #2)
> That looks good to me. Will this get deployed to validator.w3.org as well?

Yeah, I will get validator.w3.org synced up with this as well within the next week or so.