Opened 12 years ago

Closed 5 years ago

#9749 closed defect (patchwelcome)

[patch][cla]Column DnD to same spot = sort?

Reported by: haysmark Owned by: haysmark
Priority: high Milestone: 1.13
Component: DojoX Grid Version: 1.3.2
Keywords: Cc:
Blocked By: Blocking:

Description

Observe the robot's attempt to resize the message column:

​http://archive.dojotoolkit.org/nightly/checkout/dojox/grid/tests/robot/DataGrid_mouse.html

The message column in fact cannot be resized, so instead the grid attempts a drag and drop. The robot moves the column to essentially the same spot and lets go. Rather than canceling the drag, the grid sets the sort order of the grid?

Attachments (1)

9749.patch​ (1.6 KB) - added by Nathan Toone 11 years ago.
Patch which fixes this issue

Download all attachments as: .zip

Change History (9)

comment:1 Changed 11 years ago by Nathan Toone

Milestone: tbd → 1.5
Owner: changed from Bryan Forbes to Nathan Toone
severity: normal → major

I have seen this too - and it bothers me. Probably will be a 1.5 timeframe sort of thing, but I'll look at it.

comment:2 Changed 11 years ago by Nathan Toone

I have a patch that addresses this issue. Since it's not a regression (this has been around since as long as I can remember), I'll just post the patch, and we can commit post-1.4

Changed 11 years ago by Nathan Toone

Attachment: 9749.patch​ added

Patch which fixes this issue

comment:3 Changed 11 years ago by Nathan Toone

Summary: Column DnD to same spot = sort? → [patch][cla]Column DnD to same spot = sort?

comment:4 Changed 11 years ago by Douglas Hays

Possibly related to #10312

comment:5 Changed 11 years ago by Nathan Toone

Milestone: 1.5 → future
Owner: Nathan Toone deleted

Moving my tickets to future, as I am not currently working on them.

comment:6 Changed 8 years ago by dylan

Owner: set to haysmark
Status: new → pending

This would need to be updated to AMD, and verified to see if it still works.

Any interest in this, or with the move of most people to dgrid or gridx, does it matter?

comment:7 Changed 8 years ago by haysmark

Status: pending → new

This is STILL broken. I'll have a look at the patch.

comment:8 Changed 5 years ago by dylan

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

Will consider this if someone wants to update the patch to a pull request against the current codebase.

Note: See TracTickets for help on using tickets.