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));