Bug 220252

Summary: [LFC][IFC][Quirk] Let's adopt a less quirky behavior for line breaks on non-empty lines
Product: WebKit Reporter: zalan <zalan>
Component: Layout and RenderingAssignee: zalan <zalan>
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, changseok, esprehn+autocc, ews-watchlist, glenn, koivisto, kondapallykalyan, mmaxfield, pdr, simon.fraser, webkit-bug-importer, zalan
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch
none
Patch
none
Patch
none
Patch
none
Patch
none
Patch none

Description zalan 2021-01-04 12:46:47 PST 
<br> in non-standard mode stretches the line box only when the line is empty.
e.g. <div><span><br></span></div> will stretch but <div>this will not stretch to 200px<span style="font-size: 200px;"><br></span></div>
However it's way too quirky and it does not even match Chrome's behavior.
Comment 1 zalan 2021-01-04 12:52:51 PST 
Created attachment 416960 [details]
Patch
Comment 2 zalan 2021-01-10 19:59:26 PST 
Created attachment 417359 [details]
Patch
Comment 3 Antti Koivisto 2021-01-10 22:27:08 PST 
Comment on attachment 417359 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=417359&action=review

> Source/WebCore/ChangeLog:17
> +        Removing this quirk makes the inlineLevelBoxAffectsLineBox logic simpler since we don't have to check against
> +        whether the line is considered empty and the result is aligned with Chrome's behavior.

Should it be removed from legacy too?
Comment 4 zalan 2021-01-11 10:08:20 PST 
(In reply to Antti Koivisto from comment #3)
> Comment on attachment 417359 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=417359&action=review
> 
> > Source/WebCore/ChangeLog:17
> > +        Removing this quirk makes the inlineLevelBoxAffectsLineBox logic simpler since we don't have to check against
> > +        whether the line is considered empty and the result is aligned with Chrome's behavior.
> 
> Should it be removed from legacy too?
Yeah good point.
Comment 5 Radar WebKit Bug Importer 2021-01-11 12:47:12 PST 
<rdar://problem/73009786>
Comment 6 zalan 2021-01-11 16:45:09 PST 
Created attachment 417421 [details]
Patch
Comment 7 zalan 2021-01-12 12:57:12 PST 
Created attachment 417482 [details]
Patch
Comment 8 zalan 2021-01-12 13:02:43 PST 
Created attachment 417484 [details]
Patch
Comment 9 zalan 2021-01-12 21:20:12 PST 
Created attachment 417506 [details]
Patch
Comment 10 zalan 2021-01-13 20:41:22 PST 
Created attachment 417594 [details]
Patch
Comment 11 EWS 2021-01-14 06:30:23 PST 
Committed r271481: <https://trac.webkit.org/changeset/271481>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 417594 [details].