Opened 11 years ago

Closed 11 years ago

#6132 closed defect (fixed)

Replace dojo.marginBox() in the "set" mode with low-level constructs

Reported by: Eugene Lazutkin Owned by: Eugene Lazutkin
Priority: high Milestone: 1.1
Component: DnD Version: 1.0
Keywords: Cc:
Blocked By: Blocking:

Description

Two reasons:

  1. The performance problem was discovered in dojo.marginBox(), which is not warranted in such a simple case.
  2. We need low-level constructs in order to support a relatively positioned nodes.

Change History (1)

comment:1 Changed 11 years ago by Eugene Lazutkin

Resolution: → fixed
Status: new → closed

(In [13002]) dnd: removing dojo.marginBox() calls from all places in the DnD package, adding new TimedMoveable? class, added a test for the new class. Thx Douglas Hays for the idea of fps throttling on move, and for the initial patch. Fixes #6132. Fixes #6133.

Note: See TracTickets for help on using tickets.