Modify ↓
Ticket #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 | |
| Blocking: | Blocked by: |
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
-
curCSS.patch
(555 bytes) -
added by jeff_themovie 5 years ago.
-
testcase.html
(1.1 KB) -
added by jeff_themovie 4 years ago.
- test case - notice what happens when the red box is hidden in IE
Change History
comment:1 Changed 5 years ago by flesler
- Cc jeff_themovie added
- need changed from Review to Patch
- Status changed from new to assigned
- Component changed from core to event
- Owner set to flesler
comment:2 Changed 5 years ago by jeff_themovi
Patch attached. (The regex carets in the ticket description went missing for some reason...)
comment:3 Changed 5 years ago by jeff_themovi
Is there anything I should do to get the resolution process going?
Changed 4 years ago by jeff_themovie
-
attachment
testcase.html
added
test case - notice what happens when the red box is hidden in IE
comment:4 Changed 4 years ago by john
- Version changed from 1.2.6 to 1.3.2
- Component changed from event to core
- Milestone changed from 1.3 to 1.4
Please follow the bug reporting guidlines and use jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.
Note: See
TracTickets for help on using
tickets.