update examples, fix styling

13 files changed, 264 insertions, 222 deletions

diff --git a/examples/js/simple-bar/style.scss b/examples/js/simple-bar/style.scss
index 1dcf729..f5f771a 100644
--- a/examples/js/simple-bar/style.scss
+++ b/examples/js/simple-bar/style.scss
@@ -13,31 +13,45 @@ window.Bar {
     font-size: 1.1em;
     font-weight: bold;
 
-    button {
-        all: unset;
-        background-color: transparent;
+    label {
+        margin: 0 8px;
+    }
+
+    .Workspaces  {
+        button {
+            all: unset;
+            background-color: transparent;
 
-        &:hover label {
-            background-color: color.adjust($fg, $alpha: -0.84);
-            border-color: color.adjust($accent, $alpha: -0.8);
+            &:hover label {
+                background-color: color.adjust($fg, $alpha: -0.84);
+                border-color: color.adjust($accent, $alpha: -0.8);
+            }
+
+            &:active label {
+                background-color: color.adjust($fg, $alpha: -0.8)
+            }
         }
 
-        &:active label {
-            background-color: color.adjust($fg, $alpha: -0.8)
+        label {
+            transition: 200ms;
+            padding: 0 8px;
+            margin: 2px;
+            border-radius: $radius;
+            border: 1pt solid transparent;
         }
-    }
 
-    label {
-        transition: 200ms;
-        padding: 0 8px;
-        margin: 2px;
-        border-radius: $radius;
-        border: 1pt solid transparent;
+        .focused label {
+            color: $accent;
+            border-color: $accent;
+        }
     }
 
-    .Workspaces .focused label {
-        color: $accent;
-        border-color: $accent;
+    .SysTray {
+        margin-right: 8px;
+
+        button {
+            padding: 0 4px;
+        }
     }
 
     .FocusedClient {
diff --git a/examples/js/simple-bar/widget/Bar.tsx b/examples/js/simple-bar/widget/Bar.tsx
index f5e96cb..87cc20d 100644
--- a/examples/js/simple-bar/widget/Bar.tsx
+++ b/examples/js/simple-bar/widget/Bar.tsx
@@ -11,17 +11,16 @@ import Tray from "gi://AstalTray"
 function SysTray() {
     const tray = Tray.get_default()
 
-    return <box>
-        {bind(tray, "items").as(items => items.map(item => {
-            
-            return <menubutton
+    return <box className="SysTray">
+        {bind(tray, "items").as(items => items.map(item => (
+            <menubutton
                 tooltipMarkup={bind(item, "tooltipMarkup")}
                 usePopover={false}
-                actionGroup={bind(item, "action-group").as(ag => { return { prefix: "dbusmenu", actionGroup: ag}})}
-                menuModel={bind(item, "menu-model")} >
+                actionGroup={bind(item, "action-group").as(ag => ["dbusmenu", ag])}
+                menuModel={bind(item, "menu-model")}>
                 <icon gIcon={bind(item, "gicon")} />
             </menubutton>
-        }))}
+        ))}
     </box>
 }
 
diff --git a/examples/lua/simple-bar/style.scss b/examples/lua/simple-bar/style.scss
index 1dcf729..f5f771a 100644
--- a/examples/lua/simple-bar/style.scss
+++ b/examples/lua/simple-bar/style.scss
@@ -13,31 +13,45 @@ window.Bar {
     font-size: 1.1em;
     font-weight: bold;
 
-    button {
-        all: unset;
-        background-color: transparent;
+    label {
+        margin: 0 8px;
+    }
+
+    .Workspaces  {
+        button {
+            all: unset;
+            background-color: transparent;
 
-        &:hover label {
-            background-color: color.adjust($fg, $alpha: -0.84);
-            border-color: color.adjust($accent, $alpha: -0.8);
+            &:hover label {
+                background-color: color.adjust($fg, $alpha: -0.84);
+                border-color: color.adjust($accent, $alpha: -0.8);
+            }
+
+            &:active label {
+                background-color: color.adjust($fg, $alpha: -0.8)
+            }
         }
 
-        &:active label {
-            background-color: color.adjust($fg, $alpha: -0.8)
+        label {
+            transition: 200ms;
+            padding: 0 8px;
+            margin: 2px;
+            border-radius: $radius;
+            border: 1pt solid transparent;
         }
-    }
 
-    label {
-        transition: 200ms;
-        padding: 0 8px;
-        margin: 2px;
-        border-radius: $radius;
-        border: 1pt solid transparent;
+        .focused label {
+            color: $accent;
+            border-color: $accent;
+        }
     }
 
-    .Workspaces .focused label {
-        color: $accent;
-        border-color: $accent;
+    .SysTray {
+        margin-right: 8px;
+
+        button {
+            padding: 0 4px;
+        }
     }
 
     .FocusedClient {
diff --git a/examples/lua/simple-bar/widget/Bar.lua b/examples/lua/simple-bar/widget/Bar.lua
index 843e292..e7bd4ff 100644
--- a/examples/lua/simple-bar/widget/Bar.lua
+++ b/examples/lua/simple-bar/widget/Bar.lua
@@ -1,8 +1,6 @@
 local astal = require("astal")
-local App = require("astal.gtk3.app")
 local Widget = require("astal.gtk3.widget")
 local Variable = astal.Variable
-local Gdk = astal.require("Gdk", "3.0")
 local GLib = astal.require("GLib")
 local bind = astal.bind
 local Mpris = astal.require("AstalMpris")
@@ -17,18 +15,16 @@ local function SysTray()
 	local tray = Tray.get_default()
 
 	return Widget.Box({
+        class_name = "SysTray",
 		bind(tray, "items"):as(function(items)
 			return map(items, function(item)
 				return Widget.MenuButton({
 					tooltip_markup = bind(item, "tooltip_markup"),
-          use_popover = false,
+					use_popover = false,
 					menu_model = bind(item, "menu-model"),
-          action_group = bind(item, "action-group"):as(function(ag)
-            return {
-              prefix = "dbusmenu",
-              action_group = ag
-            }
-          end),
+					action_group = bind(item, "action-group"):as(function(ag)
+						return { "dbusmenu", ag }
+					end),
 					Widget.Icon({
 						g_icon = bind(item, "gicon"),
 					}),
@@ -181,10 +177,10 @@ return function(gdkmonitor)
 			}),
 			Widget.Box({
 				halign = "END",
+                SysTray(),
 				Wifi(),
 				AudioSlider(),
 				BatteryLevel(),
-				SysTray(),
 				Time("%H:%M - %A %e."),
 			}),
 		}),
diff --git a/examples/py/simple-bar/app.py b/examples/py/simple-bar/app.py
index 17b6782..d95dc0e 100755
--- a/examples/py/simple-bar/app.py
+++ b/examples/py/simple-bar/app.py
@@ -1,6 +1,7 @@
 #!/usr/bin/env python3
 import sys
 import versions
+import subprocess
 from gi.repository import AstalIO, Astal, Gio
 from widget.Bar import Bar
 from pathlib import Path
@@ -18,9 +19,8 @@ class App(Astal.Application):
 
     def do_activate(self) -> None:
         self.hold()
-        AstalIO.Process.execv(["sass", scss, css])
+        subprocess.run(["sass", scss, css])
         self.apply_css(css, True)
-        print("hello")
         for mon in self.get_monitors():
             self.add_window(Bar(mon))
 
@@ -30,7 +30,7 @@ app = App(instance_name=instance_name)
 
 if __name__ == "__main__":
     try:
-        print(app.acquire_socket())
+        app.acquire_socket()
         app.run(None)
     except Exception as e:
         print(AstalIO.send_message(instance_name, "".join(sys.argv[1:])))
diff --git a/examples/py/simple-bar/style.scss b/examples/py/simple-bar/style.scss
index 1dcf729..f5f771a 100644
--- a/examples/py/simple-bar/style.scss
+++ b/examples/py/simple-bar/style.scss
@@ -13,31 +13,45 @@ window.Bar {
     font-size: 1.1em;
     font-weight: bold;
 
-    button {
-        all: unset;
-        background-color: transparent;
+    label {
+        margin: 0 8px;
+    }
+
+    .Workspaces  {
+        button {
+            all: unset;
+            background-color: transparent;
 
-        &:hover label {
-            background-color: color.adjust($fg, $alpha: -0.84);
-            border-color: color.adjust($accent, $alpha: -0.8);
+            &:hover label {
+                background-color: color.adjust($fg, $alpha: -0.84);
+                border-color: color.adjust($accent, $alpha: -0.8);
+            }
+
+            &:active label {
+                background-color: color.adjust($fg, $alpha: -0.8)
+            }
         }
 
-        &:active label {
-            background-color: color.adjust($fg, $alpha: -0.8)
+        label {
+            transition: 200ms;
+            padding: 0 8px;
+            margin: 2px;
+            border-radius: $radius;
+            border: 1pt solid transparent;
         }
-    }
 
-    label {
-        transition: 200ms;
-        padding: 0 8px;
-        margin: 2px;
-        border-radius: $radius;
-        border: 1pt solid transparent;
+        .focused label {
+            color: $accent;
+            border-color: $accent;
+        }
     }
 
-    .Workspaces .focused label {
-        color: $accent;
-        border-color: $accent;
+    .SysTray {
+        margin-right: 8px;
+
+        button {
+            padding: 0 4px;
+        }
     }
 
     .FocusedClient {
diff --git a/examples/py/simple-bar/widget/Bar.py b/examples/py/simple-bar/widget/Bar.py
index 3b09dce..ca02dd6 100644
--- a/examples/py/simple-bar/widget/Bar.py
+++ b/examples/py/simple-bar/widget/Bar.py
@@ -108,6 +108,7 @@ class Media(Gtk.Box):
 class SysTray(Gtk.Box):
     def __init__(self) -> None:
         super().__init__()
+        Astal.widget_set_class_names(self, ["SysTray"])
         self.items = {}
         tray = Tray.get_default()
         tray.connect("item_added", self.add_item)
@@ -118,33 +119,22 @@ class SysTray(Gtk.Box):
             return
 
         item = Tray.get_default().get_item(id)
-        theme = item.get_icon_theme_path()
-
-        if theme is not None:
-            from app import app
-
-            app.add_icons(theme)
-
-        menu = item.create_menu()
-        btn = Astal.Button(visible=True)
+        btn = Gtk.MenuButton(use_popover=False, visible=True)
         icon = Astal.Icon(visible=True)
 
-        def on_clicked(btn):
-            if menu:
-                menu.popup_at_widget(btn, Gdk.Gravity.SOUTH, Gdk.Gravity.NORTH, None)
+        item.bind_property("tooltip-markup", btn, "tooltip-markup", SYNC)
+        item.bind_property("gicon", icon, "g-icon", SYNC)
+        item.bind_property("menu-model", btn, "menu-model", SYNC)
+        btn.insert_action_group("dbusmenu", item.get_action_group())
 
-        def on_destroy(btn):
-            if menu:
-                menu.destroy()
+        def on_action_group(*args):
+            btn.insert_action_group("dbusmenu", item.get_action_group())
 
-        btn.connect("clicked", on_clicked)
-        btn.connect("destroy", on_destroy)
+        item.connect("notify::action-group", on_action_group)
 
-        item.bind_property("tooltip-markup", btn, "tooltip-markup", SYNC)
-        item.bind_property("gicon", icon, "gicon", SYNC)
+        btn.add(icon)
         self.add(btn)
         self.items[id] = btn
-        self.show_all()
 
     def remove_item(self, _: Tray.Tray, id: str):
         if id in self.items:
diff --git a/examples/vala/simple-bar/meson.build b/examples/vala/simple-bar/meson.build
index 10f5dd2..5a0ef4c 100644
--- a/examples/vala/simple-bar/meson.build
+++ b/examples/vala/simple-bar/meson.build
@@ -21,22 +21,15 @@ pkgconfig_deps = [
 # needed for GLib.Math
 deps = pkgconfig_deps + meson.get_compiler('c').find_library('m')
 
-custom_target(
-  'style.css',
-  command: [
-    find_program('sass'),
-    meson.project_source_root() / 'style.scss',
-    '@OUTPUT@',
-  ],
-  output: 'style.css',
-  install: true,
-  install_dir: libdir,
-)
-
 main = configure_file(
   input: 'app.in.vala',
   output: 'app.vala',
-  configuration: {'STYLE': libdir / 'style.css'},
+  configuration: {
+    'STYLE': run_command(
+      find_program('sass'),
+      meson.project_source_root() / 'style.scss',
+    ).stdout(),
+  },
 )
 
 sources = files(
diff --git a/examples/vala/simple-bar/style.scss b/examples/vala/simple-bar/style.scss
index 1dcf729..f5f771a 100644
--- a/examples/vala/simple-bar/style.scss
+++ b/examples/vala/simple-bar/style.scss
@@ -13,31 +13,45 @@ window.Bar {
     font-size: 1.1em;
     font-weight: bold;
 
-    button {
-        all: unset;
-        background-color: transparent;
+    label {
+        margin: 0 8px;
+    }
+
+    .Workspaces  {
+        button {
+            all: unset;
+            background-color: transparent;
 
-        &:hover label {
-            background-color: color.adjust($fg, $alpha: -0.84);
-            border-color: color.adjust($accent, $alpha: -0.8);
+            &:hover label {
+                background-color: color.adjust($fg, $alpha: -0.84);
+                border-color: color.adjust($accent, $alpha: -0.8);
+            }
+
+            &:active label {
+                background-color: color.adjust($fg, $alpha: -0.8)
+            }
         }
 
-        &:active label {
-            background-color: color.adjust($fg, $alpha: -0.8)
+        label {
+            transition: 200ms;
+            padding: 0 8px;
+            margin: 2px;
+            border-radius: $radius;
+            border: 1pt solid transparent;
         }
-    }
 
-    label {
-        transition: 200ms;
-        padding: 0 8px;
-        margin: 2px;
-        border-radius: $radius;
-        border: 1pt solid transparent;
+        .focused label {
+            color: $accent;
+            border-color: $accent;
+        }
     }
 
-    .Workspaces .focused label {
-        color: $accent;
-        border-color: $accent;
+    .SysTray {
+        margin-right: 8px;
+
+        button {
+            padding: 0 4px;
+        }
     }
 
     .FocusedClient {
diff --git a/examples/vala/simple-bar/widget/Bar.vala b/examples/vala/simple-bar/widget/Bar.vala
index 3893ec5..ba9f06c 100644
--- a/examples/vala/simple-bar/widget/Bar.vala
+++ b/examples/vala/simple-bar/widget/Bar.vala
@@ -103,6 +103,7 @@ class SysTray : Gtk.Box {
     AstalTray.Tray tray = AstalTray.get_default();
 
     public SysTray() {
+        Astal.widget_set_class_names(this, { "SysTray" });
         tray.item_added.connect(add_item);
         tray.item_removed.connect(remove_item);
     }
@@ -112,23 +113,19 @@ class SysTray : Gtk.Box {
             return;
 
         var item = tray.get_item(id);
-
-        var btn = new Gtk.MenuButton() {
-          use_popover = false
-        };
-        var icon = new Astal.Icon();
+        var btn = new Gtk.MenuButton() { use_popover = false, visible = true };
+        var icon = new Astal.Icon() { visible = true };
 
         item.bind_property("tooltip-markup", btn, "tooltip-markup", BindingFlags.SYNC_CREATE);
         item.bind_property("gicon", icon, "g-icon", BindingFlags.SYNC_CREATE);
         item.bind_property("menu-model", btn, "menu-model", BindingFlags.SYNC_CREATE);
         btn.insert_action_group("dbusmenu", item.action_group);
         item.notify["action-group"].connect(() => {
-          btn.insert_action_group("dbusmenu", item.action_group);
+            btn.insert_action_group("dbusmenu", item.action_group);
         });
 
         btn.add(icon);
         add(btn);
-        btn.show_all();
         items.set(id, btn);
     }
 
diff --git a/lang/gjs/src/gtk3/astalify.ts b/lang/gjs/src/gtk3/astalify.ts
index b758976..18df132 100644
--- a/lang/gjs/src/gtk3/astalify.ts
+++ b/lang/gjs/src/gtk3/astalify.ts
@@ -69,7 +69,7 @@ export default function astalify<
         get noImplicitDestroy(): boolean { return this.__no_implicit_destroy }
         set noImplicitDestroy(value: boolean) { this.__no_implicit_destroy = value }
 
-        set actionGroup(actionGroup: ActionGroup) { this.insert_action_group(actionGroup.prefix, actionGroup.actionGroup) }
+        set actionGroup([prefix, group]: ActionGroup) { this.insert_action_group(prefix, group) }
         set_action_group(actionGroup: ActionGroup) { this.actionGroup = actionGroup }
 
         _setChildren(children: Gtk.Widget[]) {
@@ -331,4 +331,4 @@ type Cursor =
     | "zoom-in"
     | "zoom-out"
 
-type ActionGroup = { prefix: string, actionGroup: Gtk.ActionGroup }
+type ActionGroup = [prefix: string, actionGroup: Gtk.ActionGroup]
diff --git a/lang/lua/astal/gtk3/widget.lua b/lang/lua/astal/gtk3/widget.lua
index adcb65f..654da29 100644
--- a/lang/lua/astal/gtk3/widget.lua
+++ b/lang/lua/astal/gtk3/widget.lua
@@ -67,7 +67,7 @@ Gtk.Widget._attribute.click_through = {
 
 Gtk.Widget._attribute.action_group = {
   set = function (self, v)
-    self:insert_action_group(v.prefix, v.action_group)
+    self:insert_action_group(v[1], v[2])
   end
 }
 
diff --git a/lib/tray/trayItem.vala b/lib/tray/trayItem.vala
index 14a2ba7..991abce 100644
--- a/lib/tray/trayItem.vala
+++ b/lib/tray/trayItem.vala
@@ -1,4 +1,3 @@
-
 namespace AstalTray {
 public struct Pixmap {
     int width;
@@ -87,21 +86,20 @@ public class TrayItem : Object {
     private IItem proxy;
     private List<ulong> connection_ids;
 
-    
     /** The Title of the TrayItem */
     public string title { owned get { return proxy.Title; } }
-    
+
     /** The category this item belongs to */
     public Category category { get { return proxy.Category; } }
-    
-    /** the current status of this item */
+
+    /** The current status of this item */
     public Status status {  get { return proxy.Status; } }
-    
-    /** the tooltip of this item */
+
+    /** The tooltip of this item */
     public Tooltip? tooltip { owned get { return proxy.ToolTip; } }
-    
-    /** 
-    * a markup representation of the tooltip. This is basically equvivalent
+
+    /**
+    * A markup representation of the tooltip. This is basically equvivalent
     * to `tooltip.title \n tooltip.description`
     */
     public string tooltip_markup {
@@ -119,26 +117,24 @@ public class TrayItem : Object {
 
     /** the id of the item. This id is specified by the tray app.*/
     public string id { owned get { return proxy.Id ;} }
-    
-    /** 
-    * If set, this only supports the menu, so showing the menu should be prefered 
+
+    /**
+    * If set, this only supports the menu, so showing the menu should be prefered
     * over calling [[email protected]].
     */
     public bool is_menu { get { return proxy.ItemIsMenu ;} }
-    
-    /** 
-    * the icon theme path, where to look for the [[email protected]:icon-name].
-    * 
-    * It is recommended to use the [[email protected]:gicon] property, 
+
+    /**
+    * The icon theme path, where to look for the [[email protected]:icon-name].
+    * It is recommended to use the [[email protected]:gicon] property,
     * which does the icon lookups for you.
     */
     public string icon_theme_path { owned get { return proxy.IconThemePath ;} }
-    
-    /** 
-    * the name of the icon. This should be looked up in the [[email protected]:icon-theme-path]
-    * if set or in the currently used icon theme otherwise. 
-    *
-    * It is recommended to use the [[email protected]:gicon] property, 
+
+    /**
+    * The name of the icon. This should be looked up in the [[email protected]:icon-theme-path]
+    * if set or in the currently used icon theme otherwise.
+    * It is recommended to use the [[email protected]:gicon] property,
     * which does the icon lookups for you.
     */
     public string icon_name {
@@ -148,40 +144,39 @@ public class TrayItem : Object {
                 : proxy.IconName;
         }
     }
-    
-    /** 
-    * a pixbuf containing the icon.
-    *
-    * It is recommended to use the [[email protected]:gicon] property, 
+
+    /**
+    * A pixbuf containing the icon.
+    * It is recommended to use the [[email protected]:gicon] property,
     * which does the icon lookups for you.
     */
     public Gdk.Pixbuf icon_pixbuf { owned get { return _get_icon_pixbuf(); } }
 
     /**
-    * contains the items icon. This property is intended to be used with the gicon property
+    * Contains the items icon. This property is intended to be used with the gicon property
     * of the Icon widget and the recommended way to display the icon.
-    * This property unifies the [[email protected]:icon-name], 
+    * This property unifies the [[email protected]:icon-name],
     * [[email protected]:icon-theme-path] and [[email protected]:icon-pixbuf] properties.
     */
     public GLib.Icon gicon { get; private set; }
-    
-    /** the id of the item used to uniquely identify the TrayItems by this lib.*/
+
+    /** The id of the item used to uniquely identify the TrayItems by this lib.*/
     public string item_id { get; private set; }
 
     private DBusMenu.Importer menu_importer;
-     
+
     public MenuModel menu_model {
-      owned get {
-        if (menu_importer == null) return null;
-        return menu_importer.model;
-      }
+        owned get {
+            if (menu_importer == null) return null;
+            return menu_importer.model;
+        }
     }
 
     public ActionGroup action_group {
-      owned get {
-        if (menu_importer == null) return null;
-        return menu_importer.action_group;
-      }
+        owned get {
+            if (menu_importer == null) return null;
+            return menu_importer.action_group;
+        }
     }
 
     public signal void changed();
@@ -198,7 +193,8 @@ public class TrayItem : Object {
             proxy = yield Bus.get_proxy(
                 BusType.SESSION,
                 service,
-                path);
+                path
+            );
 
             connection_ids.append(proxy.NewStatus.connect(refresh_all_properties));
             connection_ids.append(proxy.NewToolTip.connect(refresh_all_properties));
@@ -211,17 +207,16 @@ public class TrayItem : Object {
                         SignalHandler.disconnect(proxy, id);
                 }
             });
-           
-            if(proxy.Menu != null) {
-              this.menu_importer = new DBusMenu.Importer(proxy.get_name_owner(), proxy.Menu);
-              this.menu_importer.notify["model"].connect(() => {
-                notify_property("menu-model");
-                notify_property("action-group");
-              });
+
+            if (proxy.Menu != null) {
+                this.menu_importer = new DBusMenu.Importer(proxy.get_name_owner(), proxy.Menu);
+                this.menu_importer.notify["model"].connect(() => {
+                    notify_property("menu-model");
+                    notify_property("action-group");
+                });
             }
 
             update_gicon();
-
             ready();
         } catch (Error err) {
             critical(err.message);
@@ -229,22 +224,33 @@ public class TrayItem : Object {
     }
 
     private void _notify() {
-        string[] props = { "category", "id", "title", "status", "is-menu", "tooltip-markup", "icon-name", "icon-pixbuf" };
+        string[] props = {
+            "category",
+            "id",
+            "title",
+            "status",
+            "is-menu",
+            "tooltip-markup",
+            "icon-name",
+            "icon-pixbuf"
+        };
 
-        foreach (string prop in props)
+        foreach (string prop in props) {
             notify_property(prop);
+        }
 
         changed();
     }
 
     private void update_gicon() {
-        if(icon_name != null && icon_name != "") {
-            if(GLib.FileUtils.test(icon_name, GLib.FileTest.EXISTS)) {
+        if (icon_name != null && icon_name != "") {
+            if (GLib.FileUtils.test(icon_name, GLib.FileTest.EXISTS)) {
                 gicon = new GLib.FileIcon(GLib.File.new_for_path(icon_name));
             }
             else if(icon_theme_path != null && icon_theme_path != "") {
                 gicon = new GLib.FileIcon(GLib.File.new_for_path(
-                    find_icon_in_theme(icon_name, icon_theme_path)));
+                    find_icon_in_theme(icon_name, icon_theme_path)
+                ));
             } else {
                 gicon = new GLib.ThemedIcon(icon_name);
             }
@@ -291,33 +297,35 @@ public class TrayItem : Object {
             }
         );
     }
-    
+
     /**
-    * send an activate request to the tray app.
+    * Send an activate request to the tray app.
     */
     public void activate(int x, int y) {
         try {
             proxy.Activate(x, y);
         } catch (Error e) {
-            if(e.domain != DBusError.quark() || e.code != DBusError.UNKNOWN_METHOD)
+            if (e.domain != DBusError.quark() || e.code != DBusError.UNKNOWN_METHOD) {
                 warning(e.message);
+            }
         }
     }
 
     /**
-    * send a secondary activate request to the tray app.
+    * Send a secondary activate request to the tray app.
     */
     public void secondary_activate(int x, int y) {
         try {
             proxy.SecondaryActivate(x, y);
         } catch (Error e) {
-            if(e.domain != DBusError.quark() || e.code != DBusError.UNKNOWN_METHOD)
+            if (e.domain != DBusError.quark() || e.code != DBusError.UNKNOWN_METHOD) {
                 warning(e.message);
+            }
         }
     }
 
     /**
-    * send a scroll request to the tray app.
+    * Send a scroll request to the tray app.
     * valid values for the orientation are "horizontal" and "vertical".
     */
     public void scroll(int delta, string orientation) {
@@ -330,31 +338,34 @@ public class TrayItem : Object {
     }
 
     private string? find_icon_in_theme(string icon_name, string theme_path){
-      if(icon_name == null || theme_path == null || icon_name == "" || theme_path == "") return null;
- 
-      try {
-          Dir dir = Dir.open (theme_path, 0);
-          string? name = null;
-
-          while ((name = dir.read_name ()) != null) {
-            string path = Path.build_filename (theme_path, name);
-            
-            if (FileUtils.test (path, FileTest.IS_DIR)) {
-              string? icon = find_icon_in_theme(icon_name, path);
-              if(icon != null) return icon;
-              else continue;
-            }
+        if (icon_name == null || theme_path == null || icon_name == "" || theme_path == "") {
+            return null;
+        }
 
-            int dot_index = name.last_index_of(".");
-            if (dot_index != -1)
-                name = name.substring(0, dot_index);
-            if (name == icon_name) return path;
+        try {
+            Dir dir = Dir.open(theme_path, 0);
+            string? name = null;
+
+            while ((name = dir.read_name ()) != null) {
+                var path = Path.build_filename(theme_path, name);
+
+                if (FileUtils.test(path, FileTest.IS_DIR)) {
+                    string? icon = find_icon_in_theme(icon_name, path);
+                    if (icon != null) return icon;
+                    else continue;
+                }
+
+                int dot_index = name.last_index_of(".");
+                if (dot_index != -1) {
+                    name = name.substring(0, dot_index);
+                }
 
-          }
-      } catch (FileError err) {
-          return null;
-      }
-      return null;
+                if (name == icon_name) return path;
+            }
+        } catch (FileError err) {
+            return null;
+        }
+        return null;
 
     }
 
@@ -371,8 +382,8 @@ public class TrayItem : Object {
             return null;
 
         Pixmap pixmap = pixmaps[0];
-        
-        for(int i = 0; i < pixmaps.length; i++){
+
+        for (int i = 0; i < pixmaps.length; i++){
             if(pixmap.width < pixmaps[i].width)
               pixmap = pixmaps[i];
         };