| Summary: | Floated <br> should not trigger a line break | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Simon Fraser (smfr) <simon.fraser> |
| Component: | Layout and Rendering | Assignee: | zalan <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 | ||
|
Description
Simon Fraser (smfr)
2021-09-08 09:25:56 PDT
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. 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.
(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. :-( repurpose it for the float case. out-of-flow case is here 259631 (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. |