1901769 - [wayland] Async clipboard "Paste" permission context menu appears in wrong position and flickers

Status: VERIFIED FIXED

(Core :: XUL, defect, P2)

Description

[danielhunterjacobs]

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:126.0) Gecko/20100101 Firefox/126.0

Steps to reproduce:

Open Firefox 126 on Fedora Linux 39 running Gnome on wayland.
Open https://jsfiddle.net/zm490d6a/
Copy some text so something is is the clipboard.
Click inside the textbox.

Actual results:

No menu with the clickable item "Paste" appeared.

Expected results:

A menu with the clickable item "Paste" should have appeared, which when click pasted the text from the clipboard. This happens on Gnome on Xorg (x11), just not on Gnome (wayland)

Comment 1

[danielhunterjacobs]

Attachment: Behavior on wayland, where the bug occurs

Behavior on wayland, where the bug occurs (regular GNOME)

Comment 2

[danielhunterjacobs]

Attachment: Behavior on x11, where the bug does not occur

Behavior on x11, where the bug does not occur (GNOME on Xorg)

Comment 3

[Edgar Chen [:edgar]]

Thanks for reporting this issue, I am not able to use wayland at the moment. Could you help to capture the log?

• Open about:logging
• Set log module to WidgetClipboard:5
• Start the logging
• Reproduce the issue
• Stop the logging
• Share the profiler link to us

Comment 4

[danielhunterjacobs]

Attachment: Screen recording of getting the Profiler info

Comment 5

[danielhunterjacobs]

I hope I did this right, I screen recorded what I did. This is the link to the profiler: https://profiler.firefox.com/from-browser/calltree/?globalTrackOrder=bc0wa&hiddenGlobalTracks=1w9&hiddenLocalTracksByPid=2296294-024wa%7E2296657-0%7E2297804-0%7E2296442-01%7E2876260-01%7E2875023-01%7E2296541-01%7E2873617-01%7E2296477-01%7E2827058-01%7E2824679-0&thread=s&v=10

Comment 6

[danielhunterjacobs]

Attachment: Profiler

Comment 7

[danielhunterjacobs]

Public profiler link: https://profiler.firefox.com/public/njqp7cvrssdxz53hx4864z9dxpc5nsxrjjg6058/calltree/?globalTrackOrder=01&thread=3&v=10

Comment 8

[danielhunterjacobs]

Attachment: Clipboard log file (not profiler)

Comment 9

[Edgar Chen [:edgar]]

On my wayland, the paste contextmenu appears on the top-left corner of the browser window.
I used to see the paste button being rendered in right position, I think it is a regression.
And I got the following regression window:
https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=d3ddc98e7113bf3be63f35e439981aa3adb17e4d&tochange=b7f6557aaf06a26f589489fc439e74fd6e75a022

After reverting https://hg.mozilla.org/mozilla-central/rev/c6030081762c7e5285b255d4494c0368e4b920e0, the paste contextmenu is rendered in right position again.

Comment 10

[Edgar Chen [:edgar]]

:stransky, could you take a look? Thanks!

Comment 11

[Kestrel]

The async clipboard Paste context menu appears for me in the top-left corner of the window on Ubuntu 24.04 (Wayland).

Async clipboard was recently enabled for text in version 125 (Bug 1877400) (dom.events.asyncClipboard.readText) and other content in version 127 (Bug 1887845) (dom.events.asyncClipboard.clipboardItem).

Comment 12

[Kestrel]

Attachment: Async clipboard Paste context menu flickering in window corner

Comment 13

[Martin Stránský [:stransky] (ni? me)]

Looks like we're getting wrong popup position from layout:
https://searchfox.org/mozilla-central/rev/2f48061aef8c8976b73749ee845e7b85751f5f2f/widget/gtk/nsWindow.cpp#2408

