Bug 439379

Summary:	Add the ability to set the separator color
Product:	[Plasma] Breeze	Reporter:	popov895 <popov895>
Component:	QStyle	Assignee:	Plasma Bugs List <plasma-bugs-null>
Status:	CONFIRMED ---
Severity:	wishlist	CC:	kde, nate, noahadvs, plasma-bugs-null, popov895
Priority:	NOR
Version First Reported In:	5.22.2
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:
Attachments:	Header

Description popov895 2021-07-01 18:16:57 UTC

SUMMARY

Currently, the header uses a text color with some transparency to draw the bottom line and it's impossible to specify custom color. It would be nice to add an additional value to the color scheme file to specify the color of the bottom line of the header, and in case of its absence, use the default value (i.e. text color with some transparency, as it is now).

SOFTWARE/OS VERSIONS

Operating System: openSUSE Tumbleweed 20210628
KDE Plasma Version: 5.22.2
KDE Frameworks Version: 5.83.0
Qt Version: 5.15.2
Graphics Platform: X11

Comment 1 David Edmundson 2021-07-02 10:01:08 UTC

Which header?

Comment 2 popov895 2021-07-02 10:16:18 UTC

Created attachment 139799 [details]
Header

Comment 3 Noah Davis 2021-07-02 12:46:03 UTC

Sounds like it would be useful to have a separator color role in color schemes

Comment 4 popov895 2021-07-03 11:32:49 UTC

Also make it accessible from svg images.

Comment 5 Bug Janitor Service 2021-07-18 04:33:35 UTC

Dear Bug Submitter,

This bug has been in NEEDSINFO status with no change for at least
15 days. Please provide the requested information as soon as
possible and set the bug status as REPORTED. Due to regular bug
tracker maintenance, if the bug is still in NEEDSINFO status with
no change in 30 days the bug will be closed as RESOLVED > WORKSFORME
due to lack of needed information.

For more information about our bug triaging procedures please read the
wiki located here:
https://community.kde.org/Guidelines_and_HOWTOs/Bug_triaging

If you have already provided the requested information, please
mark the bug as REPORTED so that the KDE team knows that the bug is
ready to be confirmed.

Thank you for helping us make KDE software even better for everyone!

Comment 6 popov895 2021-07-18 08:32:35 UTC

See Comment2

Comment 7 Nate Graham 2021-08-04 22:26:33 UTC

Yeah, let's use this to track exposing the separator color. This is something VDG has wanted for a while, and it would generically allow you to do this. Noah, didn't you have work for that? Can we finish it up?

Comment 8 Noah Davis 2021-08-05 00:23:56 UTC

(In reply to Nate Graham from comment #7)
> Yeah, let's use this to track exposing the separator color. This is
> something VDG has wanted for a while, and it would generically allow you to
> do this. Noah, didn't you have work for that? Can we finish it up?

I did, but the patch was rejected by more than one senior KDE dev. I was told to submit a patch to Qt instead, but the process of setting up a complete environment for Qt and KDE development requires so much time that I just haven't gotten close to submitting a Qt patch. On top of that, we won't even be able to use the separator color role until Plasma uses Qt 6, if I do the upstream patch. Maybe I could try to submit a patch to KColorScheme again now that it's too late for Qt 5.

Comment 9 Nate Graham 2021-08-05 02:00:47 UTC

Let's just do it here in KDE; I'm sure we can find a way to make it acceptable.

Comment 10 Nate Graham 2022-03-11 17:39:49 UTC

In progress with https://invent.kde.org/frameworks/kconfigwidgets/-/merge_requests/113