Opened 14 years ago
Closed 13 years ago
#3331 closed bug (fixed)
curCSS() doesn't convert negative em/% values properly in IE
Reported by: | jeff_themovie | Owned by: | flesler |
---|---|---|---|
Priority: | major | Milestone: | 1.4 |
Component: | core | Version: | 1.3.2 |
Keywords: | Cc: | jeff_themovie | |
Blocked by: | Blocking: |
Description
This is because line 920:
if ( !/\d+(px)?$/i.test( ret ) && /\d/.test( ret ) ) {
doesn't accept negative values. I suggest:
if ( !/-?\d+(px)?$/i.test( ret ) && /-?\d/.test( ret ) ) {
Attachments (2)
Change History (7)
comment:1 Changed 14 years ago by
Cc: | jeff_themovie added |
---|---|
Component: | core → event |
need: | Review → Patch |
Owner: | set to flesler |
Status: | new → assigned |
Changed 14 years ago by
Attachment: | curCSS.patch added |
---|
comment:2 Changed 14 years ago by
comment:3 Changed 14 years ago by
Is there anything I should do to get the resolution process going?
Changed 14 years ago by
Attachment: | testcase.html added |
---|
test case - notice what happens when the red box is hidden in IE
comment:4 Changed 13 years ago by
Component: | event → core |
---|---|
Milestone: | 1.3 → 1.4 |
Version: | 1.2.6 → 1.3.2 |
comment:5 Changed 13 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Note: See
TracTickets for help on using
tickets.
Patch attached. (The regex carets in the ticket description went missing for some reason...)