-
Notifications
You must be signed in to change notification settings - Fork 2.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Theia Electron application menu fails on macOS #1875
Comments
I verified this; it's broken since updating to v1.8.x of electron. maybe there are some api changes, and we need to catch up. BTW in the meantime electron v2 was released. |
Ok, thanks for confirming. Are there any plans yet to move the desktop version of Theia to Electron v2 anytime soon? |
Quick question as I am doing the review of the corresponding PR; it is not the context menu but the application menu, right? |
My understanding is:
So this is about the application menu. Please see the above screenshot for that. |
Closed via #2469. |
Theia Electron context menu fails on macOS. There should for instance be a View menu with entries such as Call hierarchy, Extensions, Files, Git, Git History, Outline, Output, ... But non of them are visible when run on macOS. Please see macOS screenshot.
Ran following command on macOS High Sierra 10.13.4 with node v8.11.1, npm 6.0.0, yarn 1.6.0:
When trying to do the same on Ubuntu 16.04.4 LTS with node v8.11.1, npm 6.0.0, yarn 1.6.0 the script above fails when running yarn. So, yarn has to be run twice. But the end result works. The menu shows as expected. Please see Ubuntu screenshot.
The text was updated successfully, but these errors were encountered: