swaybar: implement tray config

12 files changed, 207 insertions, 52 deletions

diff --git a/sway/commands/bar.c b/sway/commands/bar.c
index 0cf94907..507ee10a 100644
--- a/sway/commands/bar.c
+++ b/sway/commands/bar.c
@@ -8,11 +8,9 @@
 
 // Must be in alphabetical order for bsearch
 static struct cmd_handler bar_handlers[] = {
-        { "activate_button", bar_cmd_activate_button },
         { "binding_mode_indicator", bar_cmd_binding_mode_indicator },
         { "bindsym", bar_cmd_bindsym },
         { "colors", bar_cmd_colors },
-        { "context_button", bar_cmd_context_button },
         { "font", bar_cmd_font },
         { "gaps", bar_cmd_gaps },
         { "height", bar_cmd_height },
@@ -23,11 +21,11 @@ static struct cmd_handler bar_handlers[] = {
         { "output", bar_cmd_output },
         { "pango_markup", bar_cmd_pango_markup },
         { "position", bar_cmd_position },
-        { "secondary_button", bar_cmd_secondary_button },
         { "separator_symbol", bar_cmd_separator_symbol },
         { "status_command", bar_cmd_status_command },
         { "strip_workspace_name", bar_cmd_strip_workspace_name },
         { "strip_workspace_numbers", bar_cmd_strip_workspace_numbers },
+        { "tray_bindsym", bar_cmd_tray_bindsym },
         { "tray_output", bar_cmd_tray_output },
         { "tray_padding", bar_cmd_tray_padding },
         { "workspace_buttons", bar_cmd_workspace_buttons },
diff --git a/sway/commands/bar/activate_button.c b/sway/commands/bar/activate_button.c
deleted file mode 100644
index 7310e7ec..00000000
--- a/sway/commands/bar/activate_button.c
+++ /dev/null
@@ -1,8 +0,0 @@
-#include <stdlib.h>
-#include "sway/commands.h"
-#include "log.h"
-
-struct cmd_results *bar_cmd_activate_button(int argc, char **argv) {
-        // TODO TRAY
-        return cmd_results_new(CMD_INVALID, "activate_button", "TODO TRAY");
-}
diff --git a/sway/commands/bar/context_button.c b/sway/commands/bar/context_button.c
deleted file mode 100644
index 3b76885a..00000000
--- a/sway/commands/bar/context_button.c
+++ /dev/null
@@ -1,8 +0,0 @@
-#include <stdlib.h>
-#include "sway/commands.h"
-#include "log.h"
-
-struct cmd_results *bar_cmd_context_button(int argc, char **argv) {
-        // TODO TRAY
-        return cmd_results_new(CMD_INVALID, "context_button", "TODO TRAY");
-}
diff --git a/sway/commands/bar/icon_theme.c b/sway/commands/bar/icon_theme.c
index 0e30409b..9d3b6040 100644
--- a/sway/commands/bar/icon_theme.c
+++ b/sway/commands/bar/icon_theme.c
@@ -1,7 +1,28 @@
+#define _POSIX_C_SOURCE 200809L
 #include <string.h>
+#include "config.h"
 #include "sway/commands.h"
+#include "sway/config.h"
+#include "log.h"
 
 struct cmd_results *bar_cmd_icon_theme(int argc, char **argv) {
-        // TODO TRAY
-        return cmd_results_new(CMD_INVALID, "icon_theme", "TODO TRAY");
+#if HAVE_TRAY
+        struct cmd_results *error = NULL;
+        if ((error = checkarg(argc, "icon_theme", EXPECTED_EQUAL_TO, 1))) {
+                return error;
+        }
+
+        if (!config->current_bar) {
+                return cmd_results_new(CMD_FAILURE, "tray_padding", "No bar defined.");
+        }
+
+        wlr_log(WLR_DEBUG, "[Bar %s] Setting icon theme to %s",
+                        config->current_bar->id, argv[0]);
+        free(config->current_bar->icon_theme);
+        config->current_bar->icon_theme = strdup(argv[0]);
+        return cmd_results_new(CMD_SUCCESS, NULL, NULL);
+#else
+        return cmd_results_new(CMD_INVALID, "icon_theme",
+                        "Sway has been compiled without tray support");
+#endif
 }
diff --git a/sway/commands/bar/secondary_button.c b/sway/commands/bar/secondary_button.c
deleted file mode 100644
index 449124cb..00000000
--- a/sway/commands/bar/secondary_button.c
+++ /dev/null
@@ -1,8 +0,0 @@
-#include <stdlib.h>
-#include "sway/commands.h"
-#include "log.h"
-
-struct cmd_results *bar_cmd_secondary_button(int argc, char **argv) {
-        // TODO TRAY
-        return cmd_results_new(CMD_INVALID, "secondary_button", "TODO TRAY");
-}
diff --git a/sway/commands/bar/tray_bindsym.c b/sway/commands/bar/tray_bindsym.c
new file mode 100644
index 00000000..ad413446
--- /dev/null
+++ b/sway/commands/bar/tray_bindsym.c
@@ -0,0 +1,55 @@
+#include <strings.h>
+#include "config.h"
+#include "sway/commands.h"
+#include "sway/config.h"
+#include "log.h"
+
+struct cmd_results *bar_cmd_tray_bindsym(int argc, char **argv) {
+#if HAVE_TRAY
+        struct cmd_results *error = NULL;
+        if ((error = checkarg(argc, "tray_bindsym", EXPECTED_EQUAL_TO, 2))) {
+                return error;
+        }
+
+        if (!config->current_bar) {
+                return cmd_results_new(CMD_FAILURE, "tray_bindsym", "No bar defined.");
+        }
+
+        int button = 0;
+        if (strncasecmp(argv[0], "button", strlen("button")) == 0 &&
+                        strlen(argv[0]) == strlen("button0")) {
+                button = argv[0][strlen("button")] - '0';
+        }
+        if (button < 1 || button > 9) {
+                return cmd_results_new(CMD_FAILURE, "tray_bindsym",
+                                "[Bar %s] Only buttons 1 to 9 are supported",
+                                config->current_bar->id);
+        }
+
+        static const char *commands[] = {
+                "ContextMenu",
+                "Activate",
+                "SecondaryActivate",
+                "ScrollDown",
+                "ScrollLeft",
+                "ScrollRight",
+                "ScrollUp",
+                "nop"
+        };
+
+        for (size_t i = 0; i < sizeof(commands) / sizeof(commands[0]); ++i) {
+                if (strcasecmp(argv[1], commands[i]) == 0) {
+                        wlr_log(WLR_DEBUG, "[Bar %s] Binding button %d to %s",
+                                        config->current_bar->id, button, commands[i]);
+                        config->current_bar->tray_bindings[button] = commands[i];
+                        return cmd_results_new(CMD_SUCCESS, NULL, NULL);
+                }
+        }
+
+        return cmd_results_new(CMD_INVALID, "tray_bindsym",
+                        "[Bar %s] Invalid command %s", config->current_bar->id, argv[1]);
+#else
+        return cmd_results_new(CMD_INVALID, "tray_bindsym",
+                        "Sway has been compiled without tray support");
+#endif
+}
diff --git a/sway/commands/bar/tray_output.c b/sway/commands/bar/tray_output.c
index e6c77128..19ecc5c1 100644
--- a/sway/commands/bar/tray_output.c
+++ b/sway/commands/bar/tray_output.c
@@ -1,7 +1,42 @@
+#define _POSIX_C_SOURCE 200809L
 #include <string.h>
+#include "config.h"
 #include "sway/commands.h"
+#include "sway/config.h"
+#include "list.h"
+#include "log.h"
 
 struct cmd_results *bar_cmd_tray_output(int argc, char **argv) {
-        // TODO TRAY
-        return cmd_results_new(CMD_INVALID, "tray_output", "TODO TRAY");
+#if HAVE_TRAY
+        struct cmd_results *error = NULL;
+        if ((error = checkarg(argc, "tray_output", EXPECTED_EQUAL_TO, 1))) {
+                return error;
+        }
+
+        if (!config->current_bar) {
+                return cmd_results_new(CMD_FAILURE, "tray_output", "No bar defined.");
+        }
+
+        list_t *outputs = config->current_bar->tray_outputs;
+        if (!outputs) {
+                config->current_bar->tray_outputs = outputs = create_list();
+        }
+
+        if (strcmp(argv[0], "none") == 0) {
+                wlr_log(WLR_DEBUG, "Hiding tray on bar: %s", config->current_bar->id);
+                for (int i = 0; i < outputs->length; ++i) {
+                        free(outputs->items[i]);
+                }
+                outputs->length = 0;
+        } else {
+                wlr_log(WLR_DEBUG, "Showing tray on output '%s' for bar: %s", argv[0],
+                                config->current_bar->id);
+                list_add(outputs, strdup(argv[0]));
+        }
+
+        return cmd_results_new(CMD_SUCCESS, NULL, NULL);
+#else
+        return cmd_results_new(CMD_INVALID, "tray_output",
+                        "Sway has been compiled without tray support");
+#endif
 }
diff --git a/sway/commands/bar/tray_padding.c b/sway/commands/bar/tray_padding.c
index 91c56f19..eb795b00 100644
--- a/sway/commands/bar/tray_padding.c
+++ b/sway/commands/bar/tray_padding.c
@@ -1,9 +1,42 @@
 #include <stdlib.h>
 #include <strings.h>
+#include "config.h"
 #include "sway/commands.h"
+#include "sway/config.h"
 #include "log.h"
 
 struct cmd_results *bar_cmd_tray_padding(int argc, char **argv) {
-        // TODO TRAY
-        return cmd_results_new(CMD_INVALID, "tray_padding", "TODO TRAY");
+#if HAVE_TRAY
+        struct cmd_results *error = NULL;
+        if ((error = checkarg(argc, "tray_padding", EXPECTED_AT_LEAST, 1))) {
+                return error;
+        }
+        if ((error = checkarg(argc, "tray_padding", EXPECTED_AT_MOST, 2))) {
+                return error;
+        }
+
+        if (!config->current_bar) {
+                return cmd_results_new(CMD_FAILURE, "tray_padding", "No bar defined.");
+        }
+        struct bar_config *bar = config->current_bar;
+
+        char *end;
+        int padding = strtol(argv[0], &end, 10);
+        if (padding < 0 || (*end != '\0' && strcasecmp(end, "px") != 0)) {
+                return cmd_results_new(CMD_INVALID, "tray_padding",
+                                "[Bar %s] Invalid tray padding value: %s", bar->id, argv[0]);
+        }
+
+        if (argc == 2 && strcasecmp(argv[1], "px") != 0) {
+                return cmd_results_new(CMD_INVALID, "tray_padding",
+                                "Expected 'tray_padding <px> [px]'");
+        }
+
+        wlr_log(WLR_DEBUG, "[Bar %s] Setting tray padding to %d", bar->id, padding);
+        config->current_bar->tray_padding = padding;
+        return cmd_results_new(CMD_SUCCESS, NULL, NULL);
+#else
+        return cmd_results_new(CMD_INVALID, "tray_padding",
+                        "Sway has been compiled without tray support");
+#endif
 }
diff --git a/sway/config/bar.c b/sway/config/bar.c
index 45c9e998..670219f1 100644
--- a/sway/config/bar.c
+++ b/sway/config/bar.c
@@ -12,6 +12,7 @@
 #include <signal.h>
 #include "sway/config.h"
 #include "sway/output.h"
+#include "config.h"
 #include "stringop.h"
 #include "list.h"
 #include "log.h"
@@ -77,6 +78,10 @@ void free_bar_config(struct bar_config *bar) {
         free(bar->colors.binding_mode_border);
         free(bar->colors.binding_mode_bg);
         free(bar->colors.binding_mode_text);
+#if HAVE_TRAY
+        list_free_items_and_destroy(bar->tray_outputs);
+        free(bar->icon_theme);
+#endif
         free(bar);
 }
 
@@ -165,6 +170,10 @@ struct bar_config *default_bar_config(void) {
         bar->colors.binding_mode_bg = NULL;
         bar->colors.binding_mode_text = NULL;
 
+#if HAVE_TRAY
+        bar->tray_padding = 2;
+#endif
+
         list_add(config->bars, bar);
         return bar;
 cleanup:
diff --git a/sway/ipc-json.c b/sway/ipc-json.c
index 96701dc2..53e0e335 100644
--- a/sway/ipc-json.c
+++ b/sway/ipc-json.c
@@ -1,6 +1,7 @@
 #include <json-c/json.h>
 #include <stdio.h>
 #include <ctype.h>
+#include "config.h"
 #include "log.h"
 #include "sway/config.h"
 #include "sway/ipc-json.h"
@@ -785,5 +786,41 @@ json_object *ipc_json_describe_bar_config(struct bar_config *bar) {
                 }
                 json_object_object_add(json, "outputs", outputs);
         }
+#if HAVE_TRAY
+        // Add tray outputs if defined
+        if (bar->tray_outputs && bar->tray_outputs->length > 0) {
+                json_object *tray_outputs = json_object_new_array();
+                for (int i = 0; i < bar->tray_outputs->length; ++i) {
+                        const char *name = bar->tray_outputs->items[i];
+                        json_object_array_add(tray_outputs, json_object_new_string(name));
+                }
+                json_object_object_add(json, "tray_outputs", tray_outputs);
+        }
+
+        json_object *tray_bindings = json_object_new_array();
+        for (int i = 0; i < 10; ++i) {
+                if (bar->tray_bindings[i]) {
+                        json_object *bind = json_object_new_object();
+                        json_object_object_add(bind, "input_code",
+                                        json_object_new_int(i));
+                        json_object_object_add(bind, "command",
+                                        json_object_new_string(bar->tray_bindings[i]));
+                        json_object_array_add(tray_bindings, bind);
+                }
+        }
+        if (json_object_array_length(tray_bindings) > 0) {
+                json_object_object_add(json, "tray_bindings", tray_bindings);
+        } else {
+                json_object_put(tray_bindings);
+        }
+
+        if (bar->icon_theme) {
+                json_object_object_add(json, "icon_theme",
+                                json_object_new_string(bar->icon_theme));
+        }
+
+        json_object_object_add(json, "tray_padding",
+                        json_object_new_int(bar->tray_padding));
+#endif
         return json;
 }
diff --git a/sway/meson.build b/sway/meson.build
index 6d446acb..7f739287 100644
--- a/sway/meson.build
+++ b/sway/meson.build
@@ -99,11 +99,9 @@ sway_sources = files(
         'commands/workspace_layout.c',
         'commands/ws_auto_back_and_forth.c',
 
-        'commands/bar/activate_button.c',
         'commands/bar/binding_mode_indicator.c',
         'commands/bar/bindsym.c',
         'commands/bar/colors.c',
-        'commands/bar/context_button.c',
         'commands/bar/font.c',
         'commands/bar/gaps.c',
         'commands/bar/height.c',
@@ -115,12 +113,12 @@ sway_sources = files(
         'commands/bar/output.c',
         'commands/bar/pango_markup.c',
         'commands/bar/position.c',
-        'commands/bar/secondary_button.c',
         'commands/bar/separator_symbol.c',
         'commands/bar/status_command.c',
         'commands/bar/strip_workspace_numbers.c',
         'commands/bar/strip_workspace_name.c',
         'commands/bar/swaybar_command.c',
+        'commands/bar/tray_bindsym.c',
         'commands/bar/tray_output.c',
         'commands/bar/tray_padding.c',
         'commands/bar/workspace_buttons.c',
diff --git a/sway/sway-bar.5.scd b/sway/sway-bar.5.scd
index a3c6af2e..2357591d 100644
--- a/sway/sway-bar.5.scd
+++ b/sway/sway-bar.5.scd
@@ -100,27 +100,20 @@ The following commands configure the tray.
 The _button_ argument in all cases is a platform-specific button code. On Linux
 you can find a list of these at linux/input-event-codes.h.
 
-*activate\_button* <button>
-        Sets the button to be used for the _activate_ (primary click) tray item
-        event. The default is BTN\_LEFT (0x110).
-
-*context\_button* <button>
-        Sets the button to be used for the _context menu_ (right click) tray item
-        event. The default is BTN\_RIGHT (0x111).
-
-*secondary\_button* <button>
-        Sets the button to be used for the _secondary_ (middle click) tray item
-        event. The default is BTN\_MIDDLE (0x112).
-
-*tray\_output* none|all|<output>
-        Sets the output that the tray will appear on or none. Unlike i3bar, swaybar
-        is able to show icons on any number of bars and outputs without races.
-        The default is _all_.
+*tray\_bindsym* button<n> ContextMenu|Activate|SecondaryActivate|ScrollDown|ScrollLeft|ScrollRight|ScrollUp|nop
+        Binds mouse button _n_ (1 to 9) to the specified action. Use the command
+        _nop_ to disable the default action (Activate for button 1, ContextMenu for
+        button 2 and SecondaryActivate for button 3).
 
 *tray\_padding* <px> [px]
         Sets the pixel padding of the system tray. This padding will surround the
         tray on all sides and between each item. The default value for _px_ is 2.
 
+*tray\_output* none|<output>
+        Restrict the tray to a certain output, can be specified multiple times. If
+        omitted, the tray will be displayed on all outputs. Unlike i3bar, swaybar
+        can show icons on any number of bars and outputs without races.
+
 *icon\_theme* <name>
         Sets the icon theme that sway will look for item icons in. This option has
         no default value, because sway will always default to the fallback theme,