[haiku-webkit-commits] r367 - webkit/trunk/WebCore/platform/haiku

  • From: webkit@xxxxxxxxxxxxxxx
  • To: haiku-webkit-commits@xxxxxxxxxxxxx
  • Date: Fri, 02 Apr 2010 18:02:42 +0000

Author: stippi
Date: Fri Apr  2 18:02:42 2010
New Revision: 367
URL: http://mmlr.dyndns.org/changeset/367

Log:
Always reset the border style of menu list elements. It just doesn't look good
to have a custom border around a control that otherwise renders like a
BMenuField. (This is also what the Qt port does, BTW.)

Modified:
   webkit/trunk/WebCore/platform/haiku/RenderThemeHaiku.cpp

Modified: webkit/trunk/WebCore/platform/haiku/RenderThemeHaiku.cpp
==============================================================================
--- webkit/trunk/WebCore/platform/haiku/RenderThemeHaiku.cpp    Fri Apr  2 
17:11:48 2010        (r366)
+++ webkit/trunk/WebCore/platform/haiku/RenderThemeHaiku.cpp    Fri Apr  2 
18:02:42 2010        (r367)
@@ -256,7 +256,6 @@
 
 void RenderThemeHaiku::adjustMenuListStyle(CSSStyleSelector* selector, 
RenderStyle* style, Element* element) const
 {
-    style->resetBorder();
     adjustMenuListButtonStyle(selector, style, element);
 }
 
@@ -267,6 +266,9 @@
 
 void RenderThemeHaiku::adjustMenuListButtonStyle(CSSStyleSelector* selector, 
RenderStyle* style, Element* element) const
 {
+    style->resetBorder();
+    style->resetBorderRadius();
+
        int labelSpacing = be_control_look ? 
static_cast<int>(be_control_look->DefaultLabelSpacing()) : 3;
     // Position the text correctly within the select box and make the box wide 
enough to fit the dropdown button
     style->setPaddingTop(Length(3, Fixed));

Other related posts:

  • » [haiku-webkit-commits] r367 - webkit/trunk/WebCore/platform/haiku - webkit