[Mono-bugs] [Bug 508523] ColorComboBox in FontDialog should show all 16 colors (no vertical scrollbar)

[bugzilla_noreply at novell.com]

http://bugzilla.novell.com/show_bug.cgi?id=508523

User stifu at free.fr added comment
http://bugzilla.novell.com/show_bug.cgi?id=508523#c2


Thomas Goldstein <stifu at free.fr> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #295358|0                           |1
        is obsolete|                            |




--- Comment #2 from Thomas Goldstein <stifu at free.fr>  2009-05-30 11:55:44 MDT ---
Created an attachment (id=295363)
 --> (http://bugzilla.novell.com/attachment.cgi?id=295363)
Patch (faster)

I just noticed there was also a more simple solution:

MaxDropDownItems = Items.Count;

This property is .NET 1.1 compatible, but as I said above, this doesn't match
the behavior of MS .NET 1.1, which shows a scrollbar (precisely: 10 items
shown, 6 hidden), so let's keep the #if NET_2_0 condition anyway.

Note this new solution doesn't trigger the useless vertical scrollbar bug
(which I just reported: https://bugzilla.novell.com/show_bug.cgi?id=508541)

And this new patch is naturally faster, too.

-- 
Configure bugmail: http://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.