Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Custom theming fixes for Adwaita 3.20/GTK 3.20 #1137

Merged
merged 3 commits into from
Jul 16, 2016
Merged

Conversation

b4n
Copy link
Member

@b4n b4n commented Jul 14, 2016

Fixes #1135, and may help for #1101 (?)

This also restores the custom selection color there too that got lost with GTK 3.20, and adds it to GTK2 too (for some reason it never was there).

@b4n b4n added the gtk label Jul 14, 2016
@b4n b4n self-assigned this Jul 14, 2016
base[SELECTED] = "#777711111111"
# try and remove the entry background image on pixmap engine so that our
# background color is visible, and we don't end up with white text on white
# background (workaround for Adwaita 3.20).
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why does this affect 3.20? I thought gtk3 didn't use the .gtkrc file?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Adwaita 3.20.

b4n added 3 commits July 16, 2016 03:27
Adwaita 3.20 on GTK2 uses the pixmap engine to set a background image
on all states of all GtkEntries.  Earlier versions did the same but
with a transparent background, thus not hiding our background color.

Fixes geany#1135, fixes geany#1101.
Follows the GTK3 behavior, and avoid text becoming invisible if the
selection background is too light (unlikely, but possible).
@b4n b4n added this to the 1.29 milestone Jul 16, 2016
@b4n b4n merged commit 5d74eef into geany:master Jul 16, 2016
b4n added a commit that referenced this pull request Jul 16, 2016
Custom theming fixes for Adwaita 3.20/GTK 3.20
@b4n
Copy link
Member Author

b4n commented Jul 16, 2016

Maybe we should add this to a 1.28.1 release, as quite a few people are likely to suffer from that Adwaita issue?

@codebrainz
Copy link
Member

Would it be worth reporting the breakage to Adwaita also?

@codebrainz
Copy link
Member

As for 1.28.1 release, I have no opinion (except that Adwaita is ugly and I don't understand why anyone would use it :)

@b4n
Copy link
Member Author

b4n commented Jul 16, 2016

Would it be worth reporting the breakage to Adwaita also?

I doubt they would care, but it'd be great if someone tried :)

[edit] as it actually breaks existing applications, they may care a little, but that would basically mean they can't use background images anywhere, so I'm not quite sure.

@dmaphy
Copy link
Member

dmaphy commented Jul 16, 2016

Anybody knows how widely spread is the Adwaita theme outside from Fedora? If it affects Fedora only, I can back port the patch and a Geany release could wait then.

@elextr
Copy link
Member

elextr commented Jul 16, 2016

Well, as its the default theme for Gnome I guess it is gonna be available on all Gnome distros, even if the distro defaults to its own hacked theme.

@b4n
Copy link
Member Author

b4n commented Jul 16, 2016

@dmaphy as @elextr said, it's the default GNOME theme so it's likely to affect a lot of people. For example, it affects current Debian testing/unstable.
OTOH, some widespread distros like Ubuntu ship with a custom theme by default, so they won't be affected by the same issues. Who knows if there are other there, though…

But yeah, I think it's likely to have a large impact, hence why I'm considering a point release might be a good idea. Either way you may want to backport the fix in Fedora until we release it, though, your call.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants