[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    KDE/kdelibs/khtml/css
From:       Maks Orlovich <maksim () kde ! org>
Date:       2011-01-18 23:32:40
Message-ID: 20110118233240.3491CAC8B7 () svn ! kde ! org
[Download RAW message or body]

SVN commit 1215524 by orlovich:


Make sure we honor width: set on a button with display: inline. 
This is done by mutating it into inline-block (WebCore does that,
but a lot more other stuff, too, which doesn't make sense here given 
our native-widgets approach).

Fixes the star (so that's what the weird fancy character was) and loupe 
icons on google search results getting cutoff.



 M  +7 -0      cssstyleselector.cpp  


--- trunk/KDE/kdelibs/khtml/css/cssstyleselector.cpp #1215523:1215524
@@ -899,6 +899,13 @@
             }
             else
                 style->setDisplay(BLOCK);
+        } else if (e && e->id() == ID_BUTTON && \
style->isOriginalDisplayInlineType()) { +            // <button>s are supposed to be \
replaced elements; but we don't handle  +            // them as such (as they are \
rendered as CSS contexts, not natives  +            // with intrinsic sizes), so we \
must be careful not to make them fully  +            // inline, as that will display \
stuff like width:; so mutate inline-like +            // display types into \
inline-block +            style->setDisplay(INLINE_BLOCK);
         }
 
         // After performing the display mutation, check our position.  We do not \
honor position:relative on


[prev in list] [next in list] [prev in thread] [next in thread]