Opened 7 years ago

Closed 6 years ago

Last modified 6 years ago

#15941 closed defect (fixed)

dojox.html.ext-dojo.style breaks if browser supports native transform CSS declaration

Reported by: Christian Schröder Owned by: nic
Priority: undecided Milestone: 1.9
Component: Dojox Version: 1.7.2
Keywords: Cc:
Blocked By: Blocking:

Description

The issue is correctly handled for the transformOrigin attribute, but setting the transform attribute with dojo style will end up in an unbounded recursion.

this diff for dojox/html/ext-dojo/style.js fixes this behavior

We are still using 1.72, that's why i set the version to this number but this bug is also present in trunk.

@@ -61,10 +61,10 @@
 			}
 			if(this.tPropertyName){
 				this.setTransform = function(/*DomNode*/node, /*String*/ transform){
-					return DOMStyle.set(node, this.tPropertyName, transform);
+					return sset(node, this.tPropertyName, transform);
 				};
 				this.getTransform = function(/*DomNode*/node){
-					return DOMStyle.get(node, this.tPropertyName);
+					return sget(node, this.tPropertyName);
 				};
 			}else if(has("ie")){
 				this.setTransform = this._setTransformFilter;

Change History (3)

comment:1 Changed 6 years ago by bill

Component: GeneralDojox
Owner: set to nic
Status: newassigned

comment:2 Changed 6 years ago by nic

Resolution: fixed
Status: assignedclosed

In [30691]:

fixes #15941 - thanks for the diff !strict

comment:3 Changed 6 years ago by bill

Milestone: tbd1.9
Note: See TracTickets for help on using tickets.