Opened 11 years ago

Closed 10 years ago

Last modified 10 years ago

#8614 closed enhancement (fixed)

[patch][cla]The Grid should use better column move and resizing indicators

Reported by: Shane O'Sullivan Owned by: Nathan Toone
Priority: high Milestone: 1.4
Component: DojoX Grid Version: 1.2.3
Keywords: Cc: Bryan Forbes, drs
Blocked By: Blocking:

Description

Currently the dojox Grid resizes the header while the user drags the mouse. This gives the impression that it is out of sync with the body of the grid. It would be better to use a vertical bar while dragging, and resize both the header and body at the same time.

When moving a column, it is difficult to tell where it is being placed. It would be better to have one or two arrows pointing at the place where the column will be placed.

Attachments (3)

Grid_Resize_move_column.patch (10.4 KB) - added by Shane O'Sullivan 11 years ago.
Patch that implements this enhancement
Grid_Resize_move_column2.patch (8.8 KB) - added by Shane O'Sullivan 11 years ago.
Updated patch made against SVN as of 18th Feb 2009
Grid_Resize_move_column3.patch (10.0 KB) - added by Shane O'Sullivan 10 years ago.
Updated patch that takes scroll into account

Download all attachments as: .zip

Change History (10)

Changed 11 years ago by Shane O'Sullivan

Patch that implements this enhancement

comment:1 Changed 11 years ago by Nathan Toone

Milestone: tbdfuture
Owner: changed from Bryan Forbes to Nathan Toone

comment:2 Changed 11 years ago by Shane O'Sullivan

Worth noting that the CSS included in this patch uses the arrow up and arrow down images already included in the grid, just because they are already available.

A theme-specific arrow marker could be used easily, and would probably look much better.

Changed 11 years ago by Shane O'Sullivan

Updated patch made against SVN as of 18th Feb 2009

comment:3 Changed 10 years ago by Shane O'Sullivan

Is there any update on this ticket? Will it make it into 1.3?

comment:4 Changed 10 years ago by Nathan Toone

Cc: Bryan Forbes added

I'm a bit uncomfortable at including a patch like this so close to the end of the release cycle...it looks good - but I'm not sure of all the things it might affect.

Perhaps Bryan has some input...

Changed 10 years ago by Shane O'Sullivan

Updated patch that takes scroll into account

comment:5 Changed 10 years ago by Nathan Toone

Milestone: future1.4
Summary: The Grid should use better column move and resizing indicators[patch][cla]The Grid should use better column move and resizing indicators

comment:6 Changed 10 years ago by Nathan Toone

Resolution: fixed
Status: newclosed

Fixed in [17160]

comment:7 Changed 10 years ago by Adam Peller

Cc: drs added
Note: See TracTickets for help on using tickets.