mirrors/zotero
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages 1 Wiki Activity Actions 1

fx-compat: Don't run <split-menu-button> command handler when popup is opened

Browse source
This commit is contained in:
Abe Jellinek 2023-03-23 14:15:14 -04:00
parent 8f87a74398
commit 5b08eec813
1 changed files with 3 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

5
chrome/content/zotero/elements/splitMenuButton.js
View file

@ -44,7 +44,7 @@
// the popup if over the end side of the button
this.addEventListener('mousedown', (event) => {
let rect = this.querySelector('[anonid="dropmarker-box"]').getBoundingClientRect();
if (event.x >= rect.left && event.x <= rect.right
if ((!Zotero.rtl && event.clientX >= rect.left || Zotero.rtl && event.clientX <= rect.right)
&& Zotero.Utilities.Internal.showNativeElementPopup(this)) {
event.preventDefault();
}
@ -83,7 +83,8 @@
_handleClick() {
super._handleClick();
if (!this.disabled) {
let popup = this.querySelector(':scope > menupopup');
if (!this.disabled && (!popup || popup.state == 'closed')) {
this.doCommand();
}
}