| Summary: | ComboBox entry popup is not resized to delegate width | ||
|---|---|---|---|
| Product: | [Frameworks and Libraries] frameworks-qqc2-desktop-style | Reporter: | Patrick Silva <bugseforuns> |
| Component: | general | Assignee: | Marco Martin <notmart> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | a.stippich, aspotashev, bugs-kde.org, ddascalescu+kde, filipfila.kde, isma.af, kde, kishore96, kollix, launchpad, meven29, mikel5764, nate, pascal, postix, tomashnyk |
| Priority: | NOR | ||
| Version: | 5.66.0 | ||
| Target Milestone: | --- | ||
| Platform: | Arch Linux | ||
| OS: | Linux | ||
| URL: | https://doc.qt.io/qt-6/qml-qtquick-controls2-combobox.html#implicitContentWidthPolicy-prop | ||
| See Also: | https://bugs.kde.org/show_bug.cgi?id=403151 | ||
| Latest Commit: | https://invent.kde.org/frameworks/qqc2-desktop-style/commit/bd16e48af6b6a99d8008e2950fe97503a3007a13 | Version Fixed In: | 5.91 |
| Sentry Crash Report: | |||
| Attachments: |
screenshot
screenshot of screen locker settings new screenshot of Screen Locking KCM |
||
Confirmed. Possibly a QQC2-desktop-style issue? Git commit c4a35f5e36bd5d5814b7288e991e125792e24d2c by David Edmundson.
Committed on 16/01/2019 at 12:45.
Pushed by davidedmundson into branch 'master'.
Set CombooBox QStyleOptionState == On rather than Sunken to match qwidgets
Summary:
See the end of QComboBox::initStyleOption
if (d->container && d->container->isVisible())
option->state |= QStyle::State_On;
The relevant breeze code reads:
const bool hasFocus( enabled && ( state & (State_HasFocus |
State_Sunken ) ) );
const bool sunken( state & (State_On|State_Sunken) );
So we still remain sunken in breeze, but we don't assume hasFocus, which
gives the background an incorrect colour.
Test Plan:
Ran testComboBox from Aleix
Debug inside Breeze QStyle + reading Qt code
Reviewers: broulik
Reviewed By: broulik
Subscribers: broulik, plasma-devel
Tags: #plasma
Differential Revision: https://phabricator.kde.org/D18284
M +1 -2 org.kde.desktop/ComboBox.qml
https://commits.kde.org/qqc2-desktop-style/c4a35f5e36bd5d5814b7288e991e125792e24d2c
Seeing last commit, should it be considered fixed ? Yeah, I haven't seen this broken again. Let's call it fixed unless it starts happening once more! (In reply to Nate Graham from comment #4) > Yeah, I haven't seen this broken again. Let's call it fixed unless it starts > happening once more! I think it's still happening: https://phabricator.kde.org/file/data/ikkaw7cy5onaevd7uuhq/PHID-FILE-frrm7ezidk4god56xu6l/image.png It's not fixed, my patch just adjusted some colouring which is what I understood this bug to be. It originally said "text is not readable" or something. After committing it was explained and I reopened and reworded the title to be clearer. *** Bug 414443 has been marked as a duplicate of this bug. *** *** Bug 418157 has been marked as a duplicate of this bug. *** (In reply to Nate Graham from comment #8) > *** Bug 418157 has been marked as a duplicate of this bug. *** I hope this issue can be addressed at a fundamental level and not just on a case by case basis. That control is used in numerous dialogs throughout KDE and the issue of it truncating and elipsifying unnecessarily (instead utilizing more of the available horizontal space) is ubiquitous; it becomes especially emphasized when users have font-sizes set to a larger font than the 10pt default. Hopefully, developers can change the way that control acts in one code-place, but in a manner that all instances (where that control is used) will benefit from those efforts and not just a single specific dialog. Yes, that's what this bug reports is all about. :) *** Bug 421746 has been marked as a duplicate of this bug. *** *** Bug 364871 has been marked as a duplicate of this bug. *** Created attachment 130859 [details]
screenshot of screen locker settings
Operating System: KDE neon Unstable Edition
KDE Plasma Version: 5.19.80
KDE Frameworks Version: 5.74.0
Qt Version: 5.14.2
should be fixed in qt 6 with https://doc.qt.io/qt-6/qml-qtquick-controls2-combobox.html#implicitContentWidthPolicy-prop Thanks for finding that! I guess once Qt6 comes out, we can mark this as RESOLVED UPSTREAM. *** Bug 432829 has been marked as a duplicate of this bug. *** *** Bug 437362 has been marked as a duplicate of this bug. *** Eh, I guess we can do it now. at least duplicates bug 432829 and bug 421746 are still reproducible on neon unstable, I'm sure all of them are still reproducible, since this is fixed only in Qt 6 and we're still using Qt 5. :) Can the fix be backported to KDE Qt? Not sure, but that would be a question for someone else. :) Maybe ask on kde-devel@kde.org? *** Bug 445394 has been marked as a duplicate of this bug. *** *** Bug 446878 has been marked as a duplicate of this bug. *** *** Bug 448073 has been marked as a duplicate of this bug. *** The problem is that internally that is a ListView, which is not trivial to have "the implicit width of the largest of its items" as that one may not even be instantiated.. It may be replaced with a Repeater and a Layout, tough that could be havier... usually comboboxes don't have that many items, so would usually be fine, not sure Git commit bd16e48af6b6a99d8008e2950fe97503a3007a13 by Alexander Stippich. Committed on 20/01/2022 at 19:36. Pushed by astippich into branch 'master'. adapt width of combobox to its content M +16 -2 org.kde.desktop/ComboBox.qml https://invent.kde.org/frameworks/qqc2-desktop-style/commit/bd16e48af6b6a99d8008e2950fe97503a3007a13 *** Bug 449640 has been marked as a duplicate of this bug. *** Created attachment 148660 [details]
new screenshot of Screen Locking KCM
This bug persists with Screen Locking KCM.
Operating System: KDE neon Unstable Edition
KDE Plasma Version: 5.24.80
KDE Frameworks Version: 5.94.0
Qt Version: 5.15.3
Graphics Platform: Wayland
I can reproduce that issue, but it seems specific to the Screen Locking KCM. If you open the same UI (Plasma Wallpaper settings) from the desktop, the bug doesn't happen there. And it doesn't happen in any other comboboxes with long text that I tried. Can you file a new bug report about this issue in System Settings | kcm_screenlocker? Thanks! |
Created attachment 117426 [details] screenshot SUMMARY I'm testing the new virtual desktops kcm on neon dev unstable, my system language is pt-br. STEPS TO REPRODUCE 1. search for the "virtual desktops" kcm in the apps launcher and open it 2. open "Show an animation when change" dropdown 3. OBSERVED RESULT two options available are not entirely readable. See the screenshot taken on neon dev unstable please. EXPECTED RESULT all options available are entirely readable SOFTWARE/OS VERSIONS Windows: MacOS: Linux/KDE Plasma: (available in About System) KDE Plasma Version: KDE Frameworks Version: Qt Version: ADDITIONAL INFORMATION