Bug 303056 – REGRESSION (299464@main): When "widows" affects a paragraph, "text-indent" incorrectly applies to second portion

• Status: RESOLVED
• Resolution: FIXED
• Component: Layout and Rendering

[Brad Andalman]

Created attachment 477498 [details]
HTML exhibiting the bug (improperly indented text)

In the attached HTML, a paragraph is split across two columns. So that the second column doesn't contain only one line, "widows: 2" is set. This does bring over a second line to the next column, but it is incorrectly indented.

This appears in Safari Technology Preview Release 232 (actually, I verified it in Release 229, too). This also appears in the Safari that is currently part of Tahoe 26.2: "Version 26.2 (21623.1.14.11.6)".

[Brad Andalman]

Created attachment 477499 [details]
Screenshot exhibiting the bug (improperly indented text)

Adding a screenshot that shows this bug in Safari Technology Preview 232 (and not in Safari 26.1)

[Radar WebKit Bug Importer]

<rdar://problem/165373054>

[Brad Andalman]

This bug has a big impact on our application, and we'd be happy to try to find the commit that's responsible, if you think it would help speed things up. Is there an easy way for us to do that? The nightlies only go back one month (though it does look like I can construct a URL to download nightlies a little beyond that). Is there a way to go back further? Or, perhaps there's a repo of older Safari Technology Previews?

If you have any advice on how to isolate the offending change, I'd be happy to help!

[alan]

Created attachment 477538 [details]
[fast-cq]Patch

[EWS]

Committed 303615@main (38155c35b096): <https://commits.webkit.org/303615@main>

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

[Brad Andalman]

It appears that this bug hasn't been entirely fixed. By amending the original HTML so that it uses `orphans: 1` and has an orphan, the bad "text-indent" appears again.

Reopening this bug, but please let me know if you'd rather I filed an entirely new one. (I'll be attaching the repro case shortly.)

[Brad Andalman]

Created attachment 478019 [details]
HTML that exhibits the bug when orphans are involved

[alan]

I’ll move it over to a new Bugzilla. Thanks for taking the time to put together a repro case and for following up on this.

[alan]

(bug 305497)