mirrors/labwc
1
0
Fork 0
mirror of https://github.com/labwc/labwc.git synced 2025-11-04 13:30:07 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions

config: fix bug with default scroll mousebind

Browse source 
Specify attribute name when adding default mousebind actions.

This is required because in src/action.c action GoToDesktop requires an
argument with key 'to' and does not accept `action_get_first_arg()`
because the action supports other optional attributes so we could not
guarantee that 'to' is the first one.

Fixes: #948
This commit is contained in:
Johan Malm 2023-06-11 15:57:58 +01:00 • committed by Johan Malm
parent 3bd441caa0
commit 468470182d
1 changed files with 72 additions and 40 deletions
Download patch file Download diff file Expand all files Collapse all files

112
src/config/rcxml.c
View file

@ -802,46 +802,73 @@ load_default_key_bindings(void)
} }
} }
/*
* `struct mouse_combo` variable description and examples:
*
* | Variable | Description | Examples
* |-----------|----------------------------|---------------------
* | context | context name | Maximize, Root
* | button | mousebind button/direction | Left, Up
* | event | mousebind action | Click, Scroll
* | action | action name | ToggleMaximize, GoToDesktop
* | attribute | action attribute | to
* | value | action attribute value | left
*
* <mouse>
* <context name="Maximize">
* <mousebind button="Left" action="Click">
* <action name="Focus"/>
* <action name="Raise"/>
* <action name="ToggleMaximize"/>
* </mousebind>
* </context>
* <context name="Root">
* <mousebind direction="Up" action="Scroll">
* <action name="GoToDesktop" to="left" wrap="yes"/>
* </mousebind>
* </context>
* </mouse>
*/
static struct mouse_combos { static struct mouse_combos {
const char *context, *button, *event, *action, *command; const char *context, *button, *event, *action, *attribute, *value;
} mouse_combos[] = { } mouse_combos[] = {
{ "Left", "Left", "Drag", "Resize", NULL}, { "Left", "Left", "Drag", "Resize", NULL, NULL},
{ "Top", "Left", "Drag", "Resize", NULL}, { "Top", "Left", "Drag", "Resize", NULL, NULL},
{ "Bottom", "Left", "Drag", "Resize", NULL}, { "Bottom", "Left", "Drag", "Resize", NULL, NULL},
{ "Right", "Left", "Drag", "Resize", NULL}, { "Right", "Left", "Drag", "Resize", NULL, NULL},
{ "TLCorner", "Left", "Drag", "Resize", NULL}, { "TLCorner", "Left", "Drag", "Resize", NULL, NULL},
{ "TRCorner", "Left", "Drag", "Resize", NULL}, { "TRCorner", "Left", "Drag", "Resize", NULL, NULL},
{ "BRCorner", "Left", "Drag", "Resize", NULL}, { "BRCorner", "Left", "Drag", "Resize", NULL, NULL},
{ "BLCorner", "Left", "Drag", "Resize", NULL}, { "BLCorner", "Left", "Drag", "Resize", NULL, NULL},
{ "Frame", "A-Left", "Press", "Focus", NULL}, { "Frame", "A-Left", "Press", "Focus", NULL, NULL},
{ "Frame", "A-Left", "Press", "Raise", NULL}, { "Frame", "A-Left", "Press", "Raise", NULL, NULL},
{ "Frame", "A-Left", "Drag", "Move", NULL}, { "Frame", "A-Left", "Drag", "Move", NULL, NULL},
{ "Frame", "A-Right", "Press", "Focus", NULL}, { "Frame", "A-Right", "Press", "Focus", NULL, NULL},
{ "Frame", "A-Right", "Press", "Raise", NULL}, { "Frame", "A-Right", "Press", "Raise", NULL, NULL},
{ "Frame", "A-Right", "Drag", "Resize", NULL}, { "Frame", "A-Right", "Drag", "Resize", NULL, NULL},
{ "Titlebar", "Left", "Press", "Focus", NULL}, { "Titlebar", "Left", "Press", "Focus", NULL, NULL},
{ "Titlebar", "Left", "Press", "Raise", NULL}, { "Titlebar", "Left", "Press", "Raise", NULL, NULL},
{ "Title", "Left", "Drag", "Move", NULL }, { "Title", "Left", "Drag", "Move", NULL, NULL },
{ "Title", "Left", "DoubleClick", "ToggleMaximize", NULL }, { "Title", "Left", "DoubleClick", "ToggleMaximize", NULL, NULL },
{ "TitleBar", "Right", "Click", "Focus", NULL}, { "TitleBar", "Right", "Click", "Focus", NULL, NULL},
{ "TitleBar", "Right", "Click", "Raise", NULL}, { "TitleBar", "Right", "Click", "Raise", NULL, NULL},
{ "TitleBar", "Right", "Click", "ShowMenu", "client-menu"}, { "TitleBar", "Right", "Click", "ShowMenu", "menu", "client-menu"},
{ "Close", "Left", "Click", "Close", NULL }, { "Close", "Left", "Click", "Close", NULL, NULL },
{ "Iconify", "Left", "Click", "Iconify", NULL}, { "Iconify", "Left", "Click", "Iconify", NULL, NULL},
{ "Maximize", "Left", "Click", "ToggleMaximize", NULL}, { "Maximize", "Left", "Click", "ToggleMaximize", NULL, NULL},
{ "WindowMenu", "Left", "Click", "ShowMenu", "client-menu"}, { "WindowMenu", "Left", "Click", "ShowMenu", "menu", "client-menu"},
{ "Root", "Left", "Press", "ShowMenu", "root-menu"}, { "Root", "Left", "Press", "ShowMenu", "menu", "root-menu"},
{ "Root", "Right", "Press", "ShowMenu", "root-menu"}, { "Root", "Right", "Press", "ShowMenu", "menu", "root-menu"},
{ "Root", "Middle", "Press", "ShowMenu", "root-menu"}, { "Root", "Middle", "Press", "ShowMenu", "menu", "root-menu"},
{ "Root", "Up", "Scroll", "GoToDesktop", "left"}, { "Root", "Up", "Scroll", "GoToDesktop", "to", "left"},
{ "Root", "Down", "Scroll", "GoToDesktop", "right"}, { "Root", "Down", "Scroll", "GoToDesktop", "to", "right"},
{ "Client", "Left", "Press", "Focus", NULL}, { "Client", "Left", "Press", "Focus", NULL, NULL},
{ "Client", "Left", "Press", "Raise", NULL}, { "Client", "Left", "Press", "Raise", NULL, NULL},
{ "Client", "Right", "Press", "Focus", NULL}, { "Client", "Right", "Press", "Focus", NULL, NULL},
{ "Client", "Right", "Press", "Raise", NULL}, { "Client", "Right", "Press", "Raise", NULL, NULL},
{ "Client", "Middle", "Press", "Focus", NULL}, { "Client", "Middle", "Press", "Focus", NULL, NULL},
{ "Client", "Middle", "Press", "Raise", NULL}, { "Client", "Middle", "Press", "Raise", NULL, NULL},
{ NULL, NULL, NULL, NULL, NULL }, { NULL, NULL, NULL, NULL, NULL, NULL },
}; };
static void static void
@ -873,8 +900,13 @@ load_default_mouse_bindings(void)
action = action_create(current->action); action = action_create(current->action);
wl_list_append(&m->actions, &action->link); wl_list_append(&m->actions, &action->link);
if (current->command) { /*
action_arg_add_str(action, NULL, current->command); * Only one attribute/value (of string type) is required for the
* built-in binds. If more are required in the future, a
* slightly more sophisticated approach will be needed.
*/
if (current->attribute && current->value) {
action_arg_add_str(action, (char *)current->attribute, current->value);
} }
} }
wlr_log(WLR_DEBUG, "Loaded %u merged mousebinds", count); wlr_log(WLR_DEBUG, "Loaded %u merged mousebinds", count);