Merge lp:~cassidyjames/elementaryos/enable-appmenu into lp:~elementary-os/elementaryos/default-settings-loki

Proposed by Cassidy James Blaede
Status: Merged
Approved by: Danielle Foré
Approved revision: 195
Merged at revision: 194
Proposed branch: lp:~cassidyjames/elementaryos/enable-appmenu
Merge into: lp:~elementary-os/elementaryos/default-settings-loki
Diff against target: 12 lines (+1/-1)
1 file modified
debian/elementary-default-settings.gsettings-override (+1/-1)
To merge this branch: bzr merge lp:~cassidyjames/elementaryos/enable-appmenu
Reviewer Review Type Date Requested Status
elementary OS team Pending
Review via email: mp+297835@code.launchpad.net

Commit message

Enable AppMenus for GNOME apps

Description of the change

Set org.gnome.settings-daemon.plugins.xsettings overrides "{'Gtk/DialogsUseHeader': <0>, 'Gtk/ShellShowsAppMenu': <0>, 'Gtk/DecorationLayout': <'close, :menu,maximize'>}" in our overrides to enable AppMenus in GNOME apps.

To post a comment you must log in.
Revision history for this message
Danielle Foré (danrabbit) wrote :

I can confirm that this works.

As far as I can tell, you don't need the comma after "close". So you do {'Gtk/DialogsUseHeader': <0>, 'Gtk/ShellShowsAppMenu': <0>, 'Gtk/DecorationLayout': <'close:menu,maximize'>} and it works

Just as a note, if we set the button layout in org.gnome.desktop.wm.preferences it *does not* have the desired effect.

195. By Cassidy James Blaede

Remove comma and space

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/elementary-default-settings.gsettings-override'
2--- debian/elementary-default-settings.gsettings-override 2016-06-10 21:06:19 +0000
3+++ debian/elementary-default-settings.gsettings-override 2016-06-18 23:07:44 +0000
4@@ -128,7 +128,7 @@
5 [org.gnome.settings-daemon.plugins.xsettings]
6 antialiasing='rgba'
7 hinting='slight'
8-overrides={'Gtk/DialogsUseHeader': <0>}
9+overrides={'Gtk/DialogsUseHeader': <0>, 'Gtk/ShellShowsAppMenu': <0>, 'Gtk/DecorationLayout': <'close:menu,maximize'>}
10
11 [org.pantheon.desktop.gala.behavior]
12 panel-main-menu-action='wingpanel --toggle-indicator=app-launcher'

Subscribers

People subscribed via source and target branches