Merge branch 'tari01-pr/use-ayatana-a11y'

Attributes GH PR #52: https://github.com/ArcticaProject/arctica-greeter/pull/52

3 files changed, 164 insertions, 280 deletions

diff --git a/src/arctica-greeter.vala b/src/arctica-greeter.vala
index 81f0045..245c1dc 100644
--- a/src/arctica-greeter.vala
+++ b/src/arctica-greeter.vala
@@ -30,7 +30,8 @@ public class ArcticaGreeter : Object
     public signal void show_prompt (string text, LightDM.PromptType type);
     public signal void authentication_complete ();
     public signal void starting_session ();
-
+    public MainWindow main_window { get; private set; default = null; }
+    public Gtk.Window? pKeyboardWindow { get; set; default = null; }
     public bool test_mode { get; construct; default = false; }
     public bool test_highcontrast { get; construct; default = false; }
     public bool test_bigfont { get; construct; default = false; }
@@ -42,7 +43,6 @@ public class ArcticaGreeter : Object
     private SettingsDaemon settings_daemon;
 
     public bool orca_needs_kick;
-    private MainWindow main_window;
 
     private LightDM.Greeter greeter;
 
@@ -370,8 +370,6 @@ public class ArcticaGreeter : Object
         var _session = session;
         background_surface.set_device_scale (scale, scale);
 
-        main_window.before_session_start();
-
         if (test_mode)
         {
             debug ("Successfully logged in! Quitting...");
@@ -485,7 +483,6 @@ public class ArcticaGreeter : Object
         main_window.setup_window ();
         main_window.show ();
         main_window.get_window ().focus (Gdk.CURRENT_TIME);
-        main_window.set_keyboard_state ();
     }
 
     public bool is_authenticated ()
@@ -669,9 +666,9 @@ public class ArcticaGreeter : Object
             {
                 /* Check to see if this window is our onboard window, since we don't want to focus it. */
                 X.Window keyboard_xid = 0;
-                if (main_window.menubar.keyboard_window != null)
+                if (this.pKeyboardWindow != null)
                 {
-                    Gdk.X11.Window pWindow = (Gdk.X11.Window) main_window.menubar.keyboard_window.get_window ();
+                    Gdk.X11.Window pWindow = (Gdk.X11.Window) this.pKeyboardWindow.get_window ();
                     keyboard_xid = pWindow.get_xid ();
                 }
 
@@ -680,8 +677,8 @@ public class ArcticaGreeter : Object
                     win.focus (Gdk.CURRENT_TIME);
 
                     /* Make sure to keep keyboard above */
-                    if (main_window.menubar.keyboard_window != null)
-                        main_window.menubar.keyboard_window.get_window ().raise ();
+                    if (this.pKeyboardWindow != null)
+                        this.pKeyboardWindow.get_window ().raise ();
                 }
             }
         }
@@ -704,8 +701,8 @@ public class ArcticaGreeter : Object
                 main_window.get_window ().focus (Gdk.CURRENT_TIME);
 
                 /* Make sure to keep keyboard above */
-                if (main_window.menubar.keyboard_window != null)
-                    main_window.menubar.keyboard_window.get_window ().raise ();
+                if (this.pKeyboardWindow != null)
+                    this.pKeyboardWindow.get_window ().raise ();
             }
         }
         return Gdk.FilterReturn.CONTINUE;
@@ -1340,11 +1337,31 @@ public class DBusServer : Object
 {
     private DBusConnection pConnection;
     private ArcticaGreeter pGreeter;
+    private Pid nOrca = 0;
+    private Pid nOnBoard = 0;
+
+    private void closePid (ref Pid nPid)
+    {
+        if (nPid > 0)
+        {
+            Posix.kill (nPid, Posix.Signal.TERM);
+            int nStatus;
+            Posix.waitpid (nPid, out nStatus, 0);
+            nPid = 0;
+        }
+    }
+
+    private void cleanup ()
+    {
+        closePid (ref nOnBoard);
+        closePid (ref nOrca);
+    }
 
     public DBusServer (DBusConnection pConnection, ArcticaGreeter pGreeter)
     {
         this.pConnection = pConnection;
         this.pGreeter = pGreeter;
+        this.pGreeter.starting_session.connect (cleanup);
     }
 
     public void sendUserChange (string sUser) throws GLib.DBusError, GLib.IOError
@@ -1386,4 +1403,127 @@ public class DBusServer : Object
             error ("Panic: Could not set keyboard layout: %s", pError.message);
         }
     }
