[Mono-bugs] [Bug 482133] Setting RowCount on DataGridView to reduce number of rows, fails to reduce rows correctly

[bugzilla_noreply at novell.com]

https://bugzilla.novell.com/show_bug.cgi?id=482133

User contact at i-nz.net added comment
https://bugzilla.novell.com/show_bug.cgi?id=482133#c4





--- Comment #4 from Ivan Zlatev <contact at i-nz.net>  2009-03-04 16:52:14 MST ---
(In reply to comment #1)
> Something else which may be relevant to this change. On further testing I found
> that now I could delete rows from visible grids, If I did so on a grid with
> more rows than screen size (causing DataGridView::first_row_index to be set)
> Delete rows throw an exception.
> 
> I think DataGridView member var first_row_index possibly needs to be updated
> when deleting Rows.
> 
> I added the following to DataGridView::OnRowsRemovedInternal()
> 
> if (first_row_index >= Rows.Count)
>     first_row_index = Rows.Count -1;
> 
> just above validate.

Please create me a test case and file a separate bug so I can look into it,
thanks!

-- 
Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.