Fixed Change Case overlapping shortcuts

[PWrzesinski]

Description

The Change Case extension used the key combinations cmd+r and cmd+shift+r for multiple different actions that were available at the same time in some scenarios, so some actions couldn't be invoked. This PR changes the key for the "Remove Pinned Case", "Clear Pinned Cases", "Remove Recent Case" and "Clear Recent Cases" from "r" to "q". The cmd+r shortcut will invoke only the "Refresh Content" action now, regardless of state.

Example scenario demonstrating incorrect behavior:

The user has 3 different texts in 3 different files:

new york
la spezia
buenos aires

The user wants to correct the case for each one to proper Capital Case, so they copy the first one, open Raycast, go to Change Case extension, choose the Capital Case, press enter and paste the result back to the first file.

Then they go to the second file, copy second text, go back to Raycast and want to hit cmd+k, cmd+r to refresh the result with new clipboard content, however the Capital Case has moved to recently used cases, so now cmd+r instead of performing the "Refresh Content" action will trigger the "Remove Recent Case" action. The "Refresh Content" action is not available by any shortcut at this point.

Note: I've chosen the q key arbitrarily, if there is some convention I should use or there are differing opinions I'm happy to discuss in the comments.

Screencast

Checklist

• I read the extension guidelines
• I read the documentation about publishing
• I ran npm run build and tested this distribution build in Raycast
• I checked that files in the assets folder are used by the extension itself
• I checked that assets used by the README are placed outside of the metadata folder

[raycastbot]

Thank you for your first contribution! 🎉

🔔 @erics118 @pernielsentikaer @nagauta you might want to have a look.

[PWrzesinski]

BTW by "user" I very much meant myself and this is super annoying 😬

Also a little offtopic, but I wasn't sure if I should open an issue for this: I needed to pull the whole 10GB repository just to make this minor change - yikes! Perhaps it would be a good time to think about splitting the repo somehow? Just a suggestion.

[erics118]

I'd suggest using ctrl(-(shift-)x, or, Keyboard.Shortcut.Common.Remove(All)
https://developers.raycast.com/api-reference/keyboard#keyboard.shortcut.common

[erics118]

I needed to pull the whole 10GB repository

Raycast offers its own way of managing a singular extension. see: https://developers.raycast.com/basics/contribute-to-an-extension

You can also do a sparse checkout (which is what I do), as I feel more comfortable with using git commands rather than a wrapper around git.

[pernielsentikaer]

Forking the extension is the way to go for sure 🔥

[PWrzesinski]

I needed to pull the whole 10GB repository

Raycast offers its own way of managing a singular extension. see: https://developers.raycast.com/basics/contribute-to-an-extension

You can also do a sparse checkout (which is what I do), as I feel more comfortable with using git commands rather than a wrapper around git.

Ooooops sorry about that. I should've probably read more. I did the change yesterday in the middle of other tasks when this issue annoyed me enough to finally do something about it, but I didn't have much time. Also I didn't know the sparse checkout git feature - cool!

Will update the keys in a moment.

[pernielsentikaer]

Let me know when this is ready for another review 🙂

[PWrzesinski]

@erics118 @pernielsentikaer

I've changed the keyboard shortcuts to use Keyboard.Shortcut.Common. During the development (the fork extension is great!) I've also noted that the popToRoot setting is inverted:

if (preferences.popToRoot) {
    closeMainWindow();
} else {
    popToRoot();
}

I've fixed that as well - previously it would pop to root when the checkbox was unchecked:

The PR is ready for another review - thanks for being patient with me!

[pernielsentikaer]

Thanks 👏

@erics118 could you check

[erics118]

Looks good 👍

[github-actions]

Published to the Raycast Store:
https://raycast.com/erics118/change-case

[raycastbot]

🎉 🎉 🎉

We've rewarded your Raycast account with some credits. You will soon be able to exchange them for some swag.

[PWrzesinski]

🎉 🎉 🎉

We've rewarded your Raycast account with some credits. You will soon be able to exchange them for some swag.

Haha this being merged is reward enough, thanks!