Bug 245453

Summary:

REGRESSION (Safari 16): text-align-last causes table to take full space

Product:

WebKit

Reporter:

Ebrahim Byagowi <ebrahim>

Component:

Tables

Assignee:

zalan <zalan>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

koivisto, mmaxfield, webkit-bug-importer, zalan

Priority:

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

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].