Bug 482199 - KCM header appearance is broken
Summary: KCM header appearance is broken
Status: RESOLVED FIXED
Alias: None
Product: kdeconnect
Classification: Applications
Component: common (other bugs)
Version First Reported In: 24.02.0
Platform: Arch Linux Linux
: NOR wishlist
Target Milestone: ---
Assignee: Albert Vaca Cintora
URL:
Keywords: qt6
Depends on:
Blocks:
 
Reported: 2024-03-01 22:19 UTC by Onur Ankut
Modified: 2024-03-07 14:37 UTC (History)
3 users (show)

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


Attachments
comparison (57.31 KB, image/png)
2024-03-01 22:19 UTC, Onur Ankut
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Onur Ankut 2024-03-01 22:19:43 UTC
Created attachment 166279 [details]
comparison

SUMMARY
The header area has differently positioned text, different color, no outline and it shares the same color with sidebar section. Screenshot is available as an attachment.

SOFTWARE/OS VERSIONS
OS: Arch Linux
KDE Plasma Version: 6.0.0
KDE Frameworks Version: 6.0.0 
Qt Version: 6.6.2

ADDITIONAL INFORMATION
I selected the common component because KCM component doesn't show up at the list.
Comment 1 Nate Graham 2024-03-05 17:46:45 UTC
Yes, this KCM was made using QtWidgets, so its visual styling is a bit different from the newer QtQuick-based KCMs
Comment 2 Nicolas Fella 2024-03-05 17:50:08 UTC
The KCM being widgets is besides the port of the report. The header should not look like this, and does not for other widgets-based KCMs
Comment 3 Onur Ankut 2024-03-05 17:52:00 UTC
I'm not talking about whole design, just the header area. It looks coherent on my Plasma 5 installation. Also the header text is selectable.
Comment 4 Nicolas Fella 2024-03-05 18:17:04 UTC
Same thing happens with the Digital Camera KCM.

See https://bugs.kde.org/show_bug.cgi?id=478091
Comment 5 Bug Janitor Service 2024-03-05 19:52:25 UTC
A possibly relevant merge request was started @ https://invent.kde.org/network/kdeconnect-kde/-/merge_requests/645
Comment 6 Albert Vaca Cintora 2024-03-06 22:04:19 UTC
Git commit 1785160049b29f3be6c729e344774afab2f26928 by Albert Vaca Cintora, on behalf of Nicolas Fella.
Committed on 06/03/2024 at 22:03.
Pushed by albertvaka into branch 'master'.

[kcm] Use correct KCModule constructor
Related: bug 478091

M  +2    -2    kcm/kcm.cpp
M  +1    -1    kcm/kcm.h

https://invent.kde.org/network/kdeconnect-kde/-/commit/1785160049b29f3be6c729e344774afab2f26928
Comment 7 Nicolas Fella 2024-03-07 14:37:54 UTC
Git commit c5934e3d9b0b7c5b56eb69b28c2ba9e5d0c35c52 by Nicolas Fella.
Committed on 07/03/2024 at 14:37.
Pushed by nicolasfella into branch 'release/24.02'.

[kcm] Use correct KCModule constructor
Related: bug 478091
(cherry picked from commit 1785160049b29f3be6c729e344774afab2f26928)

M  +2    -2    kcm/kcm.cpp
M  +1    -1    kcm/kcm.h

https://invent.kde.org/network/kdeconnect-kde/-/commit/c5934e3d9b0b7c5b56eb69b28c2ba9e5d0c35c52