Bug 245453 - REGRESSION (Safari 16): text-align-last causes table to take full space
Summary: REGRESSION (Safari 16): text-align-last causes table to take full space
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tables (show other bugs)
Version: Safari Technology Preview
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: zalan
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2022-09-20 16:33 PDT by Ebrahim Byagowi
Modified: 2022-09-21 07:04 PDT (History)
4 users (show)

See Also:

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, zalan 		no flags Details
[fast-cq]Patch (3.27 KB, patch)
2022-09-20 20:02 PDT, zalan 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ebrahim Byagowi 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.
Comment 1 zalan 2022-09-20 17:25:49 PDT 
Created attachment 462482 [details]
Test reduction
Comment 2 Alexey Proskuryakov 2022-09-20 17:49:22 PDT 
This doesn't reproduce in Safari 15.6.1, so it's a regression.
Comment 3 Radar WebKit Bug Importer 2022-09-20 17:49:39 PDT 
<rdar://problem/100199451>
Comment 4 zalan 2022-09-20 20:02:08 PDT 
Created attachment 462483 [details]
[fast-cq]Patch
Comment 5 Ebrahim Byagowi 2022-09-21 06:34:57 PDT 
Tested the patch locally and worked just great, thanks for the quick fix, awesome.
Comment 6 zalan 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!
Comment 7 EWS 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].