Opened 9 years ago

Closed 3 years ago

#10564 closed defect (patchwelcome)

Grid breaks DND in dojox.layout.GridContainer when columns are sortable

Reported by: Shane O'Sullivan Owned by:
Priority: high Milestone: 1.13
Component: DojoX Layout Version: 1.4.0
Keywords: Cc:
Blocked By: Blocking:

Description

If a Grid is contained in dojox.layout.GridContainer?, with column sorting turned on, and it is contained in a layout widget, DND of title panes or Portlets is broken in IE. The avatar is not properly created, and the dragged item cannot be dropped.

If column sorting is turned off, it works. If the GridContainer? is not contained within a layout widget, e.g. a TabContainer? or a BorderLayout?, it works

The problem should be somewhere between line 226 and 356 of dojox/Grid/_View.js

See the attached text case.

Attachments (1)

test_PortletInGridContainer2.html​ (4.7 KB) - added by Shane O'Sullivan 9 years ago.
Test case

Download all attachments as: .zip

Change History (7)

Changed 9 years ago by Shane O'Sullivan

Attachment: test_PortletInGridContainer2.html​ added

Test case

comment:1 Changed 9 years ago by Shane O'Sullivan

(In [21090]) Refs #10564 Temporary fix for the Portlet test page to get it working again in IE. The Grid code should be fixed so that this is no longer necessary.

comment:2 Changed 9 years ago by Adam Peller

need to justify a patch for 1.4.1

comment:3 Changed 9 years ago by Adam Peller

Milestone: 1.4.2 → tbd

comment:4 Changed 9 years ago by Nathan Toone

Owner: Nathan Toone deleted

Unassigning my tickets.

comment:5 Changed 9 years ago by bill

Component: DojoX Grid → DojoX Layout

comment:6 Changed 3 years ago by dylan

Milestone: tbd → 1.12
Resolution: → patchwelcome
Status: new → closed

Given that no one has shown interest in creating a patch in the past 5+ years, I'm closing this as patchwelcome.

Note: See TracTickets for help on using tickets.