MDL-43595: The dropdowns for editpdf stopped working due to a recent change.

This commit is contained in:
Damyon Wiese 2014-01-08 09:15:58 +08:00
parent 267d4927b4
commit cf1110eb62
4 changed files with 12 additions and 9 deletions

View File

@ -1649,7 +1649,8 @@ Y.extend(DROPDOWN, M.core.dialogue, {
body.on('clickoutside', function(e) {
if (this.get('visible')) {
if (e.target !== button && e.target.ancestor() !== button) {
// Note: we need to compare ids because for some reason - sometimes button is an Object, not a Y.Node.
if (e.target.get('id') !== button.get('id') && e.target.ancestor().get('id') !== button.get('id')) {
e.preventDefault();
this.hide();
}

View File

@ -1649,7 +1649,8 @@ Y.extend(DROPDOWN, M.core.dialogue, {
body.on('clickoutside', function(e) {
if (this.get('visible')) {
if (e.target !== button && e.target.ancestor() !== button) {
// Note: we need to compare ids because for some reason - sometimes button is an Object, not a Y.Node.
if (e.target.get('id') !== button.get('id') && e.target.ancestor().get('id') !== button.get('id')) {
e.preventDefault();
this.hide();
}

View File

@ -46,7 +46,8 @@ Y.extend(DROPDOWN, M.core.dialogue, {
body.on('clickoutside', function(e) {
if (this.get('visible')) {
if (e.target !== button && e.target.ancestor() !== button) {
// Note: we need to compare ids because for some reason - sometimes button is an Object, not a Y.Node.
if (e.target.get('id') !== button.get('id') && e.target.ancestor().get('id') !== button.get('id')) {
e.preventDefault();
this.hide();
}