Bug 507183 - Click on the down arrow in "Choose application" window results in a shadow
Summary: Click on the down arrow in "Choose application" window results in a shadow
Status: CONFIRMED
Alias: None
Product: frameworks-qqc2-desktop-style
Classification: Frameworks and Libraries
Component: general (other bugs)
Version First Reported In: 6.17.0
Platform: Neon Linux
: HI normal
Target Milestone: ---
Assignee: kdelibs bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2025-07-18 04:24 UTC by Patrick Silva
Modified: 2025-08-08 12:55 UTC (History)
7 users (show)

See Also:
Latest Commit:
Version Fixed/Implemented In:
Sentry Crash Report:


Attachments
screenshot (161.43 KB, image/png)
2025-07-18 04:24 UTC, Patrick Silva
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Patrick Silva 2025-07-18 04:24:14 UTC
Created attachment 183315 [details]
screenshot

STEPS TO REPRODUCE
1. open Dolphin
2. right-click on a file 
3.  hover over "Open with" submenu and chose "Other application..."
4. click on the down arrow

OBSERVED RESULT
the shadow seen in the attached screenshot

EXPECTED RESULT
I'm not sure what exactly is expected here, but clearly the observed result is not correct

SOFTWARE/OS VERSIONS
Operating System: KDE neon Unstable Edition
KDE Plasma Version: 6.4.80
KDE Frameworks Version: 6.17.0
Qt Version: 6.9.1
Graphics Platform: Wayland
Comment 1 David Edmundson 2025-07-18 08:25:26 UTC
It's what combobox does when it makes an empty list.
Comment 2 Nate Graham 2025-07-21 15:16:05 UTC
There's an open merge request to fix this at https://invent.kde.org/plasma/xdg-desktop-portal-kde/-/merge_requests/419
Comment 3 Nate Graham 2025-07-31 22:22:00 UTC
I looked into fixing the styling in the ComboBox component when it has an empty drop-down.

There's logic in the Breeze QStyle to disable the arrow button when the drop-down is empty: https://invent.kde.org/plasma/breeze/-/blob/master/kstyle/breezestyle.cpp?ref_type=heads#L7717

However it would appear this doesn't come across the style bridge for the QML version.
Comment 4 Marco Martin 2025-08-06 08:11:23 UTC
(In reply to Nate Graham from comment #2)
> There's an open merge request to fix this at
> https://invent.kde.org/plasma/xdg-desktop-portal-kde/-/merge_requests/419

seems completely unrelated?
Comment 5 Bug Janitor Service 2025-08-06 08:55:03 UTC
A possibly relevant merge request was started @ https://invent.kde.org/frameworks/qqc2-desktop-style/-/merge_requests/475
Comment 6 Bug Janitor Service 2025-08-06 08:59:18 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/breeze/-/merge_requests/554
Comment 7 Marco Martin 2025-08-06 13:58:28 UTC
Git commit e7fbca2734afb753bc44c08335955c0448971c48 by Marco Martin.
Committed on 06/08/2025 at 13:58.
Pushed by mart into branch 'master'.

ComboBox: Disallow opening empty comboboxes

Don't let the user open a combobox which has zero items in the
popup. Unfortunately there isn't dedicated api for that and the
popup has to be dinamically assigned or unassigned, but it
works well.

the arrow won't yet change its look, as it needs adjustment in Breeze

together with https://invent.kde.org/plasma/breeze/-/merge_requests/554

M  +5    -1    org.kde.desktop/ComboBox.qml

https://invent.kde.org/frameworks/qqc2-desktop-style/-/commit/e7fbca2734afb753bc44c08335955c0448971c48
Comment 8 Marco Martin 2025-08-06 13:58:34 UTC
Git commit bd32342c80286886e626bcb2ad207dd9be509356 by Marco Martin.
Committed on 06/08/2025 at 13:58.
Pushed by mart into branch 'master'.

ComboBox: detect empty combobox on QtQuickControls as well

Also when styling a QQC combobox try to detect when the combobox is empty,
by the style exposing "count" property on the style proxy

needs https://invent.kde.org/frameworks/qqc2-desktop-style/-/merge_requests/475

M  +10   -3    kstyle/breezestyle.cpp

https://invent.kde.org/plasma/breeze/-/commit/bd32342c80286886e626bcb2ad207dd9be509356
Comment 9 Bug Janitor Service 2025-08-06 13:59:32 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/breeze/-/merge_requests/555
Comment 10 Marco Martin 2025-08-06 14:08:10 UTC
Git commit abaf76d1213d9774ae8bf0a4b73a387a24fecdfa by Marco Martin.
Committed on 06/08/2025 at 13:58.
Pushed by mart into branch 'Plasma/6.4'.

ComboBox: detect empty combobox on QtQuickControls as well

Also when styling a QQC combobox try to detect when the combobox is empty,
by the style exposing "count" property on the style proxy

needs https://invent.kde.org/frameworks/qqc2-desktop-style/-/merge_requests/475


(cherry picked from commit bd32342c80286886e626bcb2ad207dd9be509356)

28d87362 ComboBox: detect empty combobox on QtQuickControls as well

Co-authored-by: Marco Martin <notmart@gmail.com>

M  +10   -3    kstyle/breezestyle.cpp

https://invent.kde.org/plasma/breeze/-/commit/abaf76d1213d9774ae8bf0a4b73a387a24fecdfa
Comment 11 Nate Graham 2025-08-08 12:54:56 UTC
Re-opening as the commit has to be reverted because it caused regressions worse than the original issue.

https://invent.kde.org/frameworks/qqc2-desktop-style/-/commit/e5eef9ab76ff3a76a0327459239b68d6d7da4daf