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 765727 - Diff/index view colors should be inverted in dark themes
Diff/index view colors should be inverted in dark themes
Status: RESOLVED NOTABUG
Product: gitg
Classification: Applications
Component: gui
git master
Other Linux
: Normal normal
: ---
Assigned To: gitg-maint
gitg-maint
Depends on:
Blocks:
 
 
Reported: 2016-04-28 14:10 UTC by Mindaugas J.
Modified: 2016-10-24 00:17 UTC
See Also:
GNOME target: ---
GNOME version: ---



Comment 1 Ignacio Casal Quinteiro (nacho) 2016-04-28 14:13:22 UTC
We do have code in place for adwaita dark. I guess you are using the ubuntu theme?
Comment 2 Mindaugas J. 2016-04-28 14:18:44 UTC
I am on manjaro with lxdm. The system just moved to 3.20 GTK.

I tried quite a few supposedly compatible dark themes, those in the screenshots are Vertex-Maia(-Dark). It seems I don't have adwaita dark available by default. I'll need to get it from AUR
Comment 3 Mindaugas J. 2016-04-28 15:19:28 UTC
Nevermind, I apparently needed to use 'prefer dark'.

It does not work with GTK2 theming nor QT. I hate GTK...
Comment 4 Ignacio Casal Quinteiro (nacho) 2016-04-28 15:21:53 UTC
well, these are colors hard coded in gitg, we detect the user has a dark theme by checking that setting. Clearly if you set a dark theme but that setting is not set then we will not detect it in the app... I am willing to hear about better options here...
Comment 5 Mindaugas J. 2016-04-28 15:29:56 UTC
Well, yes that works for me as well when NOT using adwaita. Sorry for venting.

To summarize:
1) use a multi-toolkit dark-only theme like Vertex-Maia-Dark
2) set gtk-application-prefer-dark-theme=1 in .config/gtk-3.0/settings.ini
3) profit
Comment 6 Ignacio Casal Quinteiro (nacho) 2016-04-28 15:33:02 UTC
See that this is IMHO a downstream bug in Vertex-Maia. That theme should detect if prefer-dark is set and use Vertex-Maia if disabled or Vertex-Maia-Dark if enabled. You might want to report this downstream. Thanks anyway for reporting here.
Comment 7 Mindaugas J. 2016-10-24 00:17:05 UTC
Hello, I'm back :/

Here is how it now looks on 22 with syntax highlighting: https://i.imgur.com/bog9hVa.png

Any suggestions how to adjust this? Where do syntax colors come from?

I guess it's hard to make this feature work with white on black. I need to revert that somehow...