From: Moritz Bunkus Date: Thu, 12 Jan 2017 09:57:13 +0000 (+0100) Subject: PartPicker: Taskendrücke auf Shift, Ctrl & Alt ignorieren X-Git-Tag: release-3.5.4~1733 X-Git-Url: http://wagnertech.de/git?a=commitdiff_plain;h=a924abafa0da3e604d2b1b22e8041da82dae629a;p=kivitendo-erp.git PartPicker: Taskendrücke auf Shift, Ctrl & Alt ignorieren Andernfalls wird schon beim Drücken eines der Modifier eine Suche ausgelöst. Das kann besonders nervig sein, wenn man den Focus vom Browser wegnimmt: • Alt+Tab drücken, um zum anderen Fenster zu wechseln • Schon bei Alt wird eine Suche ausgelöst. • Durch den Focus-Verlust wird ein Blur-Event ausgelöst. • Und schon ist der interne Zustand ungültig, und das Element wird rot dargestellt. Analoges passiert, wenn man z.B. mit Ctrl+F2 zu einem anderen virtuellen Desktop wechselt. Noch schlimmer ist, dass bei Shift+Tab ebenfalls zuerst eine Suche ausgelöst und anschließend verhindert wird, dass der Focus das Element verlässt. --- diff --git a/js/autocomplete_part.js b/js/autocomplete_part.js index d6c8f23d5..cae7f01af 100644 --- a/js/autocomplete_part.js +++ b/js/autocomplete_part.js @@ -14,6 +14,9 @@ namespace('kivi', function(k){ RIGHT: 39, PAGE_UP: 33, PAGE_DOWN: 34, + SHIFT: 16, + CTRL: 17, + ALT: 18, }; var CLASSES = { PICKED: 'partpicker-picked', @@ -190,6 +193,10 @@ namespace('kivi', function(k){ select: function(event, ui) { set_item(ui.item); }, + search: function(event, ui) { + if ((event.which == KEY.SHIFT) || (event.which == KEY.CTRL) || (event.which == KEY.ALT)) + event.preventDefault(); + } }); /* In case users are impatient and want to skip ahead: * Capture key events and check if it's a unique hit. @@ -223,7 +230,7 @@ namespace('kivi', function(k){ }); return false; } - } else { + } else if ((event.which != KEY.SHIFT) && (event.which != KEY.CTRL) && (event.which != KEY.ALT)) { state = STATES.UNDEFINED; } });