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 7 #1454
Update to Electron 7 #1454
Conversation
c121f80
to
d16bb83
Compare
This was straight forward (and works on Linux). |
Needs double check on MacOS and Windows. |
I cannot export images on Mac. Could you verify it works on Linux? |
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.
I think there's a change in return values of dialogs with this electron version. Image export/Save file as etc. kinda stuff is not working. I tried to debug the image export thing a bit. It seems like awaiting showSaveFileDialog in App.js is not working now.
I can reproduce it on Linux, too. Will look into it and report back on my findings. |
Broken in Electron@7 / Chrome 78.
Fixed by updating our dialog. The Electron Dialog is promisified now. |
Works on Mac 👍 |
@philippfromme Could you give this one a look and verify it works on Windows ©️ ®️ 💥, too? |
What are your exact steps to reproduce the errors? |
|
Did you try a clean install |
I had to update The issue regarding SVGs in development mode is not related to the Electron update. |
Thanks for clarifying. ❤️ |
Works on
Closes #1448