Inez has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/69503


Change subject: Ignore shortcut keys inside key press handler
......................................................................

Ignore shortcut keys inside key press handler

Change-Id: I8f814a0f369c470038896041f157b3dbd0d7f1a7
---
M modules/ve/ce/ve.ce.Surface.js
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/VisualEditor 
refs/changes/03/69503/1

diff --git a/modules/ve/ce/ve.ce.Surface.js b/modules/ve/ce/ve.ce.Surface.js
index 1fbea19..24ed81c 100644
--- a/modules/ve/ce/ve.ce.Surface.js
+++ b/modules/ve/ce/ve.ce.Surface.js
@@ -495,7 +495,7 @@
        // deletion would occur in case when selection is not collapsed and 
user press home key for
        // instance (Firefox fires keypress for home key).
        // TODO: Should be covered with Selenium tests.
-       if ( e.which === 0 || e.charCode === 0 ) {
+       if ( e.which === 0 || e.charCode === 0 || ve.ce.isShortcutKey( e ) ) {
                return;
        }
 

-- 
To view, visit https://gerrit.wikimedia.org/r/69503
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I8f814a0f369c470038896041f157b3dbd0d7f1a7
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/VisualEditor
Gerrit-Branch: master
Gerrit-Owner: Inez <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to