1728591 - Port bug 1728368: Findbar close button escapes findbar-close-container on macOS

Status: RESOLVED FIXED

(Thunderbird :: Upstream Synchronization, task)

Comment 1

[Richard Marti (:Paenglab)]

Attachment: 1728591-simplify-findbar.patch

Port of https://hg.mozilla.org/mozilla-central/rev/f6e3c384468a

Comment 2

[Alessandro Castellani [:aleca]]

Comment on attachment 9238936 [details] [diff] [review]
1728591-simplify-findbar.patch

Review of attachment 9238936 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good, thanks

Comment 3

[Richard Marti (:Paenglab)]

Bug 1728368 is to be to go to beta and ESR.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/8fa88ac9a22c
Port bug 1728368: Simplify findbar overflow behavior. r=aleca

Comment 5

[Richard Marti (:Paenglab)]

Comment on attachment 9238936 [details] [diff] [review]
1728591-simplify-findbar.patch

The M-C bug 1728368 has now ESR91 approval
[Approval Request Comment]
User impact if declined: overflowing findbar
Testing completed (on c-c, etc.): on c-c
Risk to taking this patch (and alternatives if risky): should be low as it's already some time on c-c

Comment 6

[Wayne Mery (:wsmwk)]

Comment on attachment 9238936 [details] [diff] [review]
1728591-simplify-findbar.patch

[Triage Comment]
Approved for esr91

Comment 7

[Rob Lemley [:rjl]]

https://hg.mozilla.org/releases/comm-esr91/rev/a52cba6704ac