Bug 479574

Summary: With Qt 6.7, items in combobox menus are sized incorrectly and unreadable
Product: [Frameworks and Libraries] frameworks-qqc2-desktop-style Reporter: Patrick Silva <bugseforuns>
Component: generalAssignee: kdelibs bugs <kdelibs-bugs>
Status: RESOLVED FIXED    
Severity: normal CC: ahiemstra, dmatteo002, justamail12111, kde, nate, nicolas.fella, noahadvs, notmart, semlraug, sitter
Priority: NOR Keywords: qt6
Version: unspecified   
Target Milestone: ---   
Platform: Arch Linux   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:
Attachments: screen recording

Description Patrick Silva 2024-01-09 13:01:55 UTC
Created attachment 164777 [details]
screen recording

STEPS TO REPRODUCE
1. open Dolphin
2. right-click on a folder and choose "Properties"
3. click on "Share" tab
4. check "Share this folder with other computers on the local network" option
5. open any combobox

OBSERVED RESULT
the options from the comboboxes are not readable. Please watch the attached screen recording.

EXPECTED RESULT
the options from the comboboxes are readable. 

SOFTWARE/OS VERSIONS
Operating System: Arch Linux 
KDE Plasma Version: 5.91.0
KDE Frameworks Version: 5.247.0
Qt Version: 6.7.0
Graphics Platform: Wayland
Comment 1 Harald Sitter 2024-01-12 12:07:39 UTC
Qt bug maybe? On Qt 6.6 this still works fine.
Comment 2 Nate Graham 2024-01-18 00:48:42 UTC
Bug 479909 is another occurrence in a Plasma config window. Looks like a general issue in qqc2-desktop-style on Qt 6.7; moving there.
Comment 3 Nate Graham 2024-01-18 00:49:07 UTC
*** Bug 479909 has been marked as a duplicate of this bug. ***
Comment 4 Nicolas Fella 2024-01-18 01:03:09 UTC
I see it for very few comboboxes only, it doesn't seem to be a general issue
Comment 5 Nicolas Fella 2024-01-18 01:13:10 UTC
It depends on the type of model apparently.

This is broken:

import QtQuick
import QtQuick.Controls

Item {
    ComboBox {
        model: [
            {
                "label": "Small",
                "spacing": 0
            },
            {
                "label": "Normal",
                "spacing": 1
            },
            {
                "label": "Large",
                "spacing": 3
            },
        ]

        textRole: "label"
    }
} 


This works:

ComboBox {
    model: ["A", "B", "C"]
    currentIndex: 2
}
Comment 6 Nicolas Fella 2024-01-18 01:14:39 UTC
It's broken in 6.7 with org.kde.desktop and org.kde.breeze, it works with Material and Imagine

It works with all styles in 6.6
Comment 8 Nicolas Fella 2024-01-18 22:30:54 UTC
We probably want to follow https://codereview.qt-project.org/c/qt/qtdeclarative/+/467010 in our ComboBox impl
Comment 9 Bug Janitor Service 2024-01-18 23:16:01 UTC
A possibly relevant merge request was started @ https://invent.kde.org/frameworks/qqc2-desktop-style/-/merge_requests/358
Comment 10 Antonio Rojas 2024-01-20 10:53:06 UTC
*** Bug 480087 has been marked as a duplicate of this bug. ***
Comment 11 Antonio Rojas 2024-01-21 17:17:34 UTC
*** Bug 480140 has been marked as a duplicate of this bug. ***
Comment 12 Nicolas Fella 2024-01-21 19:30:27 UTC
Git commit 73303bf4e23d41c84f3d18e61cc9a44c6ba4f55b by Nicolas Fella.
Committed on 19/01/2024 at 00:15.
Pushed by nicolasfella into branch 'master'.

Simplify ComboBox delegate

This matches the upstream styles

See https://codereview.qt-project.org/c/qt/qtdeclarative/+/467010

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

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