diff options
author | Ananya Sen | 2012-06-18 11:36:20 -0700 |
---|---|---|
committer | Ananya Sen | 2012-06-18 11:36:20 -0700 |
commit | f47dace72a81cddfa29d89540629cfdd76d54b18 (patch) | |
tree | 8a2e9a1567b8d91d1632d9823b47c8d552aaee8f | |
parent | aefeb5f96bf6e8e993ce44c4f328b56e39196414 (diff) | |
download | ninja-f47dace72a81cddfa29d89540629cfdd76d54b18.tar.gz |
Fixed - IKNINJA-1313 - File menu drop down should close when New File dialog is opened
Signed-off-by: Ananya Sen <Ananya.Sen@motorola.com>
-rwxr-xr-x | js/components/menu/menu-entry.reel/menu-entry.js | 4 | ||||
-rwxr-xr-x | js/components/menu/menu.reel/menu.js | 9 |
2 files changed, 3 insertions, 10 deletions
diff --git a/js/components/menu/menu-entry.reel/menu-entry.js b/js/components/menu/menu-entry.reel/menu-entry.js index b9c546ed..ae3469c6 100755 --- a/js/components/menu/menu-entry.reel/menu-entry.js +++ b/js/components/menu/menu-entry.reel/menu-entry.js | |||
@@ -83,7 +83,7 @@ exports.MenuEntry = Montage.create(Component, { | |||
83 | } | 83 | } |
84 | }, | 84 | }, |
85 | 85 | ||
86 | handleClick: { | 86 | captureMousedown: { |
87 | value: function(event) { | 87 | value: function(event) { |
88 | // TODO: Hack! Rework this! | 88 | // TODO: Hack! Rework this! |
89 | this.parentComponent.ownerComponent.toggleActivation(this); | 89 | this.parentComponent.ownerComponent.toggleActivation(this); |
@@ -105,7 +105,7 @@ exports.MenuEntry = Montage.create(Component, { | |||
105 | 105 | ||
106 | this.topHeaderText.innerHTML = this.data.header; | 106 | this.topHeaderText.innerHTML = this.data.header; |
107 | 107 | ||
108 | this.element.addEventListener("click", this, false); | 108 | this.element.addEventListener("mousedown", this, true); |
109 | 109 | ||
110 | Object.defineBinding(this, "menuIsActive", { | 110 | Object.defineBinding(this, "menuIsActive", { |
111 | boundObject: this._menu, | 111 | boundObject: this._menu, |
diff --git a/js/components/menu/menu.reel/menu.js b/js/components/menu/menu.reel/menu.js index 50d3f0bc..1560100a 100755 --- a/js/components/menu/menu.reel/menu.js +++ b/js/components/menu/menu.reel/menu.js | |||
@@ -83,19 +83,12 @@ exports.Menu = Montage.create(Component, { | |||
83 | 83 | ||
84 | handleMousedown: { | 84 | handleMousedown: { |
85 | value: function(evt) { | 85 | value: function(evt) { |
86 | |||
87 | if(this.active && (this.getZIndex(evt.target) < 9000 || evt.target.id === "topMenu")) { | 86 | if(this.active && (this.getZIndex(evt.target) < 9000 || evt.target.id === "topMenu")) { |
88 | this._activeEntry.deselect(); | 87 | this._activeEntry.deselect(); |
89 | this._activeEntry = null; | 88 | this._activeEntry = null; |
90 | this.active = false; | 89 | this.active = false; |
91 | 90 | this.element.ownerDocument.removeEventListener('mousedown', this, false); | |
92 | //console.log(this.rep.objects[1]); | ||
93 | //this.controller.content[1].header = "BLAH"; | ||
94 | } | 91 | } |
95 | |||
96 | // console.log(evt.target.style['z-index']); | ||
97 | // console.log(this.getZIndex(evt.target)); | ||
98 | |||
99 | } | 92 | } |
100 | }, | 93 | }, |
101 | 94 | ||