Opened 7 years ago
Closed 6 years ago
#17556 closed defect (fixed)
Using "dojox/html/ext-dojo/style" breaks "dojox/layout/ResizeHandle"
Reported by: | chris2m | Owned by: | nic |
---|---|---|---|
Priority: | undecided | Milestone: | 1.7.6 |
Component: | Dojox | Version: | 1.9.1 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description
The problem is caused when "dojox/html/ext-dojo/style"
and "dojox/layout/ResizeHandle"
are used on the same page.
Cause
"dojox/html/ext-dojo/style"
monkey patches the get and set methods of "dojox/html/ext-dojo/style"
. ResizeHandle? calls domStyle.get
with three parameters e.g. domStyle.get(this.targetDomNode, 'width', style))
the patched method however expects only one or two arguments. As a result the ResizeHandle? breaks.
FIX
To fixed the bug, the following lines in "dojox/html/ext-dojo/style"
should be replaced:
// delete return arguments.length == 2 ? sget(node, name) : sget(node); // replace with return sget.apply(this, arguments);
and the same for the setter method:
// delete return arguments.length == 3 ? sset(n, name, value) : sset(n, name); // replace with return sset.apply(this, arguments);
Change History (6)
comment:1 Changed 6 years ago by
Owner: | set to nic |
---|---|
Status: | new → assigned |
comment:2 Changed 6 years ago by
comment:6 Changed 6 years ago by
Milestone: | tbd → 1.7.6 |
---|---|
Resolution: | → fixed |
Status: | assigned → closed |
Fixed and backported through 1.7.x
Note: See
TracTickets for help on using
tickets.
In 701d1a927d4055ac086ab5f614594b7a5f302702/dojox: