[webkit-reviews] review requested: [Bug 10801] REGRESSION: Styled form controls lose their intrinsic margins : [Attachment 10482] Rework intrinsic margin code

bugzilla-request-daemon at opendarwin.org bugzilla-request-daemon at opendarwin.org
Sat Sep 9 23:44:22 PDT 2006


Dave Hyatt <hyatt at apple.com> has asked	for review:
Bug 10801: REGRESSION: Styled form controls lose their intrinsic margins
http://bugzilla.opendarwin.org/show_bug.cgi?id=10801

Attachment 10482: Rework intrinsic margin code
http://bugzilla.opendarwin.org/attachment.cgi?id=10482&action=edit

------- Additional Comments from Dave Hyatt <hyatt at apple.com>
This patch reworks intrinsic margins and consolidates all the code in one
place.

Before this code was found in three places: old form controls, the themes for
engine controls, and the layer for ensuring resizers didn't cause jumpiness.

Now the intrinsic margin is adjusted in one place: adjustRenderStyle.  It's
done regardless of how the control is implemented (old, new, styled, unstyled,
etc.).

This means that some controls are no going to acquire intrinsic margins that
didn't before (I believe this is a good thing, since now all controls will be
consistent).  The biggest change will be the image button.



More information about the webkit-reviews mailing list