Closed Bug 1851323 Opened 9 months ago Closed 9 months ago

Table layout is broken on certain web page

Categories

(Core :: Layout, defect, P3)

Firefox 119
Desktop
Windows 10
defect

Tracking

()

VERIFIED FIXED
119 Branch
Tracking Status
firefox-esr102 --- unaffected
firefox-esr115 --- unaffected
firefox117 --- unaffected
firefox118 --- disabled
firefox119 --- verified

People

(Reporter: alice0775, Assigned: m_kato)

References

(Regression)

Details

(Keywords: nightly-community, regression)

Attachments

(2 files)

Attached image screenshot

Steps to reprodue:

  1. Open https://academysoftwarefoundation.github.io/EncodingGuidelines/tests/greyramp-osx/compare.html?.colortrc

Actual Results:
Table row height is larger than Firefox117.0 and Chrome.
Thus the gap between video elements is larger.

Expected Results:
The gap is almost 0.

Regression window:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=6d877fdb9a1e892fe6528a26aab81b53cfae55c5&tochange=d6760db62dc1cc0831c09c42f086cf57d1c2bf48

:m_kato, since you are the author of the regressor, bug 1719535, could you take a look? Also, could you set the severity field?

For more information, please visit BugBot documentation.

Flags: needinfo?(m_kato)

IS x AL shouldn't break opportunity on UAX14 spec (LB29).

Status: NEW → RESOLVED
Closed: 9 months ago
Duplicate of bug: 1848049
Flags: needinfo?(m_kato)
Resolution: --- → DUPLICATE
Status: RESOLVED → REOPENED
No longer duplicate of bug: 1848049
Resolution: DUPLICATE → ---
Assignee: nobody → m_kato
Severity: -- → S2
Priority: -- → P3

Makoto, do you know when is that patch end up in FF?

Flags: needinfo?(m_kato)

(In reply to Emilio Cobos Álvarez (:emilio) from comment #4)

Makoto, do you know when is that patch end up in FF?

Although this will fixed by ICU4X 1.3, it can import the fix since this issue is data file only. So I will fix this soon.

Flags: needinfo?(m_kato)

Since some issues such as bug 1851323 are fixed by ICU4X 1.3's data rule file,
I would like to apply it to get feedback in Gecko.

Pushed by m_kato@ga2.so-net.ne.jp:
https://hg.mozilla.org/integration/autoland/rev/a953dc51ae0f
Apply the latest line break rules. r=TYLin
Status: REOPENED → RESOLVED
Closed: 9 months ago9 months ago
Resolution: --- → FIXED
Target Milestone: --- → 119 Branch
Flags: qe-verify+

Reproduced the issue with 119.0a1 (2023-09-02) on Windows 10x64. The table row line height was bigger.
Verified fixed with Firefox 119.0 on Windows 10x64, macOS 12 and Ubuntu 22 following STR from comment 0. The table row height is correctly displayed.

Status: RESOLVED → VERIFIED
Has STR: --- → yes
Flags: qe-verify+ → qe-verify-
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: