[FEATURE] key shortcut for closing chat should change

[Dun-sin]

Description

The key shortcut for closing a chat is Ctrl + c which is also the copy shortcut, not a good mixture to close a chat should change to ctrl + shift + c that we don't have a mismatch.

You can find the code for this here and the hook code at useKeyPress.js

Whisper/client/src/components/Anonymous.jsx

Lines 103 to 104 in a253b2d

	useKeyPress(['c'], () => handleClose());
	useKeyPress(['n'], () => handleClose(true));

Screenshots

No response

Additional information

No response

👀 Have you checked if this issue has been raised before?

• I checked and didn't find similar issue

🏢 Have you read the Contributing Guidelines?

• I have read and understood the rules in the Contributing Guidelines

Are you willing to work on this issue ?

None

[github-actions]

It's great having you contribute to this project by creating an issue

Make sure you are assigned to this before you work on it and you read the Contributing.md file, Thank you! Welcome to the community 🤓

[RiccardoCafa]

Hi @Dun-sin can I take this issue in charge?

[Dun-sin]

Hi @Dun-sin can I take this issue in charge?

Definitely, thanks for wanting to contribute, Make sure you follow the rules 👉🏽here👈🏽, else your PR will not be accepted and will be closed. Good Luck

[RiccardoCafa]

The shortcut CTRL + SHIFT + C currently opens the developer tools in Google Chrome, specifically activating the "Select Element" tool.
What if we opt for CTRL + SHIFT + X instead?

[Dun-sin]

The shortcut CTRL + SHIFT + C currently opens the developer tools in Google Chrome, specifically activating the "Select Element" tool. What if we opt for CTRL + SHIFT + X instead?

oh for sure, didn't think of that since i use firefox. ctrl+ shift + x works fine

[RiccardoCafa]

@Dun-sin I noticed that there's a TypeScript version of the useKeyPress hook in this project. However, it appears that the project is configured to use only JavaScript. This file has some purpose?

Also I was wondering if would be nice if we implement this shortcut feature with a Builder design pattern so it's easy to build and maintain different kind of shortcuts in the future. Let me know what do you think!

[Dun-sin]

@Dun-sin I noticed that there's a TypeScript version of the useKeyPress hook in this project. However, it appears that the project is configured to use only JavaScript. This file has some purpose?

Also I was wondering if would be nice if we implement this shortcut feature with a Builder design pattern so it's easy to build and maintain different kind of shortcuts in the future. Let me know what do you think!

Do a git pull,and see some magic😆