Author: gburt Date: Mon Feb 18 00:41:32 2008 New Revision: 3262 URL: http://svn.gnome.org/viewvc/banshee?rev=3262&view=rev
Log: 2008-02-17 Gabriel Burt <[EMAIL PROTECTED]> * src/Libraries/Hyena.Gui/Hyena.Data.Gui/ListView/ListView_Interaction.cs: Avoid hitting the model/database when clicking where there is no row. Modified: trunk/banshee/ChangeLog trunk/banshee/src/Libraries/Hyena.Gui/Hyena.Data.Gui/ListView/ListView_Interaction.cs Modified: trunk/banshee/src/Libraries/Hyena.Gui/Hyena.Data.Gui/ListView/ListView_Interaction.cs ============================================================================== --- trunk/banshee/src/Libraries/Hyena.Gui/Hyena.Data.Gui/ListView/ListView_Interaction.cs (original) +++ trunk/banshee/src/Libraries/Hyena.Gui/Hyena.Data.Gui/ListView/ListView_Interaction.cs Mon Feb 18 00:41:32 2008 @@ -233,12 +233,16 @@ GrabFocus (); int row_index = GetRowAtY ((int)press.Y); + + if (row_index >= Model.Count) { + return true; + } if (press.Button == 1 && press.Type != Gdk.EventType.TwoButtonPress && (press.State & Gdk.ModifierType.ControlMask) == 0 && Selection.Contains (row_index)) { return true; } - + object item = model[row_index]; if (item == null) { return true; @@ -320,6 +324,11 @@ GrabFocus (); int row_index = GetRowAtY ((int)evnt.Y); + + if (row_index >= Model.Count) { + return true; + } + object item = model[row_index]; if (item == null) { return true; @@ -412,7 +421,7 @@ } } - private int GetRowAtY (int y) + protected int GetRowAtY (int y) { int page_offset = (int)vadjustment.Value % RowHeight; int first_row = (int)vadjustment.Value / RowHeight; @@ -421,7 +430,7 @@ return first_row + row_offset; } - private double GetYAtRow (int row) + protected double GetYAtRow (int row) { double y = (double) RowHeight * row; return y; _______________________________________________ SVN-commits-list mailing list (read only) http://mail.gnome.org/mailman/listinfo/svn-commits-list Want to limit the commits to a few modules? Go to above URL, log in to edit your options and select the modules ('topics') you want. Module maintainer? It is possible to set the reply-to to your development mailing list. Email [EMAIL PROTECTED] if interested.