[Parent 51917: Main Thread]: D/WidgetPopup [7fd70958ce00]: nsWindow::WaylandPopupGetPositionFromLayout
[Parent 51917: Main Thread]: D/WidgetPopup [7fd70958ce00]:   layout popup CSS anchor (0, 0) (x=1560, y=1380, w=57600, h=61080), margin (t=240, r=240, b=240, l=240) offset (0,0)
[Parent 51917: Main Thread]: D/WidgetPopup [7fd70958ce00]:     after margins (x=1320, y=1140, w=58080, h=61560)
[Parent 51917: Main Thread]: D/WidgetPopup [7fd70958ce00]:     final (x=44, y=38, w=1936, h=2052)
[Parent 51917: Main Thread]: D/WidgetPopup [7fd70958ce00]:   relative popup rect position [44, 38] -> [1936 x 2052]

But the popup is not anchored so we may not use layout anchor rect but rather create our own one:

[Parent 51917: Main Thread]: D/WidgetPopup [7fd70958ce00]: nsWindow::WaylandPopupHierarchyCalculatePositions()
[Parent 51917: Main Thread]: D/WidgetPopup [7fd70958ce00]:   popup [7fd70958ce00] set parent window [7fd732fe6a00]
[Parent 51917: Main Thread]: D/WidgetPopup [7fd70958ce00]:   popup [7fd70958ce00] bounds [783, 667] -> [63 x 24]
[Parent 51917: Main Thread]: D/WidgetPopup [7fd70958ce00]:   popup [7fd70958ce00] layout [1514, 1288] -> [126 x 48]
[Parent 51917: Main Thread]: D/WidgetPopup [7fd70958ce00]:   popup [7fd70958ce00] has toplevel as parent
[Parent 51917: Main Thread]: D/WidgetPopup [7fd70958ce00]:   popup [7fd70958ce00] transformed popup coordinates from [783, 667] to [783, 667]
[Parent 51917: Main Thread]: D/WidgetPopup [7fd70958ce00]:   popup [7fd70958ce00] matches layout [1] anchored [0] first popup [1] use move-to-rect 1

Emilio, I'm not very familiar with the layout code. Do you have an idea why popup layout anchor is different and do you think we may ignore it for non anchored popups?

Thanks.

Comment 14

[Emilio Cobos Álvarez [:emilio]]

Edgar do you know why this code is doing something different to this which is what the context menu does?

I think the wayland code is getting confused with overlap, and I can dig into that, but it'd be great to know why that diverges from the regular context menu?

Comment 15

[Edgar Chen [:edgar]]

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

Edgar do you know why this code is doing something different to this which is what the context menu does?

I think the wayland code is getting confused with overlap, and I can dig into that, but it'd be great to know why that diverges from the regular context menu?

Good point. I will check that. I don't see a reason to do something different from the regular context menu.

Comment 16

[Emilio Cobos Álvarez [:emilio]]

Attachment: Bug 1901769 - Make nsMenuPopupFrame offset handling simpler. r=edgar,NeilDeakin,TYLin

• Remove unused after_pointer position.
• Make mXPos/mYPos work in terms of margin (mExtraMargin).
• When we have no anchor, explicitly upgrade them to anchored-to-point.
• Make MenuPopupAnchorType an enum class.

Comment 17

[Emilio Cobos Álvarez [:emilio]]

I got some time to dig into this. The reason we get into the bad state is basically this.

The way we're handling this is rather complicated. Comment 16 substantially simplifies it in order to make Wayland automatically account for it.

Comment 18

[Edgar Chen [:edgar]]

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

I got some time to dig into this. The reason we get into the bad state is basically this.

The way we're handling this is rather complicated. Comment 16 substantially simplifies it in order to make Wayland automatically account for it.

Thank you!

Comment 19

[Emilio Cobos Álvarez [:emilio]]

Attachment: Bug 1901769 - De indent some code in nsMenuPopupFrame::OpenPopup. r=#layout

Comment 20

[Emilio Cobos Álvarez [:emilio]]

Attachment: Bug 1901769 - Make MenuPopupAnchorType an enum class. r=#layout

Comment 21

[Emilio Cobos Álvarez [:emilio]]

Attachment: Bug 1901769 - Remove unused after_pointer popup position. r=#layout

Comment 22

[Emilio Cobos Álvarez [:emilio]]

Attachment: Bug 1901769 - Add some braces in nsMenuPopupFrame::MoveToAttributePosition. r=#layout

