linutil
linutil copied to clipboard
refact: Add lowercase conversion for keycodes
Type of Change
- [x] Refactoring
Description
- Convert the keys to lowercase eliminating the need to specify both the capital and non-capital keycodes for a specific action.
- Keys that don't need to be converted to lowercase for example theme switching can be declared in the
KEYS_NOT_TO_NORMALIZE - Renamed function
toggle_task_list_guidetoenable_action_guidefor consistency
Testing
- Works as expected no issues.
Checklist
- [x] My code adheres to the coding and style guidelines of the project.
- [x] I have performed a self-review of my own code.
- [x] I have commented my code, particularly in hard-to-understand areas.
- [x] My changes generate no errors/warnings/merge conflicts.
I agree with @cartercanedy. I think this (if at all) should be handled within the modules that use keycodes.
Yeah, I am ok with hotkeys being cap sensitive. We might need to specify different actions for a Capital letter and I don't want to normalize this.