Bug 230048
| Summary: | Floated <br> should not trigger a line break | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Simon Fraser (smfr) <simon.fraser> |
| Component: | Layout and Rendering | Assignee: | alan <zalan> |
| Status: | NEW | ||
| Severity: | Normal | CC: | ahmad.saleem792, bfulgham, simon.fraser, webkit-bug-importer, zalan |
| Priority: | P2 | Keywords: | BrowserCompat, InRadar |
| Version: | Safari Technology Preview | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| URL: | https://jsfiddle.net/Lf79njp5/show | ||
| See Also: | https://bugs.webkit.org/show_bug.cgi?id=259631 | ||
Simon Fraser (smfr)
https://github.com/w3c/csswg-drafts/issues/5749
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Radar WebKit Bug Importer
<rdar://problem/83154050>
Ahmad Saleem
Safari / WebKit ToT matches Chrome Canary 118 while Firefox Nightly 116 differs from both.
Took examples of @Simon's post from GitHub and changed into JSFiddle and linked.
alan
This is pretty much about removing the following lines: (now that the spec is clear)
clonedStyle.setDisplay(DisplayType::Inline);
clonedStyle.setFloating(Float::None);
clonedStyle.setPosition(PositionType::Static);
in TreeBuilder::createLayoutBox.
will fix it soon.
Ahmad Saleem
(In reply to zalan from comment #3)
> This is pretty much about removing the following lines: (now that the spec
> is clear)
> clonedStyle.setDisplay(DisplayType::Inline);
> clonedStyle.setFloating(Float::None);
> clonedStyle.setPosition(PositionType::Static);
> in TreeBuilder::createLayoutBox.
>
> will fix it soon.
Deleting these three lines didn't fix it on local for me. :-(
alan
repurpose it for the float case. out-of-flow case is here 259631
alan
bug 259631
alan
(In reply to Ahmad Saleem from comment #4)
> (In reply to zalan from comment #3)
> > This is pretty much about removing the following lines: (now that the spec
> > is clear)
> > clonedStyle.setDisplay(DisplayType::Inline);
> > clonedStyle.setFloating(Float::None);
> > clonedStyle.setPosition(PositionType::Static);
> > in TreeBuilder::createLayoutBox.
> >
> > will fix it soon.
>
> Deleting these three lines didn't fix it on local for me. :-(
yeah it's slightly more involved.