+
+    public void ToggleOnBoard (bool bActive) throws GLib.DBusError, GLib.IOError
+    {
+        AGSettings.set_boolean (AGSettings.KEY_ONSCREEN_KEYBOARD, bActive);
+
+        if (this.pGreeter.pKeyboardWindow == null)
+        {
+            int nId = 0;
+
+            try
+            {
+                var sLayout = AGSettings.get_string (AGSettings.KEY_ONSCREEN_KEYBOARD_LAYOUT);
+                var sLayoutPath = "/usr/share/onboard/layouts/%s.onboard".printf (sLayout);
+                var pLayoutFile = File.new_for_path (sLayoutPath);
+                var sTheme = AGSettings.get_string (AGSettings.KEY_ONSCREEN_KEYBOARD_THEME);
+                var sThemePath = "/usr/share/onboard/themes/%s.theme".printf (sTheme);
+                var pThemeFile = File.new_for_path (sThemePath);
+                var sLayoutArgs = "";
+
+                if (pLayoutFile.query_exists ())
+                {
+                    sLayoutArgs = "--layout='%s'".printf (sLayoutPath);
+                }
+
+                var sThemeArgs = "";
+
+                if (pThemeFile.query_exists ())
+                {
+                    sThemeArgs  = "--theme='%s'".printf (sThemePath);
+                }
+
+                string sCommand = "onboard --xid %s %s".printf (sLayoutArgs, sThemeArgs);
+                string[] lArgs;
+                Shell.parse_argv (sCommand, out lArgs);
+                int nOnboardFD;
+                Process.spawn_async_with_pipes (null, lArgs, null, SpawnFlags.SEARCH_PATH, null, out nOnBoard, null, out nOnboardFD, null);
+                var pFile = FileStream.fdopen (nOnboardFD, "r");
+                var sText = new char[1024];
+
+                if (pFile.gets (sText) != null)
+                {
+                    nId = int.parse ((string) sText);
+                }
+
+            }
+            catch (Error pError)
+            {
+                warning ("Error setting up keyboard: %s", pError.message);
+
+                return;
+            }
+
+            var pSocket = new Gtk.Socket ();
+            pSocket.show ();
+            this.pGreeter.pKeyboardWindow = new Gtk.Window ();
+            this.pGreeter.pKeyboardWindow.accept_focus = false;
+            this.pGreeter.pKeyboardWindow.focus_on_map = false;
+            this.pGreeter.pKeyboardWindow.add (pSocket);
+            pSocket.add_id (nId);
+
+            var pDisplay = this.pGreeter.main_window.get_display ();
+            var pMonitor = pDisplay.get_monitor_at_window (this.pGreeter.main_window.get_window ());
+            Gdk.Rectangle cRect = pMonitor.get_geometry ();
+            this.pGreeter.pKeyboardWindow.move (cRect.x, cRect.y + cRect.height - 200);
+            this.pGreeter.pKeyboardWindow.resize (cRect.width, 200);
+        }
+
+        this.pGreeter.pKeyboardWindow.visible = bActive;
+    }
+
+    public void ToggleOrca (bool bActive) throws GLib.DBusError, GLib.IOError
+    {
+        AGSettings.set_boolean (AGSettings.KEY_SCREEN_READER, bActive);
+
+        if (bActive)
+        {
+            try
+            {
+                string[] lArgs;
+                Shell.parse_argv ("orca --replace --no-setup --disable splash-window,", out lArgs);
+                Process.spawn_async (null, lArgs, null, SpawnFlags.SEARCH_PATH, null, out nOrca);
+
+                /*
+                This is a workaround for bug https://launchpad.net/bugs/944159
+                The problem is that orca seems to not notice that it's in a
+                password field on startup.  We just need to kick orca in the
+                pants.  We do this two ways:  a racy way and a non-racy way.
+                We kick it after a second which is ideal if we win the race,
+                because the user gets to hear what widget they are in, and
+                the first character will be masked.  Otherwise, if we lose
+                that race, the first time the user types (see
+                DashEntry.key_press_event), we will kick orca again.  While
+                this is not racy with orca startup, it is racy with whether
+                orca will read the first character or not out loud.  Hence
+                why we do both.  Ideally this would be fixed in orca itself.
+                */
+                var pGreeter = new ArcticaGreeter ();
+                pGreeter.orca_needs_kick = true;
+
+                Timeout.add_seconds (1, () =>
+                {
+                    Gtk.Window pWindow = (Gtk.Window) this.pGreeter.main_window.get_toplevel ();
+                    Signal.emit_by_name (pWindow.get_focus ().get_accessible (), "focus-event", true);
+
+                    return false;
+                });
+            }
+            catch (Error pError)
+            {
+                warning ("Failed to run Orca: %s", pError.message);
+            }
+        }
+        else
+        {
+            closePid (ref nOrca);
+        }
+    }
+
+    public void ToggleHighContrast (bool bActive) throws GLib.DBusError, GLib.IOError
+    {
+        var agsettings = new AGSettings ();
+        agsettings.high_contrast = bActive;
+    }
 }
