diff options
author | GeoSot <geo.sotis@gmail.com> | 2022-03-10 14:24:47 +0300 |
---|---|---|
committer | GeoSot <geo.sotis@gmail.com> | 2022-04-07 14:18:31 +0300 |
commit | 185560d8ac37d7c730423a227228430c886f56f4 (patch) | |
tree | f882740805450f9a667dc9f00b90906e1c7b1981 | |
parent | 5b31463e35bc9af18fabb30f76186c0e77ea2b99 (diff) |
Update dropdown.jsGeoSot-patch-2
minor refactoring
-rw-r--r-- | js/src/dropdown.js | 21 |
1 files changed, 9 insertions, 12 deletions
diff --git a/js/src/dropdown.js b/js/src/dropdown.js index 65b3aa372a..dfa9a63aad 100644 --- a/js/src/dropdown.js +++ b/js/src/dropdown.js @@ -404,25 +404,22 @@ class Dropdown extends BaseComponent { } event.preventDefault() - if (!isEscapeEvent) { - event.stopPropagation() - } const getToggleButton = SelectorEngine.findOne(SELECTOR_DATA_TOGGLE, event.delegateTarget.parentNode) const instance = Dropdown.getOrCreateInstance(getToggleButton) - if (isEscapeEvent) { - if (getToggleButton.classList.contains(CLASS_NAME_SHOW)) { - instance.hide() - getToggleButton.focus() - event.stopPropagation() - } - + if (isUpOrDownEvent) { + event.stopPropagation() + instance.show() + instance._selectMenuItem(event) return } - instance.show() - instance._selectMenuItem(event) + if (instance._isShown()) { // else is escape and we check if it is shown + event.stopPropagation() + instance.hide() + getToggleButton.focus() + } } } |