Comment 23

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/75398f72448b
De indent some code in nsMenuPopupFrame::OpenPopup. r=layout-reviewers,jfkthame
https://hg.mozilla.org/integration/autoland/rev/6dba8057201b
Make MenuPopupAnchorType an enum class. r=layout-reviewers,jfkthame
https://hg.mozilla.org/integration/autoland/rev/c5a4f119cfd1
Remove unused after_pointer popup position. r=layout-reviewers,jfkthame
https://hg.mozilla.org/integration/autoland/rev/d05bacf3f608
Add some braces in nsMenuPopupFrame::MoveToAttributePosition. r=layout-reviewers,jfkthame

Comment 24

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/75398f72448b
https://hg.mozilla.org/mozilla-central/rev/6dba8057201b
https://hg.mozilla.org/mozilla-central/rev/c5a4f119cfd1
https://hg.mozilla.org/mozilla-central/rev/d05bacf3f608

Comment 25

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/4762fcad9e64
Make nsMenuPopupFrame offset handling simpler. r=edgar,NeilDeakin,TYLin

Comment 26

[Sandor Molnar[:smolnar]]

Backed out for causing mochitest failures @ test_panel_anchoradjust.xhtml

Backout link: https://hg.mozilla.org/integration/autoland/rev/48ec1a77087f3ee08e2e10f23242831912a30d46

Push with failures

Failure log -> TEST-UNEXPECTED-FAIL | toolkit/content/tests/chrome/test_panel_anchoradjust.xhtml | new anchor with offset x: 92, 78

Comment 27

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/d5c4e504dfd6
Make nsMenuPopupFrame offset handling simpler. r=edgar,NeilDeakin,TYLin

Comment 28

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/2eeba3d06bf9
Fix RTL offsets to be logical. r=fix
https://hg.mozilla.org/integration/autoland/rev/bf75c1b988a0
Make browser_test_clipboard_contextmenu.js work with native macOS context menus. r=fix

Comment 29

[tszentpeteri]

Backed out for causing mochitest failures on test_bug477754.xhtml

• Backout link
• Push with failures
• Failure Log:

[task 2024-08-05T15:12:03.709Z] 15:12:03     INFO - TEST-START | layout/xul/test/test_bug477754.xhtml
[task 2024-08-05T15:12:03.712Z] 15:12:03     INFO - GECKO(6184) | [Parent 6184, Main Thread] WARNING: NS_ENSURE_SUCCESS(rv, rv) failed with result 0x80004005 (NS_ERROR_FAILURE): file /builds/worker/checkouts/gecko/chrome/nsChromeRegistry.cpp:182
[task 2024-08-05T15:12:03.713Z] 15:12:03     INFO - GECKO(6184) | [Parent 6184, Main Thread] WARNING: 'NS_FAILED(rv)', file /builds/worker/checkouts/gecko/chrome/nsChromeProtocolHandler.cpp:73
[task 2024-08-05T15:12:03.881Z] 15:12:03     INFO - GECKO(6184) | [WARN  webrender::renderer::init] asking to enable_gpu_markers but no supporting extension was found
[task 2024-08-05T15:12:04.048Z] 15:12:04     INFO - TEST-INFO | started process screentopng
[task 2024-08-05T15:12:04.330Z] 15:12:04     INFO - TEST-INFO | screentopng: exit 0
[task 2024-08-05T15:12:04.331Z] 15:12:04     INFO - TEST-UNEXPECTED-FAIL | layout/xul/test/test_bug477754.xhtml | RTL popup's right offset should be equal to the x offset passed to openPopup - got -10, expected 10
[task 2024-08-05T15:12:04.332Z] 15:12:04     INFO - SimpleTest.is@chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:509:14
[task 2024-08-05T15:12:04.333Z] 15:12:04     INFO - doTest@chrome://mochitests/content/chrome/layout/xul/test/test_bug477754.xhtml:44:9
[task 2024-08-05T15:12:04.334Z] 15:12:04     INFO - onpopupshown@chrome://mochitests/content/chrome/layout/xul/test/test_bug477754.xhtml:1:1
[task 2024-08-05T15:12:04.336Z] 15:12:04     INFO - GECKO(6184) | MEMORY STAT | vsize 11862MB | residentFast 529MB | heapAllocated 248MB
[task 2024-08-05T15:12:04.337Z] 15:12:04     INFO - GECKO(6184) | [Parent 6184, Main Thread] WARNING: NS_ENSURE_SUCCESS(rv, rv) failed with result 0x80004005 (NS_ERROR_FAILURE): file /builds/worker/checkouts/gecko/chrome/nsChromeRegistry.cpp:182
[task 2024-08-05T15:12:04.339Z] 15:12:04     INFO - GECKO(6184) | [Parent 6184, Main Thread] WARNING: 'NS_FAILED(rv)', file /builds/worker/checkouts/gecko/chrome/nsChromeProtocolHandler.cpp:73
[task 2024-08-05T15:12:04.342Z] 15:12:04     INFO - GECKO(6184) | [Parent 6184, Main Thread] WARNING: Failed to retarget HTML data delivery to the parser thread.: file /builds/worker/checkouts/gecko/parser/html/nsHtml5StreamParser.cpp:1215
[task 2024-08-05T15:12:04.343Z] 15:12:04     INFO - TEST-OK | layout/xul/test/test_bug477754.xhtml | took 612ms
[task 2024-08-05T15:12:04.461Z] 15:12:04     INFO - TEST-START | layout/xul/test/test_bug703150.xhtml

