-
Notifications
You must be signed in to change notification settings - Fork 190
fix: splashscreen fullscreen & ui tweaks #1691
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #1691 +/- ##
========================================
Coverage 63.87% 63.87%
========================================
Files 139 139
Lines 3815 3815
Branches 797 797
========================================
Hits 2437 2437
Misses 1152 1152
Partials 226 226 Continue to review full report at Codecov.
|
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.
Looks good apart from the comment
if (process.platform !== 'darwin') { | ||
app.quit() | ||
} | ||
app.quit() |
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.
🤔 might want to discuss this to be sure if this is what we want to have happen on mac
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.
Me and Sam discussed it :colbert: it fixes a bug, plugins can't be installed after: closing all windows (not quitting the app) then opening the window again and trying to install a plugin
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.
Has anyone looked into that yet? :hmhm:
Summary
Checklist