rsjudge17 / backplanejs

Automatically exported from code.google.com/p/backplanejs
Other
0 stars 0 forks source link

Calls to UX.addClassName when on FF in XHTML mode, cause a script error #101

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Run a form with a select1.
2. Move the mouse over the select1.

What is the expected output? What do you see instead?
If Firebug is open there should be no errors. Instead we get an error each time 
the mouse is moved:

  String contains an invalid character" code: "5
    element.classList.add(classString);

Original issue reported on code.google.com by mark.bir...@gtempaccount.com on 10 Sep 2010 at 1:32

GoogleCodeExporter commented 9 years ago
Anton has established that the error is caused by the space character in the 
call to addClassName, in the StyleHoverishly() method (in eventTargetProxy.js).

Original comment by mark.bir...@gtempaccount.com on 10 Sep 2010 at 1:33

GoogleCodeExporter commented 9 years ago
Turns out that the space character is not the problem, since the 
UX.addClassName() method is designed to take spaces into account. Instead the 
problem is that the code is using a non-existent error code. Instead of this 
(in UXUtils.js):

  if (e.code != e.NS_ERROR_DOM_INVALID_CHARACTER_ERR) throw e;

we should have this:

  if (e.code != e.INVALID_CHARACTER_ERR) throw e;

Thanks to Anton for the pointer, though.

Original comment by mark.bir...@gtempaccount.com on 10 Sep 2010 at 3:19

GoogleCodeExporter commented 9 years ago
Added regression test in ra343e3b9e21c5723a5b992b234529a043855654e.

Original comment by mark.bir...@gtempaccount.com on 10 Sep 2010 at 3:42

GoogleCodeExporter commented 9 years ago
This issue was closed by revision 8cc27d4002.

Original comment by mark.bir...@gtempaccount.com on 10 Sep 2010 at 5:14