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

User [email protected] added comment
https://bugzilla.novell.com/show_bug.cgi?id=482133#c1





--- Comment #1 from tom hindle <[email protected]>  2009-03-04 16:47:45 MST ---
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.

-- 
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.
_______________________________________________
mono-bugs maillist  -  [email protected]
http://lists.ximian.com/mailman/listinfo/mono-bugs

Reply via email to