Unhandled Promise Rejection TypeError: Cannot read property 'isVisible' #4085

rob4226 · 2020-03-26T02:36:21Z

Hi,
My signal desktop app on Windows 10 just auto-updated to the next latest release version v1.32.2 but after asking me to restart it crashed and I got the below error. Now whenever I try to launch it with my shortcut that uses the "-use-tray-icon --start-in-tray flags" I get the same error that pops up with an option to quit or copy & quit. The error it copies is:

Unhandled Promise Rejection

TypeError: Cannot read property 'isVisible' of undefined
at Tray.tray.updateContextMenu ([REDACTED]\app.asar\app\tray_icon.js:71:36)
at createTrayIcon ([REDACTED]\app.asar\app\tray_icon.js:111:8)
at App. ([REDACTED]\app.asar\main.js:884:12)

My desktop shortcut I use is "C:\Users\R-MAIN\AppData\Local\Programs\signal-desktop\Signal.exe --use-tray-icon --start-in-tray"

I can launch it without those flags but not with them. This happened immdiately after asking me to restart to auto-update to v1.32.2

Any thoughts? Thank you.

The text was updated successfully, but these errors were encountered:

yz778 · 2020-03-26T03:18:23Z

Encountered same problem. As long as one of --use-tray-icon or --start-in-tray is passed as an argument, the error shows up.

Unfortunately no obvious way to rollback to prev version...

fisixer · 2020-03-26T03:44:50Z

ditto. i am also calling by "Signal.exe --use-tray-icon --start-in-tray" in a fully patched Win10 machine.

