After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 792152 - With Adwaita Dark, get light text on light background for expanded multi-line entries
With Adwaita Dark, get light text on light background for expanded multi-line...
Status: RESOLVED FIXED
Product: gnome-logs
Classification: Other
Component: general
3.26.x
Other Linux
: Normal normal
: ---
Assigned To: gnome-logs maintainer(s)
gnome-logs maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2018-01-03 08:10 UTC by krinkodot22
Modified: 2018-02-01 14:40 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
The light-on-light text entries. (16.74 KB, image/png)
2018-01-03 08:10 UTC, krinkodot22
  Details
theme: avoid using hard-coded colors (1.89 KB, patch)
2018-01-31 10:03 UTC, Jakub Steiner
none Details | Review
theme: avoid using hard-coded colors (1.60 KB, patch)
2018-01-31 10:05 UTC, Jakub Steiner
committed Details | Review

Description krinkodot22 2018-01-03 08:10:01 UTC
Created attachment 366239 [details]
The light-on-light text entries.

When using the Adwaita Dark GTK theme, any multi-line log message that has been expanded appears with light text on a light background, making it difficult to read. See the screenshot for an example.

This likely means that the colour of either the text or the background is hard-coded instead of using a colour inherited from the current GTK theme.
Comment 1 Jonathan Kang 2018-01-03 09:49:31 UTC
Thanks for the report. Issue confirmed.

(In reply to krinkodot22 from comment #0)
> This likely means that the colour of either the text or the background is
> hard-coded instead of using a colour inherited from the current GTK theme.

You are right. The colour is hard-coded atm. The dark variant theme wasn't
considered while implementing this.
Comment 2 Introoter 2018-01-24 09:37:18 UTC
Please fix this in 3.28 atleast, I've hit this too. It's annoying af.
Comment 3 Jonathan Kang 2018-01-25 09:00:38 UTC
@Jakub

Could you please help with this one providing a colour schema that works in
terms of different themes?

Thanks.
Comment 4 Jakub Steiner 2018-01-31 10:03:47 UTC
Created attachment 367678 [details] [review]
theme: avoid using hard-coded colors

- Have include a bit of a rant. This is actually bad. We are back
  at focusing on themes. Adwaita dark wasn't meant to be a global
  theme users can opt in. It was meant to be for application to opt
  in.

  Here we are again, wasting cycles on making things work for X themes
  rather than doing something useful. Like flapjack. Having an easy to
  use development environment.
Comment 5 Jakub Steiner 2018-01-31 10:05:29 UTC
Created attachment 367679 [details] [review]
theme: avoid using hard-coded colors

 Use share operations instead of hardcoded colors to make it work
 on Adwaita-dark.
Comment 6 Jonathan Kang 2018-02-01 14:39:08 UTC
Comment on attachment 367679 [details] [review]
theme: avoid using hard-coded colors

Looks good. Thanks for the patch.

I've pushed it to master as commit 0968474564d8bb114b05b69e914606455539ef1b
and gnome-3-26 as commit c4d29e4c0294962e7cbc43ba37d0ea7a23dfd01d.
Comment 7 Jonathan Kang 2018-02-01 14:40:13 UTC
I'll release 3.26.3 for this issue.

Close this as RESOLVED FIXED.