2063167 - [Nova] Title fadeout remains on wrap when unnecessary

Status: VERIFIED FIXED

(Firefox :: Address Bar, defect, P2)

Description

[Drew Willcoxon :adw]

Attachment: screenshot

The title fadeout remains when wrap is active even when it's unnecessary, at least in some cases.

STR

1. Start Firefox
2. Do a query that triggers a longish title and/or URL so that the title fadeout is triggered before wrapping happens
3. Make the window narrow so wrapping is triggered

The title will have the fadeout even when it's not necessary.

Proton doesn't have this problem and I'm guessing it's related to using grid on wrap.

Comment 1

[Drew Willcoxon :adw]

Daisuke said he'll work on this next week.

Comment 2

[Drew Willcoxon :adw]

My patch in bug 2063457 doesn't fix this.

Comment 3

[Daisuke Akatsuka (:daisuke)]

Attachment: Bug 2063167 - Keep scroll container's overflow state across frame reconstruction so underflow is fired.

The fade-out mask was left on result titles once rows started wrapping. Wrapping
switches the row body containing the title from display: flex to
display: grid, which recreates the frames inside it, and no underflow event
was fired for the recreated frames even though the title was then wide enough
for its contents.
This is because the overflow state is reset when reframing, so carry it over to
the new frame.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/719ac364b859
https://hg.mozilla.org/integration/autoland/rev/7dcaf32ecff9
Keep scroll container's overflow state across frame reconstruction so underflow is fired. r=layout-reviewers,emilio,hiro

Comment 5

[Iulian Moraru]

https://hg.mozilla.org/mozilla-central/rev/7dcaf32ecff9

Comment 6

[Ardelean Oana, Desktop Test Engineering [:oardelean]]

Reproducible on a 2026-08-12 Firefox Nightly build on Windows 10.

Verified as fixed on Firefox Nightly 156.0a1 on Windows 10, Ubuntu 22, macOS 13.

However, we did note that the fade-out effect disappears after resizing the browser window in some instances. We logged bug 2066093 for that particular scenario.

Please let us know if there’s anything else we should take in to account, thank you.