Comment 30

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/0d53cc2130df
Make nsMenuPopupFrame offset handling simpler. r=edgar,NeilDeakin,TYLin

Comment 31

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/0d53cc2130df

Comment 32

[Norisz Fay [:noriszfay]]

Backed out for causing bc failure on browser_navigator_clipboard_clickjacking.js

Backout link

Push with failures

Failure log

Comment 33

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c4dd6ccf94ac
Make nsMenuPopupFrame offset handling simpler. r=edgar,NeilDeakin,TYLin

Comment 34

[tszentpeteri]

https://hg.mozilla.org/mozilla-central/rev/c4dd6ccf94ac

Comment 35

[tszentpeteri]

Backed out for causing mochitest failures on browser_navigator_clipboard_contextmenu_suppression.js

• Backout link
• Push with failures
• Failure Log:

[task 2024-08-08T02:22:00.053Z] 02:22:00     INFO - TEST-START | dom/events/test/clipboard/browser_navigator_clipboard_contextmenu_suppression.js
[task 2024-08-08T02:22:17.932Z] 02:22:17     INFO - GECKO(1897) | 2024-08-08 02:22:17.931 firefox[1897:46346] Persistent UI failed to open file file:///Users/cltbld/Library/Saved%20Application%20State/org.mozilla.nightly.savedState/window_1.data: No such file or directory (2)
[task 2024-08-08T02:23:35.181Z] 02:23:35     INFO - TEST-INFO | started process screencapture
[task 2024-08-08T02:23:35.298Z] 02:23:35     INFO - TEST-INFO | screencapture: exit 0
[task 2024-08-08T02:23:35.298Z] 02:23:35     INFO - Buffered messages logged at 02:22:00
[task 2024-08-08T02:23:35.298Z] 02:23:35     INFO - Entering setup bound 
[task 2024-08-08T02:23:35.299Z] 02:23:35     INFO - Leaving setup bound 
[task 2024-08-08T02:23:35.299Z] 02:23:35     INFO - Entering test bound test_context_menu_suppression_sameorigin
[task 2024-08-08T02:23:35.300Z] 02:23:35     INFO - Console message: [JavaScript Warning: "The character encoding of a framed document was not declared. The document may appear different if viewed without the document framing it." {file: "https://example.com/browser/dom/events/test/clipboard/file_iframe.html" line: 0}]
[task 2024-08-08T02:23:35.300Z] 02:23:35     INFO - Console message: [JavaScript Warning: "Partitioned cookie or storage access was provided to “https://example.org/browser/dom/events/test/clipboard/file_iframe.html” because it is loaded in the third-party context and dynamic state partitioning is enabled."]
[task 2024-08-08T02:23:35.300Z] 02:23:35     INFO - Write data by clipboard.writeText()
[task 2024-08-08T02:23:35.301Z] 02:23:35     INFO - Test read from same-origin frame
[task 2024-08-08T02:23:35.301Z] 02:23:35     INFO - TEST-PASS | dom/events/test/clipboard/browser_navigator_clipboard_contextmenu_suppression.js | read should just be resolved without paste contextmenu shown - 
[task 2024-08-08T02:23:35.301Z] 02:23:35     INFO - Leaving test bound test_context_menu_suppression_sameorigin
[task 2024-08-08T02:23:35.302Z] 02:23:35     INFO - Entering test bound test_context_menu_suppression_crossorigin
[task 2024-08-08T02:23:35.302Z] 02:23:35     INFO - Console message: [JavaScript Warning: "The character encoding of a framed document was not declared. The document may appear different if viewed without the document framing it." {file: "https://example.com/browser/dom/events/test/clipboard/file_iframe.html" line: 0}]
[task 2024-08-08T02:23:35.303Z] 02:23:35     INFO - Console message: [JavaScript Warning: "Partitioned cookie or storage access was provided to “https://example.org/browser/dom/events/test/clipboard/file_iframe.html” because it is loaded in the third-party context and dynamic state partitioning is enabled."]
[task 2024-08-08T02:23:35.303Z] 02:23:35     INFO - Write data by clipboard.writeText()
[task 2024-08-08T02:23:35.303Z] 02:23:35     INFO - Test read from cross-origin frame
[task 2024-08-08T02:23:35.304Z] 02:23:35     INFO - Wait for paste button enabled
[task 2024-08-08T02:23:35.304Z] 02:23:35     INFO - Click paste button, request should be resolved
[task 2024-08-08T02:23:35.305Z] 02:23:35     INFO - Buffered messages logged at 02:22:50
[task 2024-08-08T02:23:35.305Z] 02:23:35     INFO - Longer timeout required, waiting longer...  Remaining timeouts: 1
[task 2024-08-08T02:23:35.305Z] 02:23:35     INFO - Buffered messages finished
[task 2024-08-08T02:23:35.306Z] 02:23:35     INFO - TEST-UNEXPECTED-FAIL | dom/events/test/clipboard/browser_navigator_clipboard_contextmenu_suppression.js | Test timed out - 
[task 2024-08-08T02:23:35.306Z] 02:23:35     INFO - GECKO(1897) | Completed ShutdownLeaks collections in process 1897
[task 2024-08-08T02:23:35.307Z] 02:23:35     INFO - TEST-START | Shutdown

