text-case.nvim icon indicating copy to clipboard operation
text-case.nvim copied to clipboard

Add some additional default keymaps

Open theutz opened this issue 1 year ago • 4 comments

I thought these might be a useful addition to the default keymaps as well:

  • to_title_case
  • to_dot_case
  • to_path_case

Feel free to use this PR to add default keymaps for the other methods that are missing. I didn't want to start making too many additions that weren't mnemonic with the method name.

theutz avatar Feb 07 '24 08:02 theutz

Hi @theutz, thanks for your contribution.

I didn't include them because those types are not commonly used specially as keymaps, they are actually useful for the Subs command

I'm quite curious about the usage you are making of those

johmsalas avatar Feb 11 '24 14:02 johmsalas

Thanks for the feedback! I can see where those would definitely useful for the subs command, and I understand you not wanting to clutter up the keybindings too much.

My use case for the Title case command is that I'm often moving text back-and-forth between comments/docs and code, and often in the more user-facing text I'll refer to, let's say, lists of properties that are in snake case, but I want to list them in plain English.

I don't have as strong of a use case for dot and dash case, though. Perhaps those are excessive for the key bindings.

Let me know what you think!

theutz avatar Feb 12 '24 07:02 theutz

Given this plugin scope is bigger than just code (ie: it supports unicode). I think title case is in fact a good candidate for the default list of keymaps.

johmsalas avatar Feb 13 '24 02:02 johmsalas

You know, I don’t really use that, no. I just use the basic title case.

theutz avatar Feb 13 '24 13:02 theutz

I see this pull request has been approved, but it hasn't been merged. Are there any plans to eventually merge it?

Kayzels avatar May 22 '24 09:05 Kayzels