1818330 - Remove the `closemenu="none"` from the App Menu items that don't need it

Status: RESOLVED FIXED

(Thunderbird :: Mail Window Front End, defect)

Description

[Alessandro Castellani [:aleca]]

Some App Menu items like the Settings or Addons and Themes have a closemenu="none" attribute that prevents the app menu from closing on click, and it looks like a UI issue.

We should remove those attributes and ensure that other menu items in the app menu only use that attribute if it make sense.

Comment 1

[Richard Marti (:Paenglab)]

Attachment: Bug 1818330 - Remove the `closemenu="none"` from the App Menu items that don't need it. r=#thunderbird-front-end-reviewers

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/61bde0b85fad
Remove the closemenu="none" from the App Menu items that don't need it. r=aleca

Comment 3

[Richard Marti (:Paenglab)]

Comment on attachment 9319529 [details]
Bug 1818330 - Remove the closemenu="none" from the App Menu items that don't need it. r=#thunderbird-front-end-reviewers

[Approval Request Comment]
User impact if declined: AppMenu not closing
Testing completed (on c-c, etc.): on c-c
Risk to taking this patch (and alternatives if risky): low

Comment 4

[Wayne Mery (:wsmwk)]

Comment on attachment 9319529 [details]
Bug 1818330 - Remove the closemenu="none" from the App Menu items that don't need it. r=#thunderbird-front-end-reviewers

[Triage Comment]
Approved for beta

Comment 5

[Daniel Darnell [:dandarnell]]

Thunderbird 111.0b1:
https://hg.mozilla.org/releases/comm-beta/rev/339c580964c4