Hi Julien, Thanks for the reply, there is a bug filed on this, can we continue the discussion over there. https://bugs.webkit.org/show_bug.cgi?id=79128
-Sravan. On Sun, Mar 18, 2012 at 12:11 AM, Julien Chaffraix < julien.chaffr...@gmail.com> wrote: > > I was trying to understand why only for Fieldsets following logic is > > applied? (RenderBox.cpp) > > > > // Fieldsets are currently the only objects that stretch to their > > minimum width. > > if (stretchesToMinIntrinsicLogicalWidth()) { > > > > and why this function(RenderFieldset.h) is returning true with out > checking > > for width being set by css. > > > > virtual bool stretchesToMinIntrinsicLogicalWidth() const { return > true; > > }. > > There is a comment on RenderBox.h [1]: > > // Whether or not the element shrinks to its intrinsic width (rather > than filling the width > // of a containing block). HTML4 buttons, <select>s, <input>s, > legends, and floating/compact elements do this. > > I guess that explains why we don't check CSS width for RenderFieldset > (intrinsic sizes are never impacted by CSS). If you told us why you > think this is wrong, we may be able to give a better answer. > > Thanks, > Julien > > [1] > http://trac.webkit.org/browser/trunk/Source/WebCore/rendering/RenderBox.h#L335 >
_______________________________________________ webkit-help mailing list webkit-help@lists.webkit.org http://lists.webkit.org/mailman/listinfo.cgi/webkit-help