Opened 9 years ago

Closed 9 years ago

Last modified 8 years ago

#12065 closed task (fixed)

EnhancedGrid DnD patch for 1.6 beta

Reported by: evan Owned by: Douglas Hays
Priority: high Milestone: 1.6
Component: DojoX Grid Version: 1.5
Keywords: EDG dnd Cc: hwcdl@…
Blocked By: Blocking:

Description

Delta changes based on #11723, mainly includes following changes:

  1. Improved performance by changing avatar style instead of keeping appending/removing it from the document.
  1. Support dragging content(row/cell) from other widgets into grid.
  1. Bug fix - keyboard events should be connected to doc rather inside grid as row/cell can be dragged out of grid.
  1. Bug fix - the avatar mark was not correct in RTL.
  1. Bug fix - occasional conflict with selection
  1. DnD styles moved into a separate "DnD.css"
  1. Code clean, removed unused logic
  1. Test case/data updates

Attachments (1)

dnd-12-8-2010.patch (80.0 KB) - added by evan 9 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 Changed 9 years ago by Douglas Hays

Owner: changed from bryanforbes to Douglas Hays
Status: newassigned

Changed 9 years ago by evan

Attachment: dnd-12-8-2010.patch added

comment:2 Changed 9 years ago by Douglas Hays

Resolution: fixed
Status: assignedclosed

(In [23322]) Fixes #12065, refs #11723. Proxy commit for evan (IBM, CCLA). Various EnhancedGrid? DnD bug fixes and new tests. !strict

comment:3 Changed 8 years ago by Evan

(In [23617]) Refs #12065 Remove the obsolete empty dnd folder

comment:4 Changed 8 years ago by evan

(In [23641]) Refs #12065 1. remove redundant code 2. don't rearrange store content for id column

comment:5 Changed 8 years ago by evan

(In [23677]) Refs #12065, fix a variable typo which caused failure of dnd content among Grid instances

comment:6 Changed 8 years ago by evan

(In [23790]) Fix from Xiao Wen Zhu(IBM, CCLA), fix a compatibility issue of DnD and disabled rows, refs #12065, !strict

comment:7 Changed 8 years ago by evan

(In [23818]) Trivial func naming & test case fixes, refs #12065

comment:8 Changed 8 years ago by evan

(In [24279]) Refs #12065 removing 1 obsolete line

Note: See TracTickets for help on using tickets.