[Webkit-unassigned] [Bug 79128] Fieldset unexpectedly stretches to minimum intrinsic width.
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Tue Mar 20 10:15:22 PDT 2012
https://bugs.webkit.org/show_bug.cgi?id=79128
Julien Chaffraix <jchaffraix at webkit.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #132768|review? |
Flag| |
--- Comment #13 from Julien Chaffraix <jchaffraix at webkit.org> 2012-03-20 10:15:21 PST ---
(From update of attachment 132768)
View in context: https://bugs.webkit.org/attachment.cgi?id=132768&action=review
> Source/WebCore/rendering/RenderFieldset.cpp:201
> + // If width is exclusively specified then Fieldsets should not stretch
> + if (style()->width().isPercent())
It's undesirable to have different behavior between specified widths. If fixed positions don't work here, I think our logic needs to be changed elsewhere so that they work the same.
Also the fixed position case *should* be tested.
> Source/WebCore/rendering/RenderFieldset.h:45
> + virtual bool stretchesToMinIntrinsicLogicalWidth() const;
Still extra-space.
> LayoutTests/fast/forms/fieldset-percent-width-nostretch-ifspecified-expected.html:17
> + 111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111
First a test should include:
* the bug title
* the bug number
* condition to pass (our container should be 50% here)
> LayoutTests/fast/forms/fieldset-percent-width-nostretch-ifspecified.html:1
> +<html>
Please add a doctype (those comments are also for the -expected.html):
<!DOCTYPE html>
> LayoutTests/fast/forms/fieldset-percent-width-nostretch-ifspecified.html:3
> + <style type="text/css" media="screen">
No need for the attribute.
> LayoutTests/fast/forms/fieldset-percent-width-nostretch-ifspecified.html:15
> + <div class="wrap">
the wrap class is unused. I would add it back and put a fixed size on this wrapper to limit the potential for cross-browser differences.
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list