1792611 - Box size does not adjust to text wrapping in menulists after flexbox emulation in about:preferences

Status: NEW

(Firefox :: Settings UI, defect, P3)

Description

[Shane Hughes [:aminomancer]]

Attachment: fx-prefs-menulist.png

(See attached image)

In about:preferences the menulist #blockCookiesMenu (Privacy & Security > Enhanced Tracking Protection > Custom) has a static 450px width, but the strings can be longer than that. So given a long string, the label wraps onto a 2nd line, and after bug 1790307 the wrapped text overflows the label box.

regression range

Comment 1

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

Set release status flags based on info from the regressing bug 1790307

Comment 2

[Emilio Cobos Álvarez [:emilio]]

Attachment: This is what I see on Linux fwiw... doesn't it work like that on Windows?

Comment 3

[Emilio Cobos Álvarez [:emilio]]

I'll try to poke at Windows.

Comment 4

[Shane Hughes [:aminomancer]]

Huh, I guess not. I just checked on macOS and it also looks like on Windows

Comment 5

[Sören Hentzschel]

I just checked on macOS and it also looks like on Windows

For me it works on macOS 12. And on Windows 11 via Parallels it works for me as well.

Comment 6

[Shane Hughes [:aminomancer]]

(In reply to Sören Hentzschel from comment #5)

I just checked on macOS and it also looks like on Windows

For me it works on macOS 12. And on Windows 11 via Parallels it works for me as well.

By works, do you mean that you can reproduce the bug, or that everything looks OK? Sorry haha

Comment 7

[Sören Hentzschel]

Sorry for not being clear. I mean that the truncation works and it looks for me like it does in Emilio's screenshot.

Comment 8

[Emilio Cobos Álvarez [:emilio]]

Yeah, this seems fairly low priority given:

• You need to search for some content to hit the bug.
• It's not super-broken.

So this will go away / get automatically fixed once I make nsMenuFrame not use XUL layout. I think we can live with this until then, but lmk if you disagree?

Comment 9

[Shane Hughes [:aminomancer]]

Sure that makes sense.

Comment 10

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

Set release status flags based on info from the regressing bug 1790307

Comment 11

[Jim Mathies [:jimm]]

based on comment 8