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 788575 - HighContrastInverse: Windows lack visible borders, merge into each other and dark backgrounds
HighContrastInverse: Windows lack visible borders, merge into each other and ...
Status: RESOLVED FIXED
Product: gtk+
Classification: Platform
Component: Themes
3.22.x
Other Linux
: Normal normal
: ---
Assigned To: gtk-bugs
gtk-bugs
Depends on:
Blocks:
 
 
Reported: 2017-10-05 21:54 UTC by Daniel Boles
Modified: 2017-10-05 23:42 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
screenshot (724.59 KB, image/png)
2017-10-05 21:54 UTC, Daniel Boles 		  Details
HighContrastInverse: Make window borders visible (3.25 KB, patch)
2017-10-05 23:38 UTC, Daniel Boles 		committed Details | Review
screenshot after patch (789.02 KB, image/png)
2017-10-05 23:40 UTC, Daniel Boles 		  Details

Description Daniel Boles 2017-10-05 21:54:21 UTC 
Created attachment 361002 [details]
screenshot

title and screenshot say it all

In HC normal, there is a grey border, which is OK albeit a little on the fine side.

In HC Inverse, no such luck, and the black windows easily merge into an undefined mass, same with when they're placed on dark background colours.
Comment 1 Daniel Boles 2017-10-05 23:38:18 UTC 
Created attachment 361005 [details] [review]
HighContrastInverse: Make window borders visible

They were hard-coded to a transparent black, but that is our bg colour
in HC Inverse, so windows stacked on top of each other or a dark
background blended together into a mush.

Fix this by making the $_wm_border* colours relative to the fg colour,
so that HighContrastInverse gets borders that are transparentised white.
Comment 2 Daniel Boles 2017-10-05 23:40:21 UTC 
Created attachment 361006 [details]
screenshot after patch
Comment 3 Daniel Boles 2017-10-05 23:42:07 UTC 
Attachment 361005 [details] pushed as 6c3d17b - HighContrastInverse: Make window borders visible