-
Notifications
You must be signed in to change notification settings - Fork 218
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
Adding window state keeper #1
Conversation
8b3f659
to
3683e79
Compare
@@ -193,7 +193,6 @@ | |||
"i18next-electron-language-detector": "^0.0.10", | |||
"i18next-sprintf-postprocessor": "^0.2.2", | |||
"i18next-sync-fs-backend": "^1.0.0", | |||
"npm": "^5.7.1", |
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.
Removed this since it was causing a post-install script to fail.
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.
Yeah - strange. It's still working though!
Storing window attributes and using them on electron app startup.
3683e79
to
32ecb09
Compare
@@ -97,18 +97,11 @@ JSONStream@^0.8.4: | |||
jsonparse "0.0.5" | |||
through ">=2.2.7 <3" | |||
|
|||
JSONStream@^1.3.2: |
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.
Not sure why all of these dependencies got changed when I ran 'yarn install', but can you try this setup and see if it works for you?
Looks good - works well, going to merge it in, then deal with merging my last couple days work in. |
Storing window attributes and using those on app startup.