project-repo / cagebreak

Cagebreak: A Wayland Tiling Compositor Inspired by Ratpoison
MIT License
286 stars 19 forks source link

Unify `parse_rc_line` and `parse_action` #1

Closed project-repo closed 4 years ago

project-repo commented 4 years ago

As of right now, the actions which can be run in the config file and the actions which can be run as a keybinding are parsed separately in parse.c. This leads to a lot of code duplication. Furthermore, unifying these functionalities would enable a more versatile configuration. For instance, it would enable the user to write hsplit into the configuration file to split the output on startup and workspace 2 to set the default workspace to workspace 2. Therefore, this change would simplify the code base, while at the same time increasing the feature set.

PS: As a side effect, this would allow quirky statements such as bind d bind r hsplit which would bind the d key to binding the r key to split the output...

project-repo commented 4 years ago

Fixed in 1.1.0