Bug 245453

Summary: REGRESSION (Safari 16): text-align-last causes table to take full space
Product: WebKit Reporter: Ebrahim Byagowi <ebrahim>
Component: TablesAssignee: alan <zalan>
Status: RESOLVED FIXED    
Severity: Normal CC: koivisto, mmaxfield, webkit-bug-importer, zalan
Priority: P2 Keywords: InRadar
Version: Safari Technology Preview   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Safari vs Chrome/Firefox screenshot for text-align-last effect
none
Test reduction
none
[fast-cq]Patch none

Ebrahim Byagowi
Reported 2022-09-20 16:33:19 PDT
Created attachment 462480 [details] Safari vs Chrome/Firefox screenshot for text-align-last effect Reproducible in Safari 16, Steps to reproduce Open the following page, https://fa.wikipedia.org/wiki/%D8%B1%D8%A8%D8%A7%D8%B9%DB%8C%D8%A7%D8%AA_%D8%AE%DB%8C%D8%A7%D9%85#%D9%85%D8%B6%D8%A7%D9%85%DB%8C%D9%86_%DA%A9%D9%84%DB%8C Expected: Like Firefox and Chrome, to not have table to fill whole space just because text-align-last is used Actual: The poem tables are filling the whole space just and words have unrealistic spaces between themselves.
Attachments
Safari vs Chrome/Firefox screenshot for text-align-last effect (3.12 MB, image/png)
2022-09-20 16:33 PDT, Ebrahim Byagowi 		no flags
Details
Test reduction (135 bytes, text/html)
2022-09-20 17:25 PDT, alan 		no flags
Details
[fast-cq]Patch (3.27 KB, patch)
2022-09-20 20:02 PDT, alan 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
alan
Comment 1 2022-09-20 17:25:49 PDT
Created attachment 462482 [details] Test reduction
Alexey Proskuryakov
Comment 2 2022-09-20 17:49:22 PDT
This doesn't reproduce in Safari 15.6.1, so it's a regression.
Radar WebKit Bug Importer
Comment 3 2022-09-20 17:49:39 PDT
<rdar://problem/100199451>
alan
Comment 4 2022-09-20 20:02:08 PDT
Created attachment 462483 [details] [fast-cq]Patch
Ebrahim Byagowi
Comment 5 2022-09-21 06:34:57 PDT
Tested the patch locally and worked just great, thanks for the quick fix, awesome.
alan
Comment 6 2022-09-21 07:00:07 PDT
(In reply to Ebrahim Byagowi from comment #5) > Tested the patch locally and worked just great, thanks for the quick fix, > awesome. Thank you for filing it (and testing the fix locally)! Very much appreciated!
EWS
Comment 7 2022-09-21 07:04:34 PDT
Committed 254725@main (04db6217eae0): <https://commits.webkit.org/254725@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 462483 [details].
Note You need to log in before you can comment on or make changes to this bug.