id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc
14799	Handling MOD+(i,b,u) shortcuts in more friendly way to hostile environment	Michal Aichinger		"== Steps to reproduce ==

Our application is setting shortcuts Ctrl+Alt+i in Windows and Ctrl+Cmd+i in Mac version and they works fine until editor is focused. 

We are checking exactly which modifier keys are used and act only when exact combination is pressed. We find out that editor is handling Ctrl+Cmd+i shortcut and stopping event propagation. What a nasty boy!

I digged into source code and find method getKeystroke, which do: 
 

{{{
if (this.$.ctrlKey || this.$.metaKey)  a += CKEDITOR.CTRL;
}}}


So you can make text italic with: Ctrl+i, Win+i, Cmd+i, Ctrl+Win+i, Ctrl+Cmd+i! This is not mentioned in documentation, where is explicitly mentioned Ctrl+i.


== Expected result ==

I expect that editor will distinguish Ctrl and Meta key and do what is in documentation -> handle only Ctrl+letter shortcuts. If you want to support also meta key, please, to it in smarter way, that both shouldn't be pressed:


{{{
if ((this.$.ctrlKey || this.$.metaKey) && !(this.$.ctrlKey && this.$.metaKey))  a += CKEDITOR.CTRL;
}}}



Actually there should be some configuration option to set behavior for this method. 

And last comment. What about to do not stop event propagation (https://css-tricks.com/dangers-stopping-event-propagation/) and just prevent events defaults?

== Actual result ==

== Other details (browser, OS, CKEditor version, installed plugins) ==
Used version:""4.5.4 (Standard)"",revision:""d4677a3""."	New Feature	confirmed	Normal		Accessibility	4.0