Comment 36

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a73a333c97d7
Make nsMenuPopupFrame offset handling simpler. r=edgar,NeilDeakin,TYLin

Comment 37

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/f03e7d26ef79
Keep overlap position to make macOS test happy.

Comment 38

[Norisz Fay [:noriszfay]]

https://hg.mozilla.org/mozilla-central/rev/a73a333c97d7
https://hg.mozilla.org/mozilla-central/rev/f03e7d26ef79

Comment 39

[BugBot [:suhaib / :marco/ :calixte]]

The patch landed in nightly and beta is affected.
:emilio, is this bug important enough to require an uplift?

• If yes, please nominate the patch for beta approval.
• If no, please set status-firefox130 to wontfix.

For more information, please visit BugBot documentation.

Comment 40

[Emilio Cobos Álvarez [:emilio]]

Comment on attachment 9416541 [details]
Bug 1901769 - Add some braces in nsMenuPopupFrame::MoveToAttributePosition. r=#layout

Beta/Release Uplift Approval Request

• User impact if declined: Clipboard API on Linux Wayland doesn't work properly.
• Is this code covered by automated tests?: Yes
• Has the fix been verified in Nightly?: Yes
• Needs manual test from QE?: Yes
• If yes, steps to reproduce: comment 0
• List of other uplifts needed: none
• Risk to taking this patch: Medium
• Why is the change risky/not risky? (and alternatives if risky): It's not the trivialest code, though the reason it bounced so much was mostly macOS tests due to using native context menus. Consider waiting a week or so before uplift perhaps?
• String changes made/needed: none
• Is Android affected?: Yes

