Created attachment 182363 [details] screenshot real STEPS TO REPRODUCE 1. run ``` busctl call --user org.freedesktop.portal.Desktop /org/freedesktop/portal/desktop org.freedesktop.portal.FileChooser OpenFile \ 'ssa{sv}' \ '' \ 'badly designed file chooser' \ 1 \ 'choices' \ 'a(ssa(ss)s)' \ 1 \ 'my checkbox id' \ 'my checkbox name' \ 0 \ 'true' ``` OBSERVED RESULT checkbox overlaps left side (look at screeenshot real.webp) and there is big region in top left corner where clicking doesn't work EXPECTED RESULT checkbox is below file type combo box (look at screeenshot fake.webp) SOFTWARE/OS VERSIONS KDE neon Unstable edition KDE Plasma Version: 6.4.80 KDE Frameworks Version: 6.16.0 Qt Version: 6.9.0
Created attachment 182364 [details] screehshot fake this is patch (original https://invent.kde.org/plasma/xdg-desktop-portal-kde/-/commit/72e1eb28239bcf78f3ff6df2ca80d4b9e8f62b3a) makes checkbox be below file type combo box and clicking works everywhere: ``` diff --git a/src/filechooser.cpp b/src/filechooser.cpp index 8b4ba602..1411bf36 100644 --- a/src/filechooser.cpp +++ b/src/filechooser.cpp @@ -482,7 +482,6 @@ void FileChooserPortal::OpenFile(const QDBusObjectPath &handle, if (optionsWidget) { fileDialog->m_fileWidget->setCustomWidget({}, optionsWidget); - optionsWidget->setParent(fileDialog); } delayReply(message, fileDialog, this, [fileDialog, optionsWidget, checkboxes, comboboxes](int dialogResult) -> QVariantList { ``` why does `optionsWidget->setParent(fileDialog);` exist?
What's the use case for adding a checkbox in that manner ? I didn't even know that was possible!
Created attachment 182365 [details] handbrake 1.9.2 flatpak checkboxes (In reply to Nate Graham from comment #2) > What's the use case for adding a checkbox in that manner ? handbrake 1.9.2 flatpak File → Open Directory...
A possibly relevant merge request was started @ https://invent.kde.org/plasma/xdg-desktop-portal-kde/-/merge_requests/408
Git commit a4b677e211fcc11bbf5a53fa895dd4a56adc668d by David Redondo. Committed on 23/06/2025 at 14:36. Pushed by davidre into branch 'master'. filechooser: Do not parent the optionswidget KFileWidget does already reparent the widget and it breaks the layout FIXED-IN:6.4.1 M +0 -2 src/filechooser.cpp https://invent.kde.org/plasma/xdg-desktop-portal-kde/-/commit/a4b677e211fcc11bbf5a53fa895dd4a56adc668d
Git commit 6bfd36b7f4884fff7da82795581caeabd98fb070 by David Redondo. Committed on 23/06/2025 at 14:45. Pushed by davidre into branch 'Plasma/6.4'. filechooser: Do not parent the optionswidget KFileWidget does already reparent the widget and it breaks the layout FIXED-IN:6.4.1 (cherry picked from commit a4b677e211fcc11bbf5a53fa895dd4a56adc668d) Co-authored-by: David Redondo <kde@david-redondo.de> M +0 -2 src/filechooser.cpp https://invent.kde.org/plasma/xdg-desktop-portal-kde/-/commit/6bfd36b7f4884fff7da82795581caeabd98fb070