wxGrid hidden colums can be made visible by using the drag resize feature

[hackish]

Not really sure why this happens but if you create a grid with 3 rows, 3 cols, call HideCol(1) you can still resize the hidden column and view it. I suspect the code itself should ignore a row or column with a negative size (as it's hidden). I probably won't have the time to build a patch for this but here is the report for anyone else with time.

[vadz]

What platform is this? I am almost sure I tested this after the recent changes in HideCol() code and it worked (i.e. didn't resize anything) correctly.

As always, having a minimal patch reproducing the problem in the grid sample (if it can't be already reproduced in it out of the box?) would be very welcome.

[hackish]

Platform is MSW. I'll see what I can do to build a test case. Maybe even do a patch for it :)

[hackish]

Here are the steps.

Apply the above patches to the latest SVN sample code for grid.
Select the Hide Col A menu option.
Drag column to the left of column B to enlarge it. Column A will reappear.

[hackish]

Modifications to the sample to show the fault

[hackish]

See the attached patch. wxGrid::PosToEdgeOfLine was not properly checking the visibility of the cell before it and blindly assuming if the found cell didn't match then the one before it must. It now continues to traverse every hidden cell before a non-matching entry then re-checks before returning -1. I've tested this as many ways as I could to ensure it doesn't introduce any other unwanted behaviour.

[vadz]

Thanks, the patch looks correct but I think it could be simplified a little by merging the identical checks before and inside the loop. I'll do it, but please retest and let me know if you find any problem with my changes. Thanks again!

[hackish]

There was a reason why the checks shouldn't be combined but I'd have to look at it again. Let me know when you've committed your version and I'll figure it out again.

[VZ]

(In [73399]) Fix bug with showing hidden wxGrid lines when resizing an adjacent one.

Don't blindly assume that the previous line is big enough if the current one
isn't in wxGrid::PosToEdgeOfLine() (this assumption is wrong even without
taking hiding into account...).

Closes #14947.

[VZ]

(In [73460]) Really fix resizing of wxGird column/rows after a hidden one.

The previous fix (r73399) broke resizing from the side of the preceding grid
line, restore the old code and just add a loop finding the first visible line
before the one being clicked.

See #14947.