Skip to main content

Bug Tracker

Side navigation

#969 closed bug (wontfix)

Opened February 16, 2007 06:20AM UTC

Closed March 31, 2008 02:12AM UTC

Last modified March 14, 2012 08:05PM UTC

className animation has incorrect RegExp declaration

Reported by: jeffkretz Owned by: stefan
Priority: minor Milestone: 1.1.3
Component: interface Version: 1.1
Keywords: className Cc:
Blocked by: Blocking:
Description

In animating a className, the ifx.js defines a Reg Exp to find the style rule on line 395:

var rule = new RegExp('.' + prop[p] + ' {');

This works in IE and FF, but Opera throws an exception:

Statement on line 395: Reg Exp.prototype.compile: syntax error in pattern or string.

This appears to be because the Reg Exp declaration for Opera is more strict about backslash declarations to escape specific characters. The following change to the source code worked for me in IE6, IE7, FF and Opera:

var rule = new RegExp('\\.' + prop[p] + ' \\{');

Note that the backslash itself for the period and curly brace is then passed to the Reg Exp statement.

JK

Attachments (0)
Change History (1)

Changed March 31, 2008 02:12AM UTC by scott.gonzal comment:1

description: In animating a className, the ifx.js defines a Reg Exp to find the style rule on line 395:\ \ var rule = new RegExp('.' + prop[p] + ' {');\ \ This works in IE and FF, but Opera throws an exception:\ Statement on line 395: Reg Exp.prototype.compile: syntax error in pattern or string.\ \ This appears to be because the Reg Exp declaration for Opera is more strict about backslash declarations to escape specific characters. The following change to the source code worked for me in IE6, IE7, FF and Opera:\ \ var rule = new RegExp('\\.' + prop[p] + ' \\{');\ \ Note that the backslash itself for the period and curly brace is then passed to the Reg Exp statement.\ \ JK\ In animating a className, the ifx.js defines a Reg Exp to find the style rule on line 395: \ \ var rule = new RegExp('.' + prop[p] + ' {'); \ \ This works in IE and FF, but Opera throws an exception: \ Statement on line 395: Reg Exp.prototype.compile: syntax error in pattern or string. \ \ This appears to be because the Reg Exp declaration for Opera is more strict about backslash declarations to escape specific characters. The following change to the source code worked for me in IE6, IE7, FF and Opera: \ \ var rule = new RegExp('\\.' + prop[p] + ' \\{'); \ \ Note that the backslash itself for the period and curly brace is then passed to the Reg Exp statement. \ \ JK \
need: → Review
resolution: → wontfix
status: newclosed

Interface is no longer supported; consider switching to jQuery UI.