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

--- Comment #2 from Christian Thiele <[email protected]> 2012-08-21 11:06:51 UTC 
---
As written on this page: "Because :first is a jQuery extension and not part of
the CSS specification, queries using :first cannot take advantage of the
performance boost provided by the native DOM querySelectorAll() method.". But
the "h5:first a:first" is passed to querySelectorAll().

I now had a closer look at this and firefox doesn't throw an error because of a
catch statement, but at least it seems weird to me to call a function which
unavoidable throws an error and then catch it...

The call tree is:
  line 7 of skins.vector (see original bug report)
  line 27 of mediawiki jquery (jquery constructor)
  line 192 of mediawiki jquery (init function)
  line 5431 of mediawiki jquery (find function)
  line 5163 of mediawiki jquery

on line 5163 context.querySelectorAll( "[id='" + nid + "'] " + query ) is
called and query is "h5:first a:first". This throws an SYNTAX_ERR which is
cached at line 5166.

I'm okay with closing this bug because I didn't see the catch statement. But if
there is an easy fix not calling querySelectorAll() with these parameters it
would be cool, too ;).

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
------- 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
[email protected]
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l

Reply via email to