summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorThomas Bruederli <thomas@roundcube.net>2014-04-25 18:13:32 +0200
committerThomas Bruederli <thomas@roundcube.net>2014-04-25 18:16:15 +0200
commit6c7452d667159d99d36103fbeae1f19acf64e05e (patch)
treee1cf214d9a34e12f0f5768cb13f22582ae77eb41
parent7fface125e7db1e7ab47bdfbda5306fa16ea2486 (diff)
Avoid popupmenus being closed when scrollbar is clicked (#1489832)
-rw-r--r--skins/classic/functions.js1
-rw-r--r--skins/larry/ui.js1
2 files changed, 2 insertions, 0 deletions
diff --git a/skins/classic/functions.js b/skins/classic/functions.js
index 049e4d5d4..75ae5133f 100644
--- a/skins/classic/functions.js
+++ b/skins/classic/functions.js
@@ -417,6 +417,7 @@ body_mouseup: function(evt, p)
for (i in this.popups) {
if (this.popups[i].obj.is(':visible') && target != rcube_find_object(i+'link')
&& !this.popups[i].toggle
+ && target != this.popups[i].obj.get(0) // check if scroll bar was clicked (#1489832)
&& (!this.popups[i].editable || !this.target_overlaps(target, this.popups[i].id))
&& (!this.popups[i].sticky || !rcube_mouse_is_over(evt, rcube_find_object(this.popups[i].id)))
&& !$(target).is('.folder-selector-link') && !$(target).children('.folder-selector-link').length
diff --git a/skins/larry/ui.js b/skins/larry/ui.js
index 44fc727f6..cf65dc1f4 100644
--- a/skins/larry/ui.js
+++ b/skins/larry/ui.js
@@ -378,6 +378,7 @@ function rcube_mail_ui()
config = popupconfig[id];
if (obj.is(':visible')
&& target.id != id+'link'
+ && target != obj.get(0) // check if scroll bar was clicked (#1489832)
&& !config.toggle
&& (!config.editable || !target_overlaps(target, obj.get(0)))
&& (!config.sticky || !rcube_mouse_is_over(e, obj.get(0)))