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
fix: defaults broken in shell.openExternal() options #38038
Conversation
Release Notes Persisted
|
I was unable to backport this PR to "23-x-y" cleanly; |
I was unable to backport this PR to "22-x-y" cleanly; |
I have automatically backported this PR to "25-x-y", please check out #38071 |
I have automatically backported this PR to "24-x-y", please check out #38072 |
Co-authored-by: Milan Burda <miburda@microsoft.com>
Co-authored-by: Milan Burda <miburda@microsoft.com>
fix: defaults broken in shell.openExternal() options (#38038) Co-authored-by: Milan Burda <miburda@microsoft.com>
fix: defaults broken in shell.openExternal() options (#38038) Co-authored-by: Milan Burda <miburda@microsoft.com>
Description of Change
When calling
shell.openExternal('https://example.com/');
it parsesactivate
astrue
correctly.However calling
shell.openExternal('https://example.com/', {});
would parseactivate
asfalse
.Related to:
electron/shell/common/gin_helper/dictionary.h
Lines 34 to 50 in 0240f66
Checklist
npm test
passesRelease Notes
Notes: Fixed broken defaults in
shell.openExternal()
options.