abenz1267 / walker

Application launcher similar to Rofi etc. . Wayland native.
MIT License
263 stars 12 forks source link

feat: alternate AM key #11

Closed michaelBelsanti closed 5 months ago

michaelBelsanti commented 5 months ago

Adds a config options "am_key" that lets you change the modifier key used for activation mode. This patch lets you use control or alt.

10

abenz1267 commented 5 months ago

Thanks for the PR. A few things:

michaelBelsanti commented 5 months ago
  • make the alt config a boolean use_alt, that way there's no accidential "why does the key i set not work" as it'd just silently default to ctrl.

Sounds good, wasn't sure if it'd be worth adding other modifiers, but alt is the only other one i think makes sense anyways

abenz1267 commented 5 months ago

i'll merge so you get the contribution and will quickly refactor myself, because i want to push an update shortly and would like this to be in.

thanks!

michaelBelsanti commented 5 months ago

no problem, thank you!