diff --git a/src/main-window.vala b/src/main-window.vala
index 5e500af..943ad25 100644
--- a/src/main-window.vala
+++ b/src/main-window.vala
@@ -2,6 +2,7 @@
  *
  * Copyright (C) 2011,2012 Canonical Ltd
  * Copyright (C) 2015-2017 Mike Gabriel <mike.gabriel@das-netzwerkteam.de>
+ * Copyright (C) 2023 Robert Tari
  *
  * This program is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
@@ -18,6 +19,7 @@
  * Authors: Robert Ancell <robert.ancell@canonical.com>
  *          Michael Terry <michael.terry@canonical.com>
  *          Mike Gabriel <mike.gabriel@das-netzwerkteam.de>
+ *          Robert Tari <robert@tari.in>
  */
 
 public class MainWindow : Gtk.Window
@@ -208,12 +210,6 @@ public class MainWindow : Gtk.Window
         }
     }
 
-    public void before_session_start()
-    {
-        debug ("Cleaning up menu bar related processes (i.e. orca, onboard, etc.)");
-        menubar.cleanup();
-    }
-
     /* Setup the size and position of the window */
     public void setup_window ()
     {
@@ -453,11 +449,6 @@ public class MainWindow : Gtk.Window
         return base.key_press_event (event);
     }
 
-    public void set_keyboard_state ()
-    {
-        menubar.set_keyboard_state ();
-    }
-
     public void show_shutdown_dialog (ShutdownDialogType type)
     {
         if (shutdown_dialog != null)
diff --git a/src/menubar.vala b/src/menubar.vala
index 5c6fa09..b9772fa 100644
--- a/src/menubar.vala
+++ b/src/menubar.vala
@@ -154,23 +154,7 @@ public class MenuBar : Gtk.MenuBar
         ctx.add_class ("osd");
     }
 
-    /* Due to LP #973922 the keyboard has to be loaded after the main window
-     * is shown and given focus. Therefore we don't enable the active state
-     * until now.
-     */
-    public void set_keyboard_state ()
-    {
-        var greeter = new ArcticaGreeter ();
-        if (!greeter.test_mode)
-            onscreen_keyboard_item.set_active (AGSettings.get_boolean (AGSettings.KEY_ONSCREEN_KEYBOARD));
-    }
-
     private List<Indicator.Object> indicator_objects;
