diff options
author | Szymon Kłos <szymon.klos@collabora.com> | 2024-03-30 21:04:50 +0100 |
---|---|---|
committer | Szymon Kłos <eszkadev@gmail.com> | 2024-04-01 11:26:51 +0200 |
commit | 5cdb8268299dcdb3a9aa5ff7c661e294d19de851 (patch) | |
tree | 506295c71a1abd61b154ca590a01706f887b9c8d /browser/src/map/handler/Map.Keyboard.js | |
parent | jsdialog: css: put toolbuttons rules in btns.css (diff) | |
download | online-5cdb8268299dcdb3a9aa5ff7c661e294d19de851.tar.gz online-5cdb8268299dcdb3a9aa5ff7c661e294d19de851.zip |
remove-w2ui: use JSDialog dropdown for user list
- reuse JSDialog Dropdown
- remove hack with overflow which was needed to show
any popup above notebookbar
- fixes scrollbar visible above compact toolbar when
switching from notebookbar mode
Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: I0366dbd68df20991d5fb26d116c38fb657e64c6d
Diffstat (limited to 'browser/src/map/handler/Map.Keyboard.js')
-rw-r--r-- | browser/src/map/handler/Map.Keyboard.js | 11 |
1 files changed, 3 insertions, 8 deletions
diff --git a/browser/src/map/handler/Map.Keyboard.js b/browser/src/map/handler/Map.Keyboard.js index 96991a57cc..4885101372 100644 --- a/browser/src/map/handler/Map.Keyboard.js +++ b/browser/src/map/handler/Map.Keyboard.js @@ -758,17 +758,12 @@ L.Map.Keyboard = L.Handler.extend({ return true; case this.keyCodes.P: var userListSummary = document.getElementById('userListSummary'); - var userListPopover = document.getElementById('userListPopover'); + var userListPopover = document.getElementById('userlist-dropdown'); // checking case ''(empty string) is because when element loads first time it does not have any inline display style - var isUserListPopoverVisible = userListPopover.style.display === 'none' || userListPopover.style.display === ''; + var isUserListPopoverVisible = !userListPopover || userListPopover.style.display === 'none' || userListPopover.style.display === ''; // we should only show user list when there are multiple users present and user list is hidden var showUserList = isUserListPopoverVisible && userListSummary.hasChildNodes(); - if (showUserList) { - userListSummary.click(); - } - else { - userListPopover.style.display = 'none'; - } + if (showUserList) userListSummary.click(); e.preventDefault(); return true; case this.keyCodes.pageUp: |