[Webkit-unassigned] [Bug 58960] [Chromium]Mac UI polish to add left/right padding space for autofill popup window.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Apr 20 09:57:13 PDT 2011


https://bugs.webkit.org/show_bug.cgi?id=58960





--- Comment #8 from Ilya Sherman <isherman at chromium.org>  2011-04-20 09:57:13 PST ---
(From update of attachment 90299)
View in context: https://bugs.webkit.org/attachment.cgi?id=90299&action=review

> Source/WebCore/ChangeLog:10
> +        This assumes only AutofillPopupMenuClient gives TexfieldPart appearance.

nit: "Texfield" -> "TextField"

> Source/WebCore/rendering/RenderThemeChromiumMac.mm:83
> +

nit: "autofillPopupHorizontalPadding" might be a better name for this variable -- your call

> Source/WebCore/rendering/RenderThemeChromiumMac.mm:85
> +// We assume only AutofillPopupMenuClient gives TexfieldPart appearance here.

nit: "Texfield" -> "TextField"

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list