NEW 230048
Floated <br> should not trigger a line break 
https://bugs.webkit.org/show_bug.cgi?id=230048
Summary Floated <br> should not trigger a line break
Simon Fraser (smfr)
Reported 2021-09-08 09:25:56 PDT
https://github.com/w3c/csswg-drafts/issues/5749
Attachments
Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2021-09-15 09:26:21 PDT
<rdar://problem/83154050>
Ahmad Saleem
Comment 2 2023-07-04 04:21:05 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.
zalan
Comment 3 2023-07-04 07:25:33 PDT
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
Comment 4 2023-07-29 06:37:34 PDT
(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. :-(
zalan
Comment 5 2023-07-29 15:58:16 PDT
repurpose it for the float case. out-of-flow case is here 259631
zalan
Comment 6 2023-07-29 15:58:28 PDT
bug 259631
zalan
Comment 7 2023-07-29 16:02:26 PDT
(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.
Note You need to log in before you can comment on or make changes to this bug.