ESR Uplift Approval Request

• If this is not a sec:{high,crit} bug, please state case for ESR consideration: Breaks clipboard API on Linux.
• User impact if declined: Clipboard API on Linux Wayland doesn't work properly.
• Fix Landed on Version: 131
• Risk to taking this patch: Medium
• Why is the change risky/not risky? (and alternatives if risky): See above

Comment 41

[Bogdan Maris, Desktop Test Engineering]

Verified the initial issue using Firefox 126 on Ubuntu 22.04 Wayland, with the paste button shown in the browser corner.
Verified that using latest Nightly 131.0a1 the paste button is shown where the cursor is when clicking inside the text area.

Comment 42

[Pascal Chevrel (relman team) -> :pascalc]

Comment on attachment 9416541 [details]
Bug 1901769 - Add some braces in nsMenuPopupFrame::MoveToAttributePosition. r=#layout

This is a long standing regression and we would have to land it at the end of the beta cycle for 130, let's have it ride the 131 train, thanks.

Comment 43

[Dianna Smith [:diannaS]]

Comment on attachment 9416540 [details]
Bug 1901769 - Remove unused after_pointer popup position. r=#layout

Approved for 128.3esr

Comment 44

[Pulsebot]

https://hg.mozilla.org/releases/mozilla-esr128/rev/f45d2d28031d
https://hg.mozilla.org/releases/mozilla-esr128/rev/2142781397eb

Comment 45

[Bogdan Maris, Desktop Test Engineering]

I used the latest Firefox 128 ESR build that does have the fix (https://treeherder.mozilla.org/jobs?repo=mozilla-esr128&revision=927837081d962926f7452d727eae04dfcfb6707e) and it does NOT look fixed to me, I can still reproduce the initial issue on Ubuntu 22.04 wayland.

Comment 46

[Emilio Cobos Álvarez [:emilio]]

Well yeah that's because of course only two patches were requested for uplift, ugh, my fault :(

Comment 47

[Dianna Smith [:diannaS]]

Comment on attachment 9410727 [details]
Bug 1901769 - Make nsMenuPopupFrame offset handling simpler. r=edgar,NeilDeakin,TYLin

Approved for 128.3esr

Comment 48

[Pulsebot]

https://hg.mozilla.org/releases/mozilla-esr128/rev/2ef9082b5187
https://hg.mozilla.org/releases/mozilla-esr128/rev/a4038768fc4a
https://hg.mozilla.org/releases/mozilla-esr128/rev/afb6393b8035

Comment 49

[Dianna Smith [:diannaS]]

Backed out the last three patches for causing failures on nsMenuPopupFrame.cpp
Push with failures
Failure log
fyi there was a minor conflict with bug 1823552

Comment 50

[Emilio Cobos Álvarez [:emilio]]

Attachment: Bug 1901769 - ESR patch.

Comment 51

[Emilio Cobos Álvarez [:emilio]]

Thanks, rebased it. It was just a matter of removing that line that was failing to build :)

Comment 52

[Phabricator Automation]

esr128 Uplift Approval Request

• User impact if declined: See bug
• Code covered by automated testing: yes
• Fix verified in Nightly: yes
• Needs manual QE test: yes
• Steps to reproduce for manual QE testing: see bug
• Risk associated with taking this patch: see bug
• Explanation of risk level: see bug
• String changes made/needed: none
• Is Android affected?: no

Comment 53

[Dianna Smith [:diannaS]]

Comment on attachment 9410727 [details]
Bug 1901769 - Make nsMenuPopupFrame offset handling simpler. r=edgar,NeilDeakin,TYLin

Approved on rebased patch D221481 instead

Comment 54

[Pulsebot]

https://hg.mozilla.org/releases/mozilla-esr128/rev/80a22fa775b3

Comment 55

[Bogdan Maris, Desktop Test Engineering]

Looks fixed to to me now using latest Firefox 128esr build on Ubuntu 22.04 wayland.