Skip to content
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

Bugfixes and API changes for updated Electron #309

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mosullivan93
Copy link
Contributor

@mosullivan93 mosullivan93 commented May 18, 2023

The API for performing dialog operations has changed significantly since the last version of Electron used in the package. Specifically, this PR makes the necessary backend changes to remedy issues with the dialog widgets seen with the updated Electron from Blink >=v0.12.6. This PR may require additional thought about an update to Project.toml to ensure that the mutual dependencies on the patches will be satisfied by users of both packages.

This patch provides the following improvements:

  1. A more robust override mechanism for BrowserWindow defaults.
  2. Compatibility with or without contextIsolation for asynchronous dialog operations.
  3. A user friendly mechanism for adding renderer preload scripts to extend the namespace of isolated windows.
  4. Miscellaneous bugfixes: Julia icon path, keys in window_defaults dictionary, syntax update for invoking debugger.
Blink.jl InteractBase.jl Status
Blink v0.12.5 (Electron 4.04) InteractBase v0.10.5 Working
Blink v0.12.5 (Electron 4.04) InteractBase v0.10.9 Broken (theme undefined)
Blink v0.12.5 (Electron 4.04) InteractBase #177 Working
Blink v0.12.5 (Electron 4.04) InteractBase #178 Broken (no dialog)
Blink.jl InteractBase.jl Status
Blink v0.12.6 (Electron 19.09) InteractBase v0.10.9 Broken (theme undefined)
Blink v0.12.7 (Electron 19.09) InteractBase v0.10.9 Broken (theme undefined)
Blink v0.12.7 (Electron 19.09) InteractBase #177 Broken (no dialog)
Blink #309 (Electron 19.09) InteractBase #177 Broken (no dialog)
Blink #309 (Electron 19.09) InteractBase #178 Working

The API for performing dialog operations has changed significantly since
the last working version of Electron. This patch provides the following
improvements:
1) A more robust override mechanism for BrowserWindow defaults.
2) Compatibility with or without contextIsolation for asynchronous
   dialog operations.
3) A user friendly mechanism for adding renderer preload scripts to
   extend the namespace of isolated windows.
4) Miscellaneous bugfixes: Julia icon path, keys in window_defaults
   dictionary, syntax update for invoking debugger.
@mosullivan93
Copy link
Contributor Author

Converting to draft. I would like to do a few more tests on the new features and add documentation for them.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant