Skip to main content

Bug Tracker

Side navigation

#3331 closed bug (fixed)

Opened September 05, 2008 08:54AM UTC

Closed December 05, 2009 08:12PM UTC

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)
  • curCSS.patch (0.5 KB) - added by jeff_themovie September 08, 2008 04:10AM UTC.
  • testcase.html (1.1 KB) - added by jeff_themovie May 14, 2009 02:17AM UTC.

    test case - notice what happens when the red box is hidden in IE

Change History (5)

Changed September 05, 2008 03:45PM UTC by flesler comment:1

cc: → jeff_themovie
component: coreevent
need: ReviewPatch
owner: → flesler
status: newassigned

Changed September 08, 2008 04:14AM UTC by jeff_themovi comment:2

Patch attached. (The regex carets in the ticket description went missing for some reason...)

Changed October 27, 2008 02:38AM UTC by jeff_themovi comment:3

Is there anything I should do to get the resolution process going?

Changed November 30, 2009 06:27PM UTC by john comment:4

component: eventcore
milestone: 1.31.4
version: 1.2.61.3.2

Changed December 05, 2009 08:12PM UTC by john comment:5

resolution: → fixed
status: assignedclosed