[Mono-winforms-list] ButtonBase.AutoSize Implementation
Miguel de Icaza
miguel at ximian.com
Sun Dec 17 14:12:39 EST 2006
Hello,
> thanks for the hint I've corrected the patch. It resizes now if the image,
> font or text has been changed. This should cover all cases, or am I missing
> something?
My code review comments:
Only a few comments, you code should follow the Mono naming conventions
for fields, so the field 'autoSize' should become "auto_size".
You do not need to use "this.Autosize" or "this.AutoCalculateNewSize",
you can remove the "this" from that bit (it happens in a bunch of other
places as well).
There are two magic constants in there, that am confused about: 35 and
15. What are those for, and why are they needed? Shouldnt those be
computed based on some font metric?
Miguel.
More information about the Mono-winforms-list
mailing list