Side navigation
#13258 closed bug (notabug)
Opened January 18, 2013 01:30AM UTC
Closed January 18, 2013 02:05AM UTC
.removeClass() doesn't work without parameters
Reported by: | MichaelM | Owned by: | MichaelM |
---|---|---|---|
Priority: | undecided | Milestone: | None |
Component: | unfiled | Version: | 1.9.0 |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
removeClass without parameters still passes an argument length of 1, so
proceed = arguments.length === 0 || typeof value === "string" && value;
should be
proceed = arguments.length === 1 || typeof value === "string" && value;
Attachments (0)
Change History (3)
Changed January 18, 2013 01:33AM UTC by comment:1
owner: | → MichaelM |
---|---|
status: | new → pending |
Changed January 18, 2013 02:00AM UTC by comment:2
_comment0: | I realized this issue doesn't occur unless jQuery UI 1.9.2 is also being used, my apologies. \ \ I'm going to go open a ticket on their site, but here's the jsFiddle for anyone interested: http://jsfiddle.net/nbyCC/2/ → 1358474490998787 |
---|---|
_comment1: | I realized this issue doesn't occur unless jQuery UI 1.9.2 is also being used, my apologies. \ \ I'm going to go open a ticket on their site, but here's the jsFiddle for anyone interested: http://jsfiddle.net/nbyCC/1/ → 1358474549160684 |
status: | pending → new |
I realized this issue doesn't occur unless jQuery UI 1.9.2 is also being used, my apologies.
I'm going to go open a ticket on their site, but here's the jsFiddle for anyone interested: http://jsfiddle.net/nbyCC/3/
Changed January 18, 2013 02:05AM UTC by comment:3
resolution: | → notabug |
---|---|
status: | new → closed |
We'll need proof of that, please provide a test case on jsfiddle.net.