Ticket #6132 (closed defect: fixed)

Opened 9 months ago

Last modified 9 months ago

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

Reported by: elazutkin Owned by: elazutkin
Priority: normal Milestone: 1.1
Component: DnD Version: 1.0
Severity: normal Keywords:
Cc:

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

Changed 9 months ago by elazutkin

  • status changed from new to closed
  • resolution set to fixed

(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.