[Mono-winforms-list] [PATCH] Theme.DefaultFont

Chris Toshok toshok at ximian.com
Fri Dec 29 13:07:19 EST 2006

I don't really know if there's any historical significance to using 8
instead of 8.25.  I can't imagine there is.  anyone else know?

The patch looks good to me, aside from one minor change - there's no
need to keep the readonly modifier on default_font if it's private.

Please, commit.  Thanks much.


On Fri, 2006-12-29 at 14:11 +0100, Gert Driesen wrote:
> Hi,
> The attached patch makes a small change to Theme (and derived classes)
> to make it more compatible with MS (see included unit tests).
> On both the 1.0 and 2.0 profile, Theme.DefaultFont is now MS Sans Sarif
> 8.25 (instead of 8). This matches Control.Font and
> SystemFonts.DefaultFont on MS.
> On the 2.0 profile, Theme.DefaultFont is initialized with
> SystemFonts.DefaultFont.
> I modified the accessibility of the default_font field to private, since
> Theme.DefaultFont is virtual anyway.
> Let me know if it's ok to commit.
> Gert
> _______________________________________________
> Mono-winforms-list maillist  -  Mono-winforms-list at lists.ximian.com
> http://lists.ximian.com/mailman/listinfo/mono-winforms-list

More information about the Mono-winforms-list mailing list