Bug 343436

Summary:	Systemmonitor detailed view is almost unreadable with Breeze Dark color theme
Product:	[Unmaintained] ksysguard	Reporter:	supaiku <supaiku>
Component:	ksysguard	Assignee:	KSysGuard Developers <ksysguard-bugs>
Status:	RESOLVED DUPLICATE
Severity:	normal	CC:	anselmolsm, aspotashev, bruno, bugseforuns, hrvoje.senjan, kde, razrfalcon, sebas
Priority:	NOR
Version:	5.2.0
Target Milestone:	---
Platform:	Arch Linux
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:
Attachments:	Screenshot

Description supaiku 2015-01-28 07:46:08 UTC

Use the Breeze Dark color theme and fire up systemmonitor and open the detail view for a process -> black font on dark background. Even the font looks really weird and doesn't seem to match the rest of the system (Oxygen Font).

Reproducible: Always

Steps to Reproduce:
See details.

Comment 1 Sebastian Kügler 2015-01-28 11:43:54 UTC

Could you attach a screenshot illustrating the problem? That would make it way easier to triage and fix.

Comment 2 supaiku 2015-01-28 12:18:21 UTC

Created attachment 90746 [details]
Screenshot

Comment 3 Sebastian Kügler 2015-01-28 12:43:11 UTC

Thanks, for the additional information. I've attached the correct meta information to the bug so it's easier to track for us.

The problem is that the HTML generated for the report doesn't set the right background color. We could make use of Plasma::Theme::styleSheet() for this.

Comment 4 David Edmundson 2015-01-28 13:11:50 UTC

>We could make use of Plasma::Theme::styleSheet() for this.

No we can't, we're in an application. We can't mix and match style info.

Comment 5 Sebastian Kügler 2015-01-28 13:46:21 UTC

Well, in this case we can because there won't be any clashes because it's a separate view.

Probably easier though is to just set the background of the HTML code that gets inserted there.

Comment 6 Anselmo L. S. Melo (anselmolsm) 2015-10-03 14:13:16 UTC

I also use Breeze Dark and this week I had to use the system monitor detailed view many times.

I decided to check what could be done to make that view more readable. As a workaround, I modified scripts/smaps/style.css, adding an entry to set the html body background color to #FCFCFC (the bg color when using Breeze), so the background color doesn't depend on the color scheme in use.

styles.css has many fixed color definitions (texts, links, TOC bg), so IMPOV setting a color for the body bg guarantees that all those colors work well together. Of course, to make this view adaptable to other color schemes, more changes are required, indeed.

Comment 7 Christoph Feck 2017-06-17 13:34:37 UTC

*** Bug 361673 has been marked as a duplicate of this bug. ***

Comment 8 Christoph Feck 2017-06-27 20:31:18 UTC


*** This bug has been marked as a duplicate of bug 360214 ***