[chromium] Reduce padding on popup menu entries when used on a touch screen.
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 24 Aug 2012 23:02:52 +0000 (23:02 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 24 Aug 2012 23:02:52 +0000 (23:02 +0000)
https://bugs.webkit.org/show_bug.cgi?id=94953

Patch by Kevin Ellis <kevers@chromium.org> on 2012-08-24
Reviewed by Adam Barth.

Prior to the patch, the height of entries in a popup menu was 44px,
which is excessive.  Reduced the padding to make the menu entries
30px tall to match the wrench menu.

No new tests required.

* platform/chromium/PopupMenuChromium.cpp:
(WebCore):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@126643 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/platform/chromium/PopupMenuChromium.cpp

index aed3c04..d569183 100644 (file)
@@ -1,3 +1,19 @@
+2012-08-24  Kevin Ellis  <kevers@chromium.org>
+
+        [chromium] Reduce padding on popup menu entries when used on a touch screen.
+        https://bugs.webkit.org/show_bug.cgi?id=94953
+
+        Reviewed by Adam Barth.
+
+        Prior to the patch, the height of entries in a popup menu was 44px,
+        which is excessive.  Reduced the padding to make the menu entries
+        30px tall to match the wrench menu. 
+
+        No new tests required.
+
+        * platform/chromium/PopupMenuChromium.cpp:
+        (WebCore):
+
 2012-08-24  Helder Correia  <helder.correia@nokia.com>, Roland Takacs  <rtakacs@inf.u-szeged.hu>
 
         [Texmap] Move TextureMapperGL to use GraphicsContext3D
index e588956..c9bfb32 100644 (file)
@@ -41,7 +41,7 @@
 namespace WebCore {
 
 int PopupMenuChromium::s_minimumRowHeight = 0;
-int PopupMenuChromium::s_optionPaddingForTouch = 30;
+int PopupMenuChromium::s_optionPaddingForTouch = 16;
 
 // The settings used for the drop down menu.
 // This is the delegate used if none is provided.