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
Update to Electron 4.0.2 #1172
Update to Electron 4.0.2 #1172
Conversation
4ce05cf
to
94e427b
Compare
Also conflicting with the Focus Mode shortcut (⌘⇧F) here due to a change in the KeyboardEvent. (Now needs an explicit check for shiftKey)
Rendered unnecessary by electron/electron#13992
# Conflicts: # package.json
This reverts commit 90a94e7.
This reverts commit d834229.
8056ea4
to
d91438c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested well for me, I tested on Linux and macOS.
One weird thing happened which I don't think is related to this PR. I got the app into a state where the Focus Mode button wasn't working 🤷♂️
Thanks, I'll keep an eye out for that! |
Closes #744
Fixes tangential issues related to #1171 #1090
The predictable and immediately apparent issues have been addressed. Smoke tested on Windows, Linux, and Mac.
Known issues
The "Checking for Updates" dialog on Linux is showing a menu bar when it shouldn't. This is a known Electron bug. (electron/electron#15901 and/or electron/electron#16521)
TODO
electron-builder
20.33.1 is breaking it. Doesn't seem like a known issue so it might be something on our end, but I reported it to the electron-builder repo. In the meantime I think we can keep using the last working release.