[Suggestion] On the issue of comma and the keyboard layout

[anesdiaf]

Al salamu alaikum brother mohamed,
Hope you're doing well,

I've watched your video about your extension and i did hear you talk about the problem with keyboard layouts changing with languages
(Qwerty,Azerty).

I would like to suggest to you a simple solution i've tried, that can make the shortcut constant across all keyboard layouts,
basically i've used event.code instead of event.key, the reason is that event.code property represents a physical key on the keyboard (as opposed to the character generated by pressing the key). you can refer back to https://developer.mozilla.org/en-US/docs/Web/API/KeyboardEvent/code

this is my basic implementation

keypress event is already deprecated, you could also read from the link provided that keypress event doesn't react to certain key presses events because they don't produce a character value, for example CTR key.

So that will not be as helpful as what's already written by @bengmoh

[anesdiaf]

You can simply use the "keydown" event, if you want to use the control key in it self or with another key;
As for "e.code" property not returning a character, the initial issue comes from using a property ("e.key") that returns a character based on the used keyboard layout.
The "e.code" property is the standard for setting kb shortcuts in javascript, because it's cross-browser/os compatible, and keyboard layout independent,

here's a basic implementation (content.js file)