-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Use native file and message dialogs on Linux desktop #14683
Comments
^ ignore that "mentioned this issue"--that was a typo, wrong issue. |
Note: The non-native dialogs are needed for automated testing. |
Confirmed the electron fix is in the build we are using, so this work item is ready to go. |
One slight concern, when I tried this on Ubuntu 22, the File Open dialog now has an "Open read-only" checkbox; we need to see if we can suppress this as it may be confusing for users. I opened electron/electron#42349 against Electron. |
Moving to next release to see if this gets resolved on the Electron side. |
The long-standing issue of Electron showing the file and message dialogs behind the main window has been fixed and should appear in an upcoming Electron release.
electron/electron#32857
We worked around this bug on Linux by adding an option to use the web-based dialogs (same as RStudio Server) and turned it on by default only on Linux: #11184
Once we've confirmed the fix is in the version of Electron we're using, we should consider changing the default for Linux (and possibly discuss if we should ditch this option to use web-based dialogs on Desktop altogether to simplify the testing matrix).
The text was updated successfully, but these errors were encountered: