https://bugzilla.wikimedia.org/show_bug.cgi?id=69924

--- Comment #9 from Michael M. <listenle...@gmail.com> ---
(In reply to Erik Moeller from comment #7)
> Michael, if common.js is non-blank, this will lead to site-wide errors in
> IE6 (and other disabled browsers).

My comment was limited to user scripts, and did not include site-wide scripts.
For MediaWiki:Common.js etc. I agree it is the right thing to wrap it into that
if-clause. But I don't think it should be done for User:Foo/common.js etc.
While most site-wide scripts should be fixed by now, most user scripts aren't,
because it is much more difficult to fix lots of user scripts that to fix a few
site-wide scripts.

If a user script breaks because it refers to mw without checking for its
existence though the user uses a blacklisted browser, it's solely the user's
responsibility to fix his own script. But a user script should never break,
because the software changes it in a way that changes valid syntax into invalid
(as it is in this case, a FunctionDecleration is allowed in global scope, but
not in block scope, and the if-clause changes the scope of the script from
global to block).

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
_______________________________________________
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l

Reply via email to