Opened 10 years ago
Closed 4 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)
Change History (9)
comment:1 Changed 10 years ago by
| Milestone: | tbd → 1.5 |
|---|---|
| Owner: | changed from Bryan Forbes to Nathan Toone |
| severity: | normal → major |
comment:2 Changed 10 years ago by
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
comment:3 Changed 10 years ago by
| Summary: | Column DnD to same spot = sort? → [patch][cla]Column DnD to same spot = sort? |
|---|
comment:5 Changed 10 years ago by
| Milestone: | 1.5 → future |
|---|---|
| Owner: | Nathan Toone deleted |
Moving my tickets to future, as I am not currently working on them.
comment:6 Changed 7 years ago by
| 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 7 years ago by
| Status: | pending → new |
|---|
This is STILL broken. I'll have a look at the patch.
comment:8 Changed 4 years ago by
| 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.
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.