Description Peer Törngren 2010-09-07 06:15:43 EDT

Build Identifier: 20100617-1415

Due to what appears to be a bug (or at least counter-intuitive behavior) in
the Grid, the column must be moveable in order to be sortable. This appears to be because the code in 'Grid.handleColumnHeaderPush(int, int)' explicitly checks if the column is movable, and bails out to disable "column header push" behavior if it isn't. Hence, no "selection events" are sent unless the column is movable. 

This makes sense in order to disable "push" for moving, but it also disables sorting. The same (false) assumption is made elsewhere; a non-movable grid column is rendered flat to indicate that it cannot be pushed - but you may still want to push it in order to sort it.

This is the actual code in 'Grid.handleColumnHeaderPush(int, int)':

if (cellSelectionEnabled && overThis.getMoveable() == false) {
    return false;
}


Reproducible: Always

Steps to Reproduce:
1. Create a default grid
2. Make the columns non-movable (setMoveable(false))
3. Add a SelectionListener to the column, have it print all events on System out
4. Run the grid and click the column - no events are printed
5. column.setMoveable(true)
6. Run the grid and click the column - events are printed