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 767960 - Regression for theming of xchat/hexchat's input box
Regression for theming of xchat/hexchat's input box
Status: RESOLVED OBSOLETE
Product: gnome-themes-standard
Classification: Core
Component: Adwaita GTK2 theme
3.20
Other Linux
: Normal normal
: ---
Assigned To: gnome-themes-standard-maint
gnome-themes-standard-maint
Depends on:
Blocks:
 
 
Reported: 2016-06-23 01:39 UTC by Patrick Griffis (tingping)
Modified: 2018-01-24 14:55 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Patrick Griffis (tingping) 2016-06-23 01:39:00 UTC
In [he]xchat there is an option in Preferences > Input box > Use text box font and colors that dynamically modifies the gtk theme. This worked in 3.18 but 3.20 does not.

git bisect led me to https://git.gnome.org/browse/gnome-themes-standard/commit/?id=ba9c19414b0efbc0636996a3ce14a53772832e7e which does have mention of the widget "*xchat-inputbox".

Alternatively if you know a better way for the application to handle this that would be ok too.
Comment 1 Juraj Fiala 2016-06-26 14:55:51 UTC
Hey, I removed the [he]xchat hack with a bunch of others since they were really old and usually long fixed, only to find out several months later that the issue was still there, but I haven't got to adding it back.

Funnily enough it has nothing to do with this bug. It's about the bug with entries having a white area around themselves, but if you are interested in fixing it just put the affected entries inside a GtkEventBox.

Sadly _this_ bug is a WONTFIX. I've covered the details in #767562.

I'm not sure what you can do from the application but overriding the theme not to use the entry assets should do it, at the expense of losing the look. The only real solution is to switch to gtk3.

Cheers.
Comment 2 Patrick Griffis (tingping) 2016-06-29 20:13:51 UTC
> overriding the theme not to use the entry assets should do it

How exactly would you do that? I tried copying what you posted in
#767562 but it didn't seem to work.
Comment 3 Patrick Griffis (tingping) 2016-06-29 20:18:17 UTC
Nevermind I did get that to work though it still has the border you were mentioning
any tips on how to get rid of that.

http://i.imgur.com/DgxEXwk.png
Comment 4 Juraj Fiala 2016-06-30 19:35:59 UTC
(In reply to Patrick Griffis (tingping) from comment #3)
> Nevermind I did get that to work

How? I'm quite curious.

> though it still has the border you were
> mentioning
> any tips on how to get rid of that.

IIRC you need to lower the border values for the entry-border.png, but it'll cut of the corners. However that's pulled deep from my memory so it may not be correct. I'll get back to you tomorrow.
Comment 5 Patrick Griffis (tingping) 2016-06-30 20:45:12 UTC
(In reply to Juraj Fiala from comment #4)
> (In reply to Patrick Griffis (tingping) from comment #3)
> > Nevermind I did get that to work
> 
> How? I'm quite curious.
> 

    engine "pixmap" {
      image {
        function = FLAT_BOX
        state    = NORMAL
      }
      image {
        function = FLAT_BOX
        state    = ACTIVE
      }
    }
Comment 6 Juraj Fiala 2016-07-01 07:21:14 UTC
(In reply to Patrick Griffis (tingping) from comment #5)
>     engine "pixmap" {
>       image {
>         function = FLAT_BOX
>         state    = NORMAL
>       }
>       image {
>         function = FLAT_BOX
>         state    = ACTIVE
>       }
>     }

But I mean did you do this from the application side or just manually patch the file?
Comment 7 Patrick Griffis (tingping) 2016-07-01 13:45:04 UTC
(In reply to Juraj Fiala from comment #6)
> But I mean did you do this from the application side or just manually patch
> the file?

gtk_rc_parse_string()
Comment 8 Juraj Fiala 2016-10-09 10:37:17 UTC
Why do I do this everytime? I know this is a very late "tomorrow" but I'll answer anyway:

I've thought about this and the best way to fix this without ruining the experience for 99% of applications is to override with your own entry assets that do enable changing the background. If you want I can make these assets for along with the rc segment to get them working.
Comment 9 Patrick Griffis (tingping) 2016-10-09 15:14:59 UTC
I've already been shipping that workaround for months and honestly seems fine.
Comment 10 Juraj Fiala 2016-10-09 18:29:51 UTC
Sure you don't want the assets? Just tried it, seems fine, but it would look a bit better.
Comment 11 Patrick Griffis (tingping) 2016-10-09 20:03:55 UTC
If you already put in the work I suppose it is worth a shot.
Comment 12 GNOME Infrastructure Team 2018-01-24 14:55:33 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to GNOME's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.gnome.org/GNOME/gnome-themes-extra/issues/5.