Bug 479574 - With Qt 6.7, items in combobox menus are sized incorrectly and unreadable
Summary: With Qt 6.7, items in combobox menus are sized incorrectly and unreadable
Status: RESOLVED FIXED
Alias: None
Product: frameworks-qqc2-desktop-style
Classification: Frameworks and Libraries
Component: general (show other bugs)
Version: unspecified
Platform: Arch Linux Linux
: NOR normal
Target Milestone: ---
Assignee: kdelibs bugs
URL:
Keywords: qt6
: 479909 480087 480140 (view as bug list)
Depends on:
Blocks:
 
Reported: 2024-01-09 13:01 UTC by Patrick Silva
Modified: 2024-01-21 19:30 UTC (History)
10 users (show)

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


Attachments
screen recording (1.01 MB, video/webm)
2024-01-09 13:01 UTC, Patrick Silva
Details

Note You need to log in before you can comment on or make changes to this bug.
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