-    private Gtk.CheckMenuItem high_contrast_item;
-    private Gtk.CheckMenuItem big_font_item;
-    private Pid keyboard_pid = 0;
-    private Pid reader_pid = 0;
-    private Gtk.CheckMenuItem onscreen_keyboard_item;
 
     construct
     {
@@ -296,30 +280,6 @@ public class MenuBar : Gtk.MenuBar
         }
 
         setup_indicators ();
-
-        var greeter = new ArcticaGreeter ();
-        greeter.starting_session.connect (cleanup);
-    }
-
-    private void close_pid (ref Pid pid)
-    {
-        if (pid > 0)
-        {
-#if VALA_0_40
-            Posix.kill (pid, Posix.Signal.TERM);
-#else
-            Posix.kill (pid, Posix.SIGTERM);
-#endif
-            int status;
-            Posix.waitpid (pid, out status, 0);
-            pid = 0;
-        }
-    }
-
-    public void cleanup ()
-    {
-        close_pid (ref keyboard_pid);
-        close_pid (ref reader_pid);
     }
 
     public override void get_preferred_height (out int min, out int nat)
@@ -355,52 +315,8 @@ public class MenuBar : Gtk.MenuBar
         }
     }
 
-    private Gtk.Widget make_a11y_indicator ()
-    {
-        var a11y_item = new Gtk.MenuItem ();
-        var hbox = new Gtk.Box (Gtk.Orientation.HORIZONTAL, 3);
-        hbox.show ();
-        a11y_item.add (hbox);
-        var image = new Gtk.Image.from_file (Path.build_filename (Config.PKGDATADIR, "a11y.svg"));
-        image.show ();
-        hbox.add (image);
-        a11y_item.show ();
-        a11y_item.set_submenu (new Gtk.Menu ());
-        onscreen_keyboard_item = new Gtk.CheckMenuItem.with_label (_("Onscreen keyboard"));
-        onscreen_keyboard_item.toggled.connect (keyboard_toggled_cb);
-        onscreen_keyboard_item.show ();
-        unowned Gtk.Menu submenu = a11y_item.submenu;
-        submenu.append (onscreen_keyboard_item);
-        high_contrast_item = new Gtk.CheckMenuItem.with_label (_("High Contrast"));
-        high_contrast_item.toggled.connect (high_contrast_toggled_cb);
-        high_contrast_item.add_accelerator ("activate", accel_group, Gdk.Key.h, Gdk.ModifierType.CONTROL_MASK, Gtk.AccelFlags.VISIBLE);
-        high_contrast_item.show ();
-        submenu.append (high_contrast_item);
-        var agsettings = new AGSettings ();
-        debug ("Initializing high contrast menu item to state %s", agsettings.high_contrast.to_string ());
-        high_contrast_item.set_active (agsettings.high_contrast);
-
-/* Hide the Big Font feature until it's available.
-        big_font_item = new Gtk.CheckMenuItem.with_label (_("Big Font"));
-        big_font_item.toggled.connect (big_font_toggled_cb);
-        big_font_item.add_accelerator ("activate", accel_group, Gdk.Key.b, Gdk.ModifierType.CONTROL_MASK, Gtk.AccelFlags.VISIBLE);
-        big_font_item.show ();
-        submenu.append (big_font_item);
-*/
-
-        big_font_item.set_active (agsettings.big_font);
-        var item = new Gtk.CheckMenuItem.with_label (_("Screen Reader"));
-        item.toggled.connect (screen_reader_toggled_cb);
-        item.add_accelerator ("activate", accel_group, Gdk.Key.s, Gdk.ModifierType.SUPER_MASK | Gdk.ModifierType.MOD1_MASK, Gtk.AccelFlags.VISIBLE);
-        item.show ();
-        submenu.append (item);
-        item.set_active (AGSettings.get_boolean (AGSettings.KEY_SCREEN_READER));
-        return a11y_item;
-    }
-
     private Indicator.Object? load_indicator_file (string indicator_name)
     {
-
         string dir = Config.INDICATOR_FILE_DIR;
         string path;
         Indicator.Object io;
@@ -451,26 +367,18 @@ public class MenuBar : Gtk.MenuBar
         var greeter = new ArcticaGreeter ();
         if (!greeter.test_mode)
         {
-            if (indicator_name == "ug-accessibility")
-            {
-                var a11y_item = make_a11y_indicator ();
-                insert (a11y_item, (int) get_children ().length () - 1);
-            }
-            else
+            var io = load_indicator_file (indicator_name);
+
+            if (io == null)
+                io = load_indicator_library (indicator_name);
+
+            if (io != null)
             {
-                var io = load_indicator_file (indicator_name);
-
-                if (io == null)
-                    io = load_indicator_library (indicator_name);
-
-                if (io != null)
-                {
-                    indicator_objects.append (io);
-                    io.entry_added.connect (indicator_added_cb);
-                    io.entry_removed.connect (indicator_removed_cb);
-                    foreach (var entry in io.get_entries ())
-                        indicator_added_cb (io, entry);
-                }
+                indicator_objects.append (io);
+                io.entry_added.connect (indicator_added_cb);
+                io.entry_removed.connect (indicator_removed_cb);
+                foreach (var entry in io.get_entries ())
+                    indicator_added_cb (io, entry);
             }
         }
     }
