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

User [EMAIL PROTECTED] added comment
https://bugzilla.novell.com/show_bug.cgi?id=375788#c2


Andy Hume <[EMAIL PROTECTED]> changed:

           What    |Removed                                         |Added
----------------------------------------------------------------------------
             Status|RESOLVED                                        |REOPENED
         Resolution|FIXED                                           |




--- Comment #2 from Andy Hume <[EMAIL PROTECTED]>  2008-04-08 04:29:56 MST ---
Looks good, but it would be more user-friendly and consistent with the
behaviour in other apps (e.g. closing tabs in Firefox, etc etc), if the item at
the same index as the one removed was selected afterwards.

e.g. in CollectionEditor.doRemove_Click instead do:
    // select the item at the same index; handling off-end and list empty cases
    itemsList.SelectedIndex = Math.Min(selected[0], itemsList.Items.Count -
1)); 


((And, the last conditional in that method seems odd, is it a NOP? (Count==0 =>
SI==-1 => SO==null))).


-- 
Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
_______________________________________________
mono-bugs maillist  -  [email protected]
http://lists.ximian.com/mailman/listinfo/mono-bugs

Reply via email to