note that this error/bug was in the beta (the issue is listed a few items below this one.

morrow95 · 2020-03-26T05:56:28Z

Same here - updated earlier tonight and having this error when using these options as well.

morten-b · 2020-03-26T06:31:14Z

I'm having the same error on Ubuntu 19.10 / Regolith 1.3.1

lhzw · 2020-03-26T06:40:54Z

I hope there's an option for us to control when to update, much better if we can specify the version.
I always think the latest version is not that stable, I'd like the previous one.

ghost · 2020-03-26T06:51:56Z

Same here as well with '--start-in-tray' , Xubuntu 19.10.

Is there usually any testing done on a new Signal version? I started to get reluctant to update, confirmed by this issue now, having the impression that too frequently some thing stops to work correctly with a new version (and despite many updates, still no audio/video, which would be so useful these days).

scma-esrich · 2020-03-26T07:44:04Z

+1

Windows 10

natrius · 2020-03-26T07:46:14Z

Same here Win10, using --start-in-tray

EDIT: Works when removing the line. Annoying.

@Sirius-C You might try matrix or better the client riot.im there you have video /audio everywhere. But its another messenger.

ax4 · 2020-03-26T07:56:38Z

Same here. Trouble with either --use-tray-icon or --start-in-tray flag enabled.

Unhandled Promise Rejection

TypeError: Cannot read property 'isVisible' of undefined
    at Tray.tray.updateContextMenu ([REDACTED]\app.asar\app\tray_icon.js:71:36)
    at createTrayIcon ([REDACTED]\app.asar\app\tray_icon.js:111:8)
    at App.<anonymous> ([REDACTED]\app.asar\main.js:884:12)

I guess this might caused by commit 750e508 at 8 hours ago, on March 26, 2020. And the latest release of v1.32.2 at 7 hours ago (https://github.com/signalapp/Signal-Desktop/releases/tag/v1.32.2) included this change to the tray_icon.js file (see difference at 750e508#diff-1e93a15ff1db4bd6b0fce52ca01e7161).

EDIT:
See @ElderThorin 's reply and his correction. Thanks! Appreciate the correction!

thevops · 2020-03-26T08:01:45Z

same here in Linux mint 19

mhantsch · 2020-03-26T08:30:41Z

Same problem here (Windows 10). The use of the '--start-in-tray' commandline option triggers this problem.

R4PS · 2020-03-26T09:01:12Z

Same problem here, KDE neon 5.18

mandree · 2020-03-26T09:05:48Z

People, stop posting "me too"!!

scma-esrich · 2020-03-26T09:11:18Z

People, stop posting "me too"!!

This should give more info about the systems and the amount of users affected by this issue.
Or do I confuse things?

mandree · 2020-03-26T09:15:09Z

People, stop posting "me too"!!

This should give more info about the systems and the amount of users affected by this issue.
Or do I confuse things?

I think it is pretty clear that this affects all systems. No need to spam the issue with noise.

mandree · 2020-03-26T09:16:11Z

I have bisected this in Git on Fedora Linux 31.

$ git bisect bad
52e594901f6f30269e7bbe45b94b68eb9f7419f7 is the first bad commit
commit 52e594901f6f30269e7bbe45b94b68eb9f7419f7
Author: Ken Powers <[email protected]>
Date:   Wed Mar 25 20:32:20 2020 -0400

    Use Electron Spellchecker

 _locales/en/messages.json    |  16 ++++
 app/spell_check.js           | 102 +++++++++++++++++++++++++
 js/background.js             |  14 ----
 js/spell_check.js            | 172 -------------------------------------------
 js/views/settings_view.js    |  21 +++++-
 main.js                      |  25 ++++++-
 package.json                 |   3 -
 patches/typo-js+1.1.0.patch  | Bin 553 -> 0 bytes
 preload.js                   |  24 +++---
 settings.html                |   3 +
 settings_preload.js          |   2 +
 test/index.html              |   1 -
 test/spellcheck_test.js      |  15 ----
 ts/util/lint/exceptions.json |  56 ++++++--------
 yarn.lock                    | 103 --------------------------
 15 files changed, 199 insertions(+), 358 deletions(-)
 create mode 100644 app/spell_check.js
 delete mode 100644 js/spell_check.js
 delete mode 100644 patches/typo-js+1.1.0.patch
 delete mode 100644 test/spellcheck_test.js

scma-esrich · 2020-03-26T09:17:52Z

Okay, understood. From: mandree <[email protected]> Sent: Donnerstag, 26. März 2020 10:16 To: signalapp/Signal-Desktop <[email protected]> Cc: Markus Schlager <[email protected]>; Comment <[email protected]> Subject: Re: [signalapp/Signal-Desktop] Unhandled Promise Rejection TypeError: Cannot read property 'isVisible' (#4085) I have bisected this in Git $ git bisect bad 52e5949<52e5949> is the first bad commit commit 52e5949<52e5949> Author: Ken Powers [email protected]<mailto:[email protected]> Date: Wed Mar 25 20:32:20 2020 -0400 Use Electron Spellchecker _locales/en/messages.json | 16 ++++ app/spell_check.js | 102 +++++++++++++++++++++++++ js/background.js | 14 ---- js/spell_check.js | 172 ------------------------------------------- js/views/settings_view.js | 21 +++++- main.js | 25 ++++++- package.json | 3 - patches/typo-js+1.1.0.patch | Bin 553 -> 0 bytes preload.js | 24 +++--- settings.html | 3 + settings_preload.js | 2 + test/index.html | 1 - test/spellcheck_test.js | 15 ---- ts/util/lint/exceptions.json | 56 ++++++-------- yarn.lock | 103 -------------------------- 15 files changed, 199 insertions(+), 358 deletions(-) create mode 100644 app/spell_check.js delete mode 100644 js/spell_check.js delete mode 100644 patches/typo-js+1.1.0.patch delete mode 100644 test/spellcheck_test.js — You are receiving this because you commented. Reply to this email directly, view it on GitHub<#4085 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AK52JD7IIMHUDP643PLAYP3RJMMOVANCNFSM4LT42XVQ>.

Lion-box · 2020-03-26T12:56:19Z

Confirm the same error on Ubuntu-Mate 19.10
removing --use-tray-icon from command line does work, but feels like quite a ridiculous workaround

ElderThorin · 2020-03-26T13:30:14Z

Same here. Trouble with either --use-tray-icon or --start-in-tray flag enabled.
Unhandled Promise Rejection

TypeError: Cannot read property 'isVisible' of undefined
    at Tray.tray.updateContextMenu ([REDACTED]\app.asar\app\tray_icon.js:71:36)
    at createTrayIcon ([REDACTED]\app.asar\app\tray_icon.js:111:8)
    at App.<anonymous> ([REDACTED]\app.asar\main.js:884:12)
I guess this might caused by commit 750e508 at 8 hours ago, on March 26, 2020. And the latest release of v1.32.2 at 7 hours ago (https://github.com/signalapp/Signal-Desktop/releases/tag/v1.32.2) included this change to the tray_icon.js file (see difference at 750e508#diff-1e93a15ff1db4bd6b0fce52ca01e7161).

I believe that change is on the dev branch and actually will fix this issue. Don't use github much, could be misreading.

ax4 · 2020-03-26T14:05:41Z

I believe that change is on the dev branch and actually will fix this issue. Don't use github much, could be misreading.

Yes, I agree with @ElderThorin 's correction! After I re-read the release page of v1.32.2 and v1.33.0-beta.4 , I should make correction to my previous comment!

So, in Releases it says

v1.32.2 @scottnonnenberg-signal scottnonnenberg-signal released this 13 hours ago · 33 commits to development since this release
v1.33.0-beta.4
@scottnonnenberg-signal scottnonnenberg-signal released this 14 hours ago
Fixes an issue where Signal Desktop might crash on startup if --use-tray-icon or --start-in-tray was provided
Fixes an issue where Signal Desktop was unable to react to messages in a group

This means, there're currently 33 new commits in the development branch. One of the commits should be able to fix (or maybe not fix). And the label v1.33.0-beta.4 will be working on to fix this tray icon issue soon!

See @mandree is working on commit https://github.com/mandree/Signal-Desktop/commit/c2294ea8aedafe1a802eca382703456b3e20313a and PR #4090 to try to fix the issue. Thanks ElderThorin for correction. And thanks mandree for working on the solution!

lhzw · 2020-03-26T14:15:22Z

I think a backup before updating is a good way.
It's too scary and inconvenient when it crashes on start.

mandree · 2020-03-26T14:21:19Z

@ax4 For a released branch I was going for the minimum workable change. Let someone else sort out the betas.

mandree · 2020-03-26T14:38:47Z

Sorry for the update storm - this was minimizing the fix, the minimum result is adding await before createWindow();

mandree · 2020-03-26T18:34:58Z

Maintainers, feel free to clone this comment into a new issue and mention me.

Note that - from looking at the code - apparently also the development branch as of 750e508 and checking 9c3196a again is apparently fixing the wrong place.

While it fixes the immediate broken Promise and the resulting crash, the actual issue to me appears that the createWindow() function was very recently made async with the exchange of the spell checker, but the invocation in the app.on('ready', ...) handler in main.js (and there is one more macOS-specific handler which I know nothing about) does not await the completion of createWindow() before it tries to add the tray icon - this is what appears to be causing the createTrayIcon() to fail in the first place: the window hasn't been created and cannot be referenced.

So I propose to also cherry-pick 569fa1f from my fork to development also, and possibly revert the addition of mainWindow && to app/tray_icon.js as of 750e508. I'll add that to my fork for your convenience.

mandree · 2020-03-26T18:42:05Z

So I've stripped the master branch in my fork down to just the "await" change, removing the node version change and the version bump. The two commits immediately referenced above are the changes for the v1.33-beta development (9ad42fe) and v1.32.x master branch (6251b71), respectively.

Between v1.32.1 and v1.32.2, createMainWindow() was made an async function, so createTrayIcon() and setupMenu() were called too early. await createMainWindow() before adding the tray icon and menu. Fixes #4085. Co-authored-by: Matthias Andree <[email protected]>

scottnonnenberg-signal · 2020-03-26T18:51:50Z

All, do not download Signal from any location other than https://signal.org/download. I've deleted the previous comment.

rob4226 · 2020-03-26T19:31:02Z

My signal desktop app on Win10 just auto-updated to v1.32.3 and now the "--use-tray-icon --start-in-tray" flags work again! Thanks for the quick fix!

This issue should now be resolved with 1.32.3 signalapp/Signal-Desktop#4085

josh-signal · 2020-03-26T19:36:45Z

Please install v1.32.3 folks.

morten-b · 2020-03-26T20:41:03Z

After updating to 1.32.3 I no longer see any errors, but no icon is showing either.

Debug log: https://paste.ubuntu.com/p/2mqJF83MPp/

mandree · 2020-03-26T20:53:31Z

@morten-b What operating system (with version) and what desktop environment are you using?

Lion-box · 2020-03-26T20:56:45Z

v1.32.3 working as expected here on Ubuntu-Mate 19.10 including tray icon

mandree · 2020-03-26T22:04:50Z

For missing icon, see #3974 and #4082.

CryptoD · 2020-03-26T22:42:25Z

Version 1.32.3 is working fine again on Kubuntu 19.10

lhzw · 2020-03-27T06:44:00Z

Great job, very well done, fine on win7.

natrius · 2020-03-27T07:04:02Z

Win10Enterprise, working, symbols seems fine, also after locking and unlocking the pc. Thanks!

itkfm · 2020-03-27T07:11:20Z

Is there usually any testing done on a new Signal version?

Unfortunately, the tray icon is still considered Beta and a hidden feature… Maybe used by too few people.

People, stop posting "me too"!!

This should give more info about the systems and the amount of users affected by this issue.

It's probably better to upvote previous comments if their poster is on the same platform.

scma-esrich · 2020-03-27T07:12:30Z

It's probably better to upvote previous comments if their poster is on the same platform.

Will do in the future.

morten-b · 2020-03-27T07:14:45Z

I'm om Ubuntu 19.10 with Regolith 1.3.1 which is baser on i3. Nextcloud tray icon i showing as expected.

…

-------- Original Message --------

On 26 Mar 2020, 21:53, mandree wrote: ***@***.***(https://github.com/morten-b) What operating system (with version) and what desktop environment are you using? — You are receiving this because you were mentioned. Reply to this email directly, [view it on GitHub](#4085 (comment)), or [unsubscribe](https://github.com/notifications/unsubscribe-auth/AHUMUXSJXEFT3HWGBTZWUUDRJO6FXANCNFSM4LT42XVQ).

mhantsch · 2020-03-27T10:04:51Z

Windows 10, Signal v1.32.3: --start-in-tray option works again.

mandree · 2020-03-27T10:35:56Z

@morten-b when you Google, you'll find that Electron 8 and i3 appear to be incompatible WRT tray icon support. Different worlds. Electron 8 talks new world, i3 understands old world. Sorry, I didn't save the reference. Note that your situation has nothing to do with the bug that was reported here, and does not belong into this issue.

Reports since 1.32.3 have been about incompatibility between Electron 8 and the Linux Desktop Environment's respective app/tray icon handling and belong either into existing, or possibly new, issues, and possibly not even with Signal-Desktop, and it appears the madness of the various trayicon/appindicator/kstatusnotifier schemes in the fragmented Linux desktop universe continues.

morten-b · 2020-03-27T11:20:17Z

Excellent feedback! Out of curiosity, when was this issue introduced? I have had the icon until recently.

…

-------- Original Message --------

On 27 Mar 2020, 11:36, mandree wrote: ***@***.***(https://github.com/morten-b) when you Google, you'll find that Electron 8 and i3 appear to be incompatible WRT tray icon support. Different worlds. Electron 8 talks new world, i3 understands old world. Sorry, I didn't save the reference. Note that your situation has nothing to do with the bug that was reported here, and does not belong into this issue. Reports since 1.32.3 have been about incompatibility between Electron 8 and the Linux Desktop Environment's respective app/tray icon handling and belong either into existing, or possibly new, issues, and possibly not even with Signal-Desktop, and it appears the madness of the various trayicon/appindicator/kstatusnotifier schemes in the fragmented Linux desktop universe continues. — You are receiving this because you were mentioned. Reply to this email directly, [view it on GitHub](#4085 (comment)), or [unsubscribe](https://github.com/notifications/unsubscribe-auth/AHUMUXVHPF7Q43FBCTROBYDRJR6R3ANCNFSM4LT42XVQ).

ax4 referenced this issue Mar 26, 2020

Introduce dialog for long database migrations

750e508

lhzw mentioned this issue Mar 26, 2020

Error after an update #4087

Closed

1 task

ghost mentioned this issue Mar 26, 2020

--use-tray-icon --start-in-tray doesnt work anymore #4089

Closed

kenpowers-signal added the Bug label Mar 26, 2020

mandree mentioned this issue Mar 26, 2020

[master] Fix TypeError: Cannot read property 'isVisible' of undefined in Tray.tray.updateContextMenu #4090

Closed

7 tasks

mandree mentioned this issue Mar 26, 2020

Fix --use-tray-icon and menu. #4092

Closed

7 tasks

josh-signal mentioned this issue Mar 26, 2020

Tray icon stopped working after the latest update #4096

Closed

1 task

signalapp deleted a comment from lburtts Mar 26, 2020

scottnonnenberg-signal mentioned this issue Mar 26, 2020

--start-in-tray --use-tray-icon is not working anymore #3613

Closed

mandree mentioned this issue Mar 26, 2020

Fix #4085 on development, too. #4097

Closed

scottnonnenberg-signal mentioned this issue Mar 26, 2020

Tray icon goes away after locking/unlocking the screen #4082

Open

1 task

popey pushed a commit to snapcrafters/signal-desktop that referenced this issue Mar 26, 2020

Update snapcraft.yaml

c62dc50

This issue should now be resolved with 1.32.3 signalapp/Signal-Desktop#4085

josh-signal added the Should Be Fixed label Mar 26, 2020

agottardo mentioned this issue Mar 26, 2020

Signal is requesting a restart due to update, but it fails to restart. When restarting manually, the update hasn't come into effect. #4091

Closed

1 task

This was referenced Mar 27, 2020

Feature request: close/minimize to system tray #2210

Closed

Signal.exe --start-in-tray option does not carry across updates #2368

Closed

josh-signal closed this as completed Mar 27, 2020

Unhandled Promise Rejection TypeError: Cannot read property 'isVisible' #4085

Unhandled Promise Rejection TypeError: Cannot read property 'isVisible' #4085

Comments

rob4226 commented Mar 26, 2020 • edited Loading

yz778 commented Mar 26, 2020

fisixer commented Mar 26, 2020

morrow95 commented Mar 26, 2020

morten-b commented Mar 26, 2020

lhzw commented Mar 26, 2020

ghost commented Mar 26, 2020

scma-esrich commented Mar 26, 2020

natrius commented Mar 26, 2020 • edited Loading

ax4 commented Mar 26, 2020 • edited Loading

thevops commented Mar 26, 2020

mhantsch commented Mar 26, 2020 • edited Loading

R4PS commented Mar 26, 2020

mandree commented Mar 26, 2020

scma-esrich commented Mar 26, 2020

mandree commented Mar 26, 2020

mandree commented Mar 26, 2020 • edited Loading

scma-esrich commented Mar 26, 2020 via email

Lion-box commented Mar 26, 2020

ElderThorin commented Mar 26, 2020 • edited Loading

ax4 commented Mar 26, 2020 • edited Loading

lhzw commented Mar 26, 2020

mandree commented Mar 26, 2020

mandree commented Mar 26, 2020

mandree commented Mar 26, 2020

mandree commented Mar 26, 2020

scottnonnenberg-signal commented Mar 26, 2020

rob4226 commented Mar 26, 2020

josh-signal commented Mar 26, 2020

morten-b commented Mar 26, 2020

mandree commented Mar 26, 2020

Lion-box commented Mar 26, 2020

mandree commented Mar 26, 2020 • edited Loading

CryptoD commented Mar 26, 2020

lhzw commented Mar 27, 2020

natrius commented Mar 27, 2020

itkfm commented Mar 27, 2020

scma-esrich commented Mar 27, 2020 • edited Loading

morten-b commented Mar 27, 2020 via email

mhantsch commented Mar 27, 2020

mandree commented Mar 27, 2020

morten-b commented Mar 27, 2020 via email

rob4226 commented Mar 26, 2020 •

edited

Loading

natrius commented Mar 26, 2020 •

edited

Loading

ax4 commented Mar 26, 2020 •

edited

Loading

mhantsch commented Mar 26, 2020 •

edited

Loading

mandree commented Mar 26, 2020 •

edited

Loading

ElderThorin commented Mar 26, 2020 •

edited

Loading

ax4 commented Mar 26, 2020 •

edited

Loading

mandree commented Mar 26, 2020 •

edited

Loading

scma-esrich commented Mar 27, 2020 •

edited

Loading