Bug 720913 - update window button settings for csd
update window button settings for csd
Status: RESOLVED FIXED
Product: gnome-tweak-tool
Classification: Applications
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: GNOME Tweak Tool maintainer(s)
GNOME Tweak Tool maintainer(s)
:
Depends on:
Blocks:
  Show dependency tree
 
Reported: 2013-12-22 03:08 UTC by Matthias Clasen
Modified: 2014-02-12 17:08 UTC (History)
3 users (show)

See Also:
GNOME target: 3.12
GNOME version: ---


Attachments
add support for csd windows with header bars (3.13 KB, patch)
2013-12-24 15:42 UTC, Alex Muñoz
none Details | Diff | Review
add support for csd windows with header bars (3.17 KB, patch)
2014-01-14 06:48 UTC, Alex Muñoz
committed Details | Diff | Review

Description Matthias Clasen 2013-12-22 03:08:59 UTC
For csd windows with header bars, GTK+ now looks at the Gtk/DecorationLayout xsetting. The format of the string looks like this:

menu:minimize,maximize,close

for a menu button (app menu fallback) on the left, and min/max/close on the right. To set this, you should add a suitable entry to the org.gnome.settings-daemon.plugins.xsettings overrides setting, e.g.:

{'Gtk/DecorationLayout': <':minimize,maximize,close'>}
Comment 1 Alex Muñoz 2013-12-24 15:42:29 UTC
Created attachment 264844 [details] [review]
add support for csd windows with header bars
Comment 2 Matthias Clasen 2014-01-13 13:10:52 UTC
Ping, would be great to get this merged
Comment 3 John Stowers 2014-01-13 15:02:13 UTC
Alex - go ahead and commit this to master. We actually depend on this new layout since bug 722017 was committed.
Comment 4 Alex Muñoz 2014-01-14 06:48:21 UTC
Created attachment 266231 [details] [review]
add support for csd windows with header bars
Comment 5 Giovanni Campagna 2014-02-05 12:42:10 UTC
See also bug 723669, which has a different approach on this (and would require no change from gnome-tweak-tool)
Comment 6 John Stowers 2014-02-05 12:56:52 UTC
Thanks Giovanni - I will keep an eye on that bug.

Alex, what was the problem you mentioned you had with your patch?
Comment 7 Alex Muñoz 2014-02-10 15:40:22 UTC
Sorry for the delay!
the patch works, the problem is when more than one tweak that depends of the xsettings is changed, only  the last changed tweak is applied.
Comment 8 John Stowers 2014-02-10 16:12:43 UTC
(In reply to comment #7)
> Sorry for the delay!
> the patch works, the problem is when more than one tweak that depends of the
> xsettings is changed, only  the last changed tweak is applied.

Did you try and fix this? I think the xsettingsmanager should be a singleton, so you are probbably just seeing some async race condition.

It should not be too hard to fix...
Comment 9 Alex Muñoz 2014-02-12 17:06:10 UTC
Review of attachment 266231 [details] [review]:

Pushed as f79ab44

Note You need to log in before you can comment on or make changes to this bug.