mintrayr icon indicating copy to clipboard operation
mintrayr copied to clipboard

Fix minimize-on-close for TB 52 on Linux

Open rsjtdrjgfuzkfg opened this issue 7 years ago • 21 comments

This PR attempts to fix #178, #175 and #172 – but is untested on Windows and old versions of Thunderbird. I'd welcome some feedback on that, as I don't have the time to do proper testing right now.

I consider it very likely that we don't need to observe titlebar-close anymore after merging this. It does not call goQuitApplication but window.close – at least in the versions of TB I looked at, but I'm not comfortable doing the removal without any tests on Windows.

Also, the maintainer might want to use a different version number after merging; I merely added something in order to distinguish built xpis from official releases.

rsjtdrjgfuzkfg avatar Jul 26 '17 17:07 rsjtdrjgfuzkfg

Hi, Thanks for providing the fix. Can you share built file for extension, to try testing it?

undera avatar Aug 03 '17 10:08 undera

@undera I don't have a full mozilla build environment set up, so my builds emit localization warnings and most likely cause various issues when run on a locale whose translation is not complete.

If still want a build: mintrayr-1.3.2+tb52fix.xpi.zip (unzip once to get the XPI to install, GitHub does not permit XPI attachments).

rsjtdrjgfuzkfg avatar Aug 03 '17 14:08 rsjtdrjgfuzkfg

Just as a tip - you can create a release on GitHub from your repo and it will allow XPI to be attached there. The original repo seems to be abandoned, I guess many people would enjoy your improvements.

undera avatar Aug 03 '17 14:08 undera

I installed it, but it does not help. My issue is that I see no tray icon at all. I see in processes that thunderbird is still running, so the "hiding" part works fine, just no tray icon appears.

undera avatar Aug 03 '17 14:08 undera

@undera did it work with the previous version (when minimizing to tray through the menu)? I did not change anything in the actual minimization code, only fixed the trigger for Thunderbird.

What OS are you on? Version of Thunderbird? Are there messages in the error console?

Edit: Oh, and just in case: what locale are you on?

rsjtdrjgfuzkfg avatar Aug 03 '17 14:08 rsjtdrjgfuzkfg

Just checked with the stock version - no tray icon.

Andrey Pokhilko

On 08/03/2017 05:38 PM, rsjtdrjgfuzkfg wrote:

@undera https://github.com/undera did it work with the previous version (when minimizing to tray through the menu)? I did not change anything in the actual minimization code, only fixed the trigger for Thunderbird.

What OS are you on? Version of Thunderbird? Are there messages in the error console?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/nmaier/mintrayr/pull/179#issuecomment-319988499, or mute the thread https://github.com/notifications/unsubscribe-auth/AB7r5YrjD8nx8MMVJCEpq7hZ1yk9U9UOks5sUdtugaJpZM4OkOTh.

undera avatar Aug 03 '17 16:08 undera

@rsjtdrjgfuzkfg your fixed build indeed works for me. I'm on Ubuntu with Unity where there is a permanent tray icon present even without the extension, so not facing the same problem as @undera.

dcervenkov avatar Aug 07 '17 08:08 dcervenkov

Works for me, using gentoo and thunderbird 52.2.0. Thank you very much!

Gotos avatar Aug 07 '17 10:08 Gotos

Doesn't work on thunderbird 52.2.1, gnome 3.24.3, fedora 26.

dmnrmr avatar Aug 08 '17 07:08 dmnrmr

Works for me, using Debian 9.1 (x86-64) and Thunderbird 52.2.1 (64-bits), Cinnamon 3.2.7. Thank you very much!

endrigo avatar Aug 16 '17 14:08 endrigo

Works for me, thank you very much! Ubuntu 16.04.3, GNOME Shell 3.18.5, Thunderbird 52.2.1.

sebadorn avatar Aug 18 '17 17:08 sebadorn

@nmaier What's the status on this?

qwertychouskie avatar Aug 27 '17 01:08 qwertychouskie

Works for me on arch with GNOME 3.24.2 and TB 52.3.0. Great job @rsjtdrjgfuzkfg 👍

fkopietz avatar Sep 05 '17 20:09 fkopietz

Works with FC 26, TB 52.3.0

mckatzenelson avatar Sep 08 '17 15:09 mckatzenelson

This fixed the closing to tray issue, but now is not minimizing to tray.

Linux Mint 18.1 Cinnamon 64 bits Thunderbird 52.3.0

grottas avatar Sep 25 '17 14:09 grottas

Fixed, XFCE on Ubuntu 16.04

Brianetta avatar Sep 28 '17 22:09 Brianetta

It's working for me - both close and minimise cause Thunderbird to sit in the tray!

Thunderbird 52.4.0 (64-bit) XFCE 4.12 in Arch Linux

Thanks for tracking this down and submitting a fix!

bhrgunatha avatar Oct 25 '17 10:10 bhrgunatha

It mostly works for me (52.4.0 64-bit, Linux Mint 17.3).

Notes: It still minimizes to taskbar if I click the minimize button, and I have to enable "always show tray icon" otherwise it vanishes when I close. But if I enable that, then it stays in the tray only.

Edit: I was also running "Minimize on start and close" and this was interfering, I think it was overriding mintrayr's minimize to tray on close, so it was just minimizing (to taskbar) instead of closing. When I disabled start-and-close, then mintrayr minimizes to tray on close button, without having to have it always showing. Interesting that with "always show tray" turned OFF and "minimize on start and close" turned ON, the close button would appear to completely close the app, as if ignoring both add-ons. I wonder if each add-on is using some kind of toggle to change a value so they're cancelling each other out? Neither extension seems able to properly minimise to tray with the minimize button.

schrodingers-ape avatar Oct 28 '17 17:10 schrodingers-ape

The fix it's working for me too, both closing and minimizing to tray.

Thunderbird 52.5.0 (64-bit) Linux mint 18.3 mate

mattmart3 avatar Jan 01 '18 15:01 mattmart3

It seems to be fixed.

On Jan 1, 2018 10:20 AM, "Matteo Martelli" [email protected] wrote:

The fix it's working for me too, both closing and minimizing to tray.

Thunderbird 52.5.0 (64-bit) Linux mint 18.3 mate

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/nmaier/mintrayr/pull/179#issuecomment-354658475, or mute the thread https://github.com/notifications/unsubscribe-auth/AF2tvuQ3a0JJOdNE-iqcrkjrcUSsXB7eks5tGPefgaJpZM4OkOTh .

mckatzenelson avatar Jan 02 '18 01:01 mckatzenelson

Aqui também esta funcionando Ubuntu 17.10 Thunderbird 52

Obrigado

Danielti2011 avatar Feb 20 '18 23:02 Danielti2011