appearance in wayland: Support changing theme, icon theme, and fonts

*If we are in a wayland session, write the gtk theme and icon theme to both org.mate and org.gnome
*This changes wayland and xwayland themes simultaniously

*Under wayland, set the application and document fonts for both MATE and GNOME
*Otherwise some applications will ignore font changes set from the appearance capplet

*Only load GNOME interface  gsettings when running under wayland

*fail gracefully if the gnome interface schema is not present
    *Do not depend on it but use it if it is present to allow setting themes in compositors such as wayfire that use the GNOME gsettings values

5 files changed, 137 insertions, 6 deletions

diff --git a/capplets/appearance/appearance-font.c b/capplets/appearance/appearance-font.c
index 1c1d96d5..8e157676 100644
--- a/capplets/appearance/appearance-font.c
+++ b/capplets/appearance/appearance-font.c
@@ -421,20 +421,40 @@ dpi_from_pixels_and_mm (int pixels, int mm)
 }
 
 static double
-get_dpi_from_x_server (void)
+get_dpi_from_x_server_or_monitor (void)
 {
   GdkScreen  *screen;
+  GdkDisplay *display;
   double dpi;
 
   screen = gdk_screen_get_default ();
+  display = gdk_display_get_default();
 
   if (screen) {
     double width_dpi, height_dpi;
 
-    Screen *xscreen = gdk_x11_screen_get_xscreen (screen);
+    if (GDK_IS_X11_DISPLAY (display))
+    {
+      Screen *xscreen = gdk_x11_screen_get_xscreen (screen);
 
-    width_dpi = dpi_from_pixels_and_mm (WidthOfScreen (xscreen), WidthMMOfScreen (xscreen));
-    height_dpi = dpi_from_pixels_and_mm (HeightOfScreen (xscreen), HeightMMOfScreen (xscreen));
+      width_dpi = dpi_from_pixels_and_mm (WidthOfScreen (xscreen), WidthMMOfScreen (xscreen));
+      height_dpi = dpi_from_pixels_and_mm (HeightOfScreen (xscreen), HeightMMOfScreen (xscreen));
+    }
+    else
+    {
+      GdkMonitor *monitor;
+      GdkRectangle geometry = {0};
+
+      /*FIXME: we have to use the main monitor for this
+       *which may not always be the leftmost monitor
+       *Separate per-monitor settings for this would be ideal
+       */
+      monitor = gdk_display_get_monitor (display, 0);
+      gdk_monitor_get_geometry (monitor, &geometry);
+
+      width_dpi = dpi_from_pixels_and_mm (geometry.width, gdk_monitor_get_width_mm(monitor));
+      height_dpi = dpi_from_pixels_and_mm (geometry.height, gdk_monitor_get_height_mm(monitor));
+    }
 
     if (width_dpi < DPI_LOW_REASONABLE_VALUE || width_dpi > DPI_HIGH_REASONABLE_VALUE ||
         height_dpi < DPI_LOW_REASONABLE_VALUE || height_dpi > DPI_HIGH_REASONABLE_VALUE)
@@ -465,7 +485,7 @@ dpi_load (GSettings     *settings,
   dpi = g_settings_get_double (settings, FONT_DPI_KEY);
 
   if (dpi == 0)
-    dpi = get_dpi_from_x_server ();
+    dpi = get_dpi_from_x_server_or_monitor ();
 
   dpi *= (double)scale;
   dpi = CLAMP(dpi, DPI_LOW_REASONABLE_VALUE, DPI_HIGH_REASONABLE_VALUE);
@@ -781,6 +801,38 @@ void font_init(AppearanceData* data)
 			  G_CALLBACK (marco_changed),
 			  data);
 
+	/*In a wayland session we must manage MATE font settings for xwayland
+	 *and also manage GNOME font settings for native wayland applications
+	 *so if not running under x11, set the GNOME interface keys too
+	 *Ignore this if for any reason the GNOME schema was not found,
+	 *As we can only set this for compositors using either the MATE or the GNOME gsettings
+	 */
+	if (data->interface_gnome_settings)
+	{
+		widget = appearance_capplet_get_widget(data, "application_font");
+		g_settings_bind (data->interface_gnome_settings,
+				 GTK_FONT_KEY,
+				 G_OBJECT (widget),
+				 "font-name",
+				 G_SETTINGS_BIND_DEFAULT);
+
+		widget = appearance_capplet_get_widget (data, "document_font");
+		g_settings_bind (data->interface_gnome_settings,
+				 DOCUMENT_FONT_KEY,
+				 G_OBJECT (widget),
+				 "font-name",
+				 G_SETTINGS_BIND_DEFAULT);
+
+/* The monospace font seems to apply properly if and only if set only for MATE
+		widget = appearance_capplet_get_widget (data, "monospace_font");
+		g_settings_bind (data->interface_gnome_settings,
+				 MONOSPACE_FONT_KEY,
+				 G_OBJECT (widget),
+				 "font-name",
+				 G_SETTINGS_BIND_DEFAULT);
+*/
+	}
+
 	g_signal_connect (appearance_capplet_get_widget (data, "add_new_font"), "clicked", G_CALLBACK (cb_add_new_font), data);
 
 	marco_titlebar_load_sensitivity(data);
diff --git a/capplets/appearance/appearance-main.c b/capplets/appearance/appearance-main.c
index 2a9b92eb..8e19c02b 100644
--- a/capplets/appearance/appearance-main.c
+++ b/capplets/appearance/appearance-main.c
@@ -31,6 +31,7 @@
 #include "theme-thumbnail.h"
 #include "activate-settings-daemon.h"
 #include "capplet-util.h"
+#include <gdk/gdkx.h>
 
 static AppearanceData *
 init_appearance_data (int *argc, char ***argv, GOptionContext *context)
@@ -56,6 +57,21 @@ init_appearance_data (int *argc, char ***argv, GOptionContext *context)
 
   data->filechooser_settings = g_settings_new (FILECHOOSER_SCHEMA);
   data->interface_settings = g_settings_new (INTERFACE_SCHEMA);
+
+  data->interface_gnome_settings = NULL;
+  /*Load the gnome interface schema if we are running under wayland and it is present*/
+  if (!(GDK_IS_X11_DISPLAY (gdk_display_get_default())))
+  {
+    GSettingsSchemaSource *source = g_settings_schema_source_get_default ();
+
+    if (source)
+    {
+      GSettingsSchema *schema = g_settings_schema_source_lookup (source, INTERFACE_GNOME_SCHEMA, TRUE);
+
+      if (schema)
+        data->interface_gnome_settings = g_settings_new_full (schema, NULL, NULL);
+    }
+  }
   data->marco_settings = g_settings_new (MARCO_SCHEMA);
   data->mouse_settings = g_settings_new (MOUSE_SCHEMA);
   data->font_settings = g_settings_new (FONT_RENDER_SCHEMA);
diff --git a/capplets/appearance/appearance.h b/capplets/appearance/appearance.h
index 0a01ddcf..9a16afcf 100644
--- a/capplets/appearance/appearance.h
+++ b/capplets/appearance/appearance.h
@@ -58,6 +58,7 @@
 #define GTK_THEME_KEY                "gtk-theme"
 #define ICON_THEME_KEY               "icon-theme"
 #define INTERFACE_SCHEMA             "org.mate.interface"
+#define INTERFACE_GNOME_SCHEMA       "org.gnome.desktop.interface"
 #define MENU_ICONS_KEY               "menus-have-icons"
 #define MONOSPACE_FONT_KEY           "monospace-font-name"
 #define TOOLBAR_STYLE_KEY            "toolbar-style"
@@ -92,6 +93,10 @@ typedef struct {
 	GSettings* caja_settings;
 	GSettings* filechooser_settings;
 	GSettings* interface_settings;
+	/*We have to accomodate wayland theme setting here
+	 *whether we are using it or not
+	 */
+	GSettings* interface_gnome_settings;
 	GSettings* marco_settings;
 	GSettings* mouse_settings;
 	GSettings* font_settings;
diff --git a/capplets/common/mate-theme-apply.c b/capplets/common/mate-theme-apply.c
index 48b40bc0..77682cdf 100644
--- a/capplets/common/mate-theme-apply.c
+++ b/capplets/common/mate-theme-apply.c
@@ -26,8 +26,10 @@
 #include <libmate-desktop/mate-gsettings.h>
 #include "mate-theme-apply.h"
 #include "gtkrc-utils.h"
+#include <gdk/gdkx.h>
 
 #define INTERFACE_SCHEMA        "org.mate.interface"
+#define INTERFACE_GNOME_SCHEMA  "org.gnome.desktop.interface"
 #define GTK_THEME_KEY           "gtk-theme"
 #define COLOR_SCHEME_KEY        "gtk-color-scheme"
 #define ICON_THEME_KEY          "icon-theme"
@@ -48,12 +50,27 @@ void
 mate_meta_theme_set (MateThemeMetaInfo *meta_theme_info)
 {
   GSettings *interface_settings;
+  GSettings *interface_gnome_settings = NULL;
   GSettings *marco_settings;
   GSettings *mouse_settings;
   GSettings *notification_settings = NULL;
   gchar *old_key;
   gint old_key_int;
 
+  /*Load the gnome interface schema if we are running under wayland and it is present*/
+  if (!(GDK_IS_X11_DISPLAY (gdk_display_get_default())))
+  {
+    GSettingsSchemaSource *source = g_settings_schema_source_get_default ();
+
+    if (source)
+    {
+      GSettingsSchema *schema = g_settings_schema_source_lookup (source, INTERFACE_GNOME_SCHEMA, TRUE);
+
+      if (schema)
+        interface_gnome_settings = g_settings_new_full (schema, NULL, NULL);
+    }
+  }
+
   interface_settings = g_settings_new (INTERFACE_SCHEMA);
   marco_settings = g_settings_new (MARCO_SCHEMA);
   mouse_settings = g_settings_new (MOUSE_SCHEMA);
@@ -68,6 +85,11 @@ mate_meta_theme_set (MateThemeMetaInfo *meta_theme_info)
   if (compare (old_key, meta_theme_info->gtk_theme_name))
     {
       g_settings_set_string (interface_settings, GTK_THEME_KEY, meta_theme_info->gtk_theme_name);
+
+      if (interface_gnome_settings)
+          g_settings_set_string (interface_gnome_settings, 
+                                 GTK_THEME_KEY, meta_theme_info->gtk_theme_name);
+
     }
   g_free (old_key);
 
@@ -97,12 +119,17 @@ mate_meta_theme_set (MateThemeMetaInfo *meta_theme_info)
 
   /* Set the wm key */
   g_settings_set_string (marco_settings, MARCO_THEME_KEY, meta_theme_info->marco_theme_name);
+  /*To also control decoration theme in wayland we need a decorator that uses marco themes
+   *and queries gsettings to determine which theme to use to this is a TODO */
 
   /* set the icon theme */
   old_key = g_settings_get_string (interface_settings, ICON_THEME_KEY);
   if (compare (old_key, meta_theme_info->icon_theme_name))
     {
       g_settings_set_string (interface_settings, ICON_THEME_KEY, meta_theme_info->icon_theme_name);
+      if (interface_gnome_settings)
+        g_settings_set_string (interface_gnome_settings,
+                             ICON_THEME_KEY, meta_theme_info->icon_theme_name);
     }
   g_free (old_key);
 
@@ -125,6 +152,12 @@ mate_meta_theme_set (MateThemeMetaInfo *meta_theme_info)
   if (compare (old_key, meta_theme_info->cursor_theme_name))
     {
       g_settings_set_string (mouse_settings, CURSOR_THEME_KEY, meta_theme_info->cursor_theme_name);
+      if (interface_gnome_settings)
+      {
+        /*Note that this key is in a different place in GNOME than it is in MATE*/
+        g_settings_set_string (interface_gnome_settings,
+                             CURSOR_THEME_KEY, meta_theme_info->cursor_theme_name);
+      }
     }
 
   old_key_int = g_settings_get_int (mouse_settings, CURSOR_SIZE_KEY);
diff --git a/capplets/common/wm-common.c b/capplets/common/wm-common.c
index a8a31bcb..3030c071 100644
--- a/capplets/common/wm-common.c
+++ b/capplets/common/wm-common.c
@@ -30,9 +30,12 @@ wm_common_get_window_manager_property (Atom atom)
   if (wm_window == None)
     return NULL;
 
+  display = gdk_display_get_default ();
+  if (!(GDK_IS_X11_DISPLAY (display)))
+    return NULL;
+
   utf8_string = gdk_x11_get_xatom_by_name ("UTF8_STRING");
 
-  display = gdk_display_get_default ();
   gdk_x11_display_error_trap_push (display);
 
   val = NULL;
@@ -64,6 +67,9 @@ wm_common_get_window_manager_property (Atom atom)
 char*
 wm_common_get_current_window_manager (void)
 {
+  if (!(GDK_IS_X11_DISPLAY (gdk_display_get_default())))
+    return g_strdup (WM_COMMON_UNKNOWN);
+
   Atom atom = gdk_x11_get_xatom_by_name ("_NET_WM_NAME");
   char *result;
 
@@ -77,6 +83,15 @@ wm_common_get_current_window_manager (void)
 char**
 wm_common_get_current_keybindings (void)
 {
+
+  if (!(GDK_IS_X11_DISPLAY (gdk_display_get_default())))
+  {
+    /*This should never reached in wayland as compositors control
+     *and limit keybindings
+     */
+    return NULL;
+  }
+
   Atom keybindings_atom = gdk_x11_get_xatom_by_name ("_MATE_WM_KEYBINDINGS");
   char *keybindings = wm_common_get_window_manager_property (keybindings_atom);
   char **results;
@@ -115,6 +130,10 @@ update_wm_window (void)
   gulong bytes_after;
 
   display = gdk_display_get_default ();
+
+  if (!(GDK_IS_X11_DISPLAY (display)))
+    return;
+
   XGetWindowProperty (GDK_DISPLAY_XDISPLAY(display), GDK_ROOT_WINDOW (),
 		      gdk_x11_get_xatom_by_name ("_NET_SUPPORTING_WM_CHECK"),
 		      0, G_MAXLONG, False, XA_WINDOW, &type, &format,
@@ -146,6 +165,9 @@ wm_window_event_filter (GdkXEvent *xev,
 			GdkEvent  *event,
 			gpointer   data)
 {
+  if (!(GDK_IS_X11_DISPLAY (gdk_display_get_default())))
+    return GDK_FILTER_CONTINUE;
+
   WMCallbackData *ncb_data = (WMCallbackData*) data;
   XEvent *xevent = (XEvent *)xev;
 
@@ -172,6 +194,9 @@ wm_common_register_window_manager_change (GFunc    func,
 {
   WMCallbackData *ncb_data;
 
+  if (!(GDK_IS_X11_DISPLAY (gdk_display_get_default())))
+    return;
+
   ncb_data = g_new0 (WMCallbackData, 1);
 
   ncb_data->func = func;