Dietmar Schwertberger
Dietmar Schwertberger
With [324430b](https://github.com/wxWidgets/wxWidgets/pull/24362/commits/324430b38d2eaea3403698190a790434817250e3) now `m_dragLabel` is being used. `DoGridDragResize` is still shared. I'm not sure whether some of the other methods should be shared as well.
The last two commits do simplify grid again a bit, but complicate `GridOperations`. The same could probably be done for `DoEndDragResizeRow` and `DoEndDragResizeCol`. What do you think? Still open: for...
The last commits somewhat reduce complexity. At many points there is code like ``` oper = (m_cursorMode == WXGRID_CURSOR_RESIZE_ROW ? (wxGridOperations*) new wxGridRowOperations() : wxGridOperations*) new wxGridColumnOperations()); ``` I think...
OK, a helper like this could be used at four points: ```C++ wxGridOperations* wxGrid::DoGetOperationsFromCursorMode() { if ( m_cursorMode == WXGRID_CURSOR_RESIZE_ROW ) return new wxGridRowOperations(); if ( m_cursorMode == WXGRID_CURSOR_RESIZE_COL )...
Thanks. I think I have now all infos and will modify accordingly. Yes, the events in question are the mouse events only. I will compare the handling in the different...
About event handling and `event.Skip()`: `ProcessGridCellMouseEvent` is calling `event.Skip()` only in some cases at the end, but e.g. not on `Entering` or `Leaving`. E.g. for `event.Moving()` it's calling `DoGridMouseMoveEvent` and...
A single commit would be fine for me.