@@ -502,19 +410,6 @@ public class MenuBar : Gtk.MenuBar
 
         var indicator_list = AGSettings.get_strv(AGSettings.KEY_INDICATORS);
 
-        var update_indicator_list = false;
-        for (var i = 0; i < indicator_list.length; i++)
-        {
-            if (indicator_list[i] == "ug-keyboard")
-            {
-                indicator_list[i] = "org.ayatana.indicator.keyboard";
-                update_indicator_list = true;
-            }
-        }
-
-        if (update_indicator_list)
-            AGSettings.set_strv(AGSettings.KEY_INDICATORS, indicator_list);
-
         foreach (var indicator in indicator_list)
             load_indicator(indicator);
 
@@ -533,148 +428,6 @@ public class MenuBar : Gtk.MenuBar
         debug ("LANG=%s LANGUAGE=%s", Environment.get_variable ("LANG"), Environment.get_variable ("LANGUAGE"));
     }
 
-    private void keyboard_toggled_cb (Gtk.CheckMenuItem item)
-    {
-        /* FIXME: The below would be sufficient if gnome-session were running
-         * to notice and run a screen keyboard in /etc/xdg/autostart...  But
-         * since we're not running gnome-session, we hardcode onboard here. */
-        /* var settings = new Settings ("org.gnome.desktop.a11y.applications");*/
-        /*settings.set_boolean ("screen-keyboard-enabled", item.active);*/
-
-        AGSettings.set_boolean (AGSettings.KEY_ONSCREEN_KEYBOARD, item.active);
-
-        if (keyboard_window == null)
-        {
-            int id = 0;
-
-            try
-            {
-                string[] argv;
-                string cmd;
-                int onboard_stdout_fd;
-                var arg_layout   = "";
-                var arg_theme    = "";
-                var layout       = AGSettings.get_string (AGSettings.KEY_ONSCREEN_KEYBOARD_LAYOUT);
-                var theme        = AGSettings.get_string (AGSettings.KEY_ONSCREEN_KEYBOARD_THEME);
-                var fname_layout = "/usr/share/onboard/layouts/%s.onboard".printf (layout);
-                var fname_theme  = "/usr/share/onboard/themes/%s.theme".printf (theme);
-                var file_layout  = File.new_for_path (fname_layout);
-                var file_theme   = File.new_for_path (fname_theme);
-                if (file_layout.query_exists ()) {
-                    arg_layout  = "--layout='%s'".printf (fname_layout);
-                }
-                if (file_theme.query_exists ()) {
-                    arg_theme  = "--theme='%s'".printf (fname_theme);
-                }
-                cmd = "onboard --xid %s %s".printf (arg_layout, arg_theme);
-                Shell.parse_argv (cmd, out argv);
-                Process.spawn_async_with_pipes (null,
-                                                argv,
-                                                null,
-                                                SpawnFlags.SEARCH_PATH,
-                                                null,
-                                                out keyboard_pid,
-                                                null,
-                                                out onboard_stdout_fd,
-                                                null);
-                var f = FileStream.fdopen (onboard_stdout_fd, "r");
-                var stdout_text = new char[1024];
-                if (f.gets (stdout_text) != null)
-                    id = int.parse ((string) stdout_text);
-
-            }
-            catch (Error e)
-            {
-                warning ("Error setting up keyboard: %s", e.message);
-                return;
-            }
-
-            var keyboard_socket = new Gtk.Socket ();
-            keyboard_socket.show ();
-            keyboard_window = new Gtk.Window ();
-            keyboard_window.accept_focus = false;
-            keyboard_window.focus_on_map = false;
-            keyboard_window.add (keyboard_socket);
-            keyboard_socket.add_id (id);
-
-            /* Put keyboard at the bottom of the screen */
-            var display = get_display ();
-            var monitor = display.get_monitor_at_window (get_window ());
-            Gdk.Rectangle geom;
-            geom = monitor.get_geometry ();
-            keyboard_window.move (geom.x, geom.y + geom.height - 200);
-            keyboard_window.resize (geom.width, 200);
-        }
-
-        keyboard_window.visible = item.active;
-    }
-
-    private void high_contrast_toggled_cb (Gtk.CheckMenuItem item)
-    {
-        var agsettings = new AGSettings ();
-        agsettings.high_contrast = item.active;
-    }
-
-    /*private void big_font_toggled_cb (Gtk.CheckMenuItem item)
-    {
-        var agsettings = new AGSettings ();
-        agsettings.big_font = item.active;
-    }*/
-
-    private void screen_reader_toggled_cb (Gtk.CheckMenuItem item)
-    {
-        /* FIXME: The below would be sufficient if gnome-session were running
-         * to notice and run a screen reader in /etc/xdg/autostart...  But
-         * since we're not running gnome-session, we hardcode orca here.
-        /*var settings = new Settings ("org.gnome.desktop.a11y.applications");*/
-        /*settings.set_boolean ("screen-reader-enabled", item.active);*/
-
-        AGSettings.set_boolean (AGSettings.KEY_SCREEN_READER, item.active);
-
-        /* Hardcoded orca: */
-        if (item.active)
-        {
-            try
-            {
-                string[] argv;
-                Shell.parse_argv ("orca --replace --no-setup --disable splash-window,", out argv);
-                Process.spawn_async (null,
-                                     argv,
-                                     null,
-                                     SpawnFlags.SEARCH_PATH,
-                                     null,
-                                     out reader_pid);
-                // This is a workaround for bug https://launchpad.net/bugs/944159
-                // The problem is that orca seems to not notice that it's in a
-                // password field on startup.  We just need to kick orca in the
-                // pants.  We do this two ways:  a racy way and a non-racy way.
-                // We kick it after a second which is ideal if we win the race,
-                // because the user gets to hear what widget they are in, and
-                // the first character will be masked.  Otherwise, if we lose
-                // that race, the first time the user types (see
-                // DashEntry.key_press_event), we will kick orca again.  While
-                // this is not racy with orca startup, it is racy with whether
-                // orca will read the first character or not out loud.  Hence
-                // why we do both.  Ideally this would be fixed in orca itself.
-                var greeter = new ArcticaGreeter ();
-                greeter.orca_needs_kick = true;
-                Timeout.add_seconds (1, () =>
-                {
-                    Gtk.Window pWindow = (Gtk.Window) get_toplevel ();
-                    Signal.emit_by_name (pWindow.get_focus ().get_accessible (), "focus-event", true);
-
-                    return false;
-                });
-            }
-            catch (Error e)
-            {
-                warning ("Failed to run Orca: %s", e.message);
-            }
-        }
-        else
-            close_pid (ref reader_pid);
-    }
-
     private uint get_indicator_index (Indicator.Object object)
     {
         uint index = 0;