-
Notifications
You must be signed in to change notification settings - Fork 16.8k
refactor: use //ui/shell_dialogs on Linux
#42045
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
Conversation
|
@codebytere also noticed https://issues.chromium.org/issues/41469294#comment10, which says that upstream likely wants to support GtkFileChooserNative on GTK4 |
ckerr
left a comment
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.
Great idea, I love it. Especially with upstream indicating they will support the native dialogs in GTK4.
I have some implementation suggestions here
patches/chromium/feat_add_support_for_missing_dialog_features_to_shell_dialogs.patch
Outdated
Show resolved
Hide resolved
patches/chromium/feat_add_support_for_missing_dialog_features_to_shell_dialogs.patch
Outdated
Show resolved
Hide resolved
patches/chromium/feat_add_support_for_missing_dialog_features_to_shell_dialogs.patch
Outdated
Show resolved
Hide resolved
Co-authored-by: Charles Kerr <[email protected]>
a5020e0 to
09f223d
Compare
ckerr
left a comment
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.
LGTM.
|
I have automatically backported this PR to "31-x-y", please check out #42109 |
|
I have automatically backported this PR to "30-x-y", please check out #42110 |
|
@codebytere has manually backported this PR to "29-x-y", please check out #42144 |
|
If this is the change where you search for command line utilities to use as your file-chooser, like Any fileChooser scheme should not depend on how the See the bug linked by @daiyam above Linux - vscodium no longer displayes the file-dialog after update to Ver. 1.9.0. This affects more than just TDE/KDE3 as noted by the comments in that bug report. It also effects vscode/vscodium identically. Several work-arounds (ugly hacks) are included in the linked report that either involve intercepting the call to the command-line utility and rewriting the command or to alter the It may be convenient to simply search the Linux Lastly, there needs to be a clear way to disable this new behavior and restore the default Gtk dialogs on desktop that this breaks fileChooser display on. Mozilla provides Hopefully the behavior of this commit can be made more robust to prevent the dialog failures it causes. Absent that a clear option to disable it and restore default dialogs would be greatly appreciated. |
Description of Change
Closes #32857
Closes #41903
Refs #19159
This PR replaces our dialog implementation on Linux with upstream
//shell_dialogs. This both streamlines and simplifies our dialog logic, and is ideally a first step towards using//shell_dialogscross-platform. Per #32857 (comment), this is the easiest to port over as it overlaps most completely with what's available upstream. By using Chromium's implementation, we also gain access to kde and portal-specific implementations of dialogs on Linux.Notes:
showOverwriteConfirmationas @aiddya noted in the chart comment, but that should be done in a discrete PR (it's the default on GTK 4+)cc @bpasero
Checklist
npm testpassesRelease Notes
Notes: none