From 0c495eecde82c3b7a2827c713badfd13536f5507 Mon Sep 17 00:00:00 2001 From: Mykyta Holubakha Date: Sun, 8 May 2016 17:17:35 +0300 Subject: [PATCH] Remove FSB_GAPS_INNER and FSB_GAPS_OUTER --- include/config.h | 11 ++--------- sway/commands.c | 29 +++++++---------------------- sway/config.c | 9 ++++----- sway/handlers.c | 35 +++++------------------------------ sway/sway.5.txt | 11 ++--------- 5 files changed, 20 insertions(+), 75 deletions(-) diff --git a/include/config.h b/include/config.h index 6bd09151a..e15ba311f 100644 --- a/include/config.h +++ b/include/config.h @@ -166,12 +166,6 @@ enum edge_border_types { E_BOTH /**< hide vertical and horizontal edge borders */ }; -enum floating_scroll_behavior { - FSB_GAPS_OUTER, /**< Adjust outer gaps */ - FSB_GAPS_INNER, /**< Adjust inner gaps */ - FSB_CUSTOM /**< Perform a user-defined action */ -}; - /** * The configuration struct. The result of loading a config file. */ @@ -190,9 +184,8 @@ struct sway_config { uint32_t floating_mod; uint32_t dragging_key; uint32_t resizing_key; - enum floating_scroll_behavior floating_scroll; - char *fsb_up; - char *fsb_down; + char *floating_scroll_up_cmd; + char *floating_scroll_down_cmd; enum swayc_layouts default_orientation; enum swayc_layouts default_layout; char *font; diff --git a/sway/commands.c b/sway/commands.c index a4fcde787..5a5925559 100644 --- a/sway/commands.c +++ b/sway/commands.c @@ -711,34 +711,19 @@ static struct cmd_results *cmd_floating_scroll(int argc, char **argv) { if ((error = checkarg(argc, "floating_scroll", EXPECTED_AT_LEAST, 1))) { return error; } - if (!strcasecmp("behavior", argv[0])) { + if (!strcasecmp("up", argv[0])) { + free(config->floating_scroll_up_cmd); if (argc < 2) { - error = cmd_results_new(CMD_INVALID, "floating_scroll", "Insufficient parameters given"); - return error; - } - if (!strcasecmp("gaps_inner", argv[1])) { - config->floating_scroll = FSB_GAPS_INNER; - } else if (!strcasecmp("gaps_outer", argv[1])) { - config->floating_scroll = FSB_GAPS_OUTER; - } else if (!strcasecmp("custom", argv[1])) { - config->floating_scroll = FSB_CUSTOM; + config->floating_scroll_up_cmd = strdup(""); } else { - error = cmd_results_new(CMD_INVALID, "floating_scroll", "Unknown behavior: '%s'", argv[1]); - return error; - } - } else if (!strcasecmp("up", argv[0])) { - free(config->fsb_up); - if (argc < 2) { - config->fsb_up = strdup(""); - } else { - config->fsb_up = join_args(argv + 1, argc - 1); + config->floating_scroll_up_cmd = join_args(argv + 1, argc - 1); } } else if (!strcasecmp("down", argv[0])) { - free(config->fsb_down); + free(config->floating_scroll_down_cmd); if (argc < 2) { - config->fsb_down = strdup(""); + config->floating_scroll_down_cmd = strdup(""); } else { - config->fsb_down = join_args(argv + 1, argc - 1); + config->floating_scroll_down_cmd = join_args(argv + 1, argc - 1); } } else { error = cmd_results_new(CMD_INVALID, "floating_scroll", "Unknown command: '%s'", argv[0]); diff --git a/sway/config.c b/sway/config.c index 15aed11b1..6c1d21c8d 100644 --- a/sway/config.c +++ b/sway/config.c @@ -131,8 +131,8 @@ void free_config(struct sway_config *config) { list_free(config->active_bar_modifiers); free_flat_list(config->config_chain); free(config->font); - free(config->fsb_up); - free(config->fsb_down); + free(config->floating_scroll_up_cmd); + free(config->floating_scroll_down_cmd); free(config); } @@ -161,9 +161,8 @@ static void config_defaults(struct sway_config *config) { config->floating_mod = 0; config->dragging_key = M_LEFT_CLICK; config->resizing_key = M_RIGHT_CLICK; - config->floating_scroll = FSB_GAPS_INNER; - config->fsb_up = strdup(""); - config->fsb_down = strdup(""); + config->floating_scroll_up_cmd = strdup(""); + config->floating_scroll_down_cmd = strdup(""); config->default_layout = L_NONE; config->default_orientation = L_NONE; config->font = strdup("monospace 10"); diff --git a/sway/handlers.c b/sway/handlers.c index 23c9985e6..67275575a 100644 --- a/sway/handlers.c +++ b/sway/handlers.c @@ -723,36 +723,11 @@ static bool handle_pointer_button(wlc_handle view, uint32_t time, const struct w bool handle_pointer_scroll(wlc_handle view, uint32_t time, const struct wlc_modifiers* modifiers, uint8_t axis_bits, double _amount[2]) { if (!(modifiers->mods ^ config->floating_mod)) { - switch (config->floating_scroll) { - case FSB_GAPS_INNER: - case FSB_GAPS_OUTER: - { - int amount = (int)_amount[0]; - int i,j; - for (i = 0; i < root_container.children->length; ++i) { - swayc_t *op = root_container.children->items[i]; - for (j = 0; j < op->children->length; ++j) { - swayc_t *ws = op->children->items[j]; - if (config->floating_scroll == FSB_GAPS_INNER) { - container_map(ws, add_gaps, &amount); - } else { - ws->gaps += amount; - } - } - } - arrange_windows(&root_container, -1, -1); - break; - } - case FSB_CUSTOM: - { - int amount = (int)_amount[0]; - if (amount > 0) { - handle_command(config->fsb_up); - } else if (amount < 0) { - handle_command(config->fsb_down); - } - break; - } + int amount = (int)_amount[0]; + if (amount > 0) { + handle_command(config->floating_scroll_up_cmd); + } else if (amount < 0) { + handle_command(config->floating_scroll_down_cmd); } } return EVENT_PASSTHROUGH; diff --git a/sway/sway.5.txt b/sway/sway.5.txt index fdb7818c5..252290335 100644 --- a/sway/sway.5.txt +++ b/sway/sway.5.txt @@ -156,17 +156,10 @@ or triggered at runtime. enabled, left click is used for resizing and right click for dragging. The mode paramenter is optional and defaults to _normal_ if it isn't defined. -**floating_scroll** behavior :: - If set to _gaps_inner_, adjusts inner gaps of the windows on scrolling - while holding the floating modifier. If set to _gaps_outer_, adjusts - outer gaps. If set to _custom_, performs user-defined commands, - specified in *floating_scroll up|down*. - **floating_scroll** [command]:: Sets the command to be executed on scrolling up and down - (respectively) while holding the floating modifier, when - *floating_scroll behavior* is set to _custom_. Resets the command, - when given none. + (respectively) while holding the floating modifier. Resets the + command, when given no arguments. **focus_follows_mouse** :: If set to _yes_, the currently focused view will change as you move your