feat: add case sensitivity for command search and improve suggestion rendering

M1n-74316D65 commented 2 months ago

I made this for me, so why not do a PR :)

Added a case sensitivity config for commands
Improved the suggestion when adding a /

xvvvyz commented 1 month ago

Hi @M1n-74316D65, thanks for the PR! I am not sure I understand the purpose of case insensitive commands. If it's just a stylistic thing so the grid has uppercase keys you could text-transform on .key with CSS. As for the improved suggestions, how are they improved? A screenshot or description would be helpful (:

M1n-74316D65 commented 1 month ago

Ohh yeah, it’s pretty lacking in explanation 😅. start.m1n.land

The case sensitivity was added so that if you write a command in uppercase and it's stored in lowercase (or vice versa), it'll still work, making the command matching more flexible regardless of case.
As for the improved suggestions, previously when writing a subcommand and adding a /, the suggestions disappeared. Now they stay visible, making it easier to continue selecting commands.

I put this together pretty quickly, half with AI 😄, so if you need any changes or prefer to handle it yourself, feel free! I’m not super familiar with vanilla JS. If this doesn’t align with the direction you're going, no hard feelings at all if you decide to close the PR.

xvvvyz commented 1 month ago

Ah, understood. Yeah, I don't have a need for case insensitivity and if I am going to use a command suggestion I just tab to it and don't continue typing. I appreciate the PR and explanation, though.

xvvvyz / tilde

feat: add case sensitivity for command search and improve suggestion rendering #36