Merge branch 'refs/heads/master' into CSSPanelUpdates

2 files changed, 31 insertions, 98 deletions

diff --git a/js/panels/Materials/materials-library-panel.reel/materials-library-panel.html b/js/panels/Materials/materials-library-panel.reel/materials-library-panel.html
index eede02bd..6ba4ac75 100755
--- a/js/panels/Materials/materials-library-panel.reel/materials-library-panel.html
+++ b/js/panels/Materials/materials-library-panel.reel/materials-library-panel.html
@@ -10,14 +10,18 @@
         <link rel="stylesheet" type="text/css" href="materials-library-panel.css">
          <script type="text/montage-serialization">
         {
+             "materialInfo": {
+                 "module": "js/panels/Materials/materials-popup.reel",
+                 "name": "MaterialsPopup"
+             },
             "owner": {
                 "module": "js/panels/Materials/materials-library-panel.reel",
                 "name": "MaterialsLibraryPanels",
                 "properties": {
-                    "element": {"#": "materials_library_panel"}
+                    "element": {"#": "materials_library_panel"},
+                    "_materialInfo": {"@": "materialInfo"}
                 }
             }
-
         }
                 </script>
                 
diff --git a/js/panels/Materials/materials-library-panel.reel/materials-library-panel.js b/js/panels/Materials/materials-library-panel.reel/materials-library-panel.js
index f97e1a27..b16c4376 100755
--- a/js/panels/Materials/materials-library-panel.reel/materials-library-panel.js
+++ b/js/panels/Materials/materials-library-panel.reel/materials-library-panel.js
@@ -7,7 +7,8 @@ No rights, expressed or implied, whatsoever to this software are provided by Mot
 var Tree = require("js/components/tree.reel").Tree,
     Button = require("js/components/button.reel").Button,
     MaterialsPopup = require("js/panels/Materials/materials-popup.reel").MaterialsPopup,
-    PopupMananger =             require("js/components/popup-manager.reel").PopupMananger;
+    PopupMananger =             require("js/components/popup-manager.reel").PopupMananger,
+    Popup = require("montage/ui/popup/popup.reel").Popup;
 
 exports.MaterialsLibraryPanel = (require("montage/core/core").Montage).create(require("montage/ui/component").Component, {
 
@@ -92,20 +93,7 @@ exports.MaterialsLibraryPanel = (require("montage/core/core").Montage).create(re
     captureChange: {
         value:function(e) {
             var tNode = e._event.treeNode;
-            var left, top;
-            //TODO: Figure out if this is the best way to detect where user clicked
-            var mouseEvent = e._event.mouseEvent;
-                if (mouseEvent.clientX && mouseEvent.clientY) {
-                        if (mouseEvent.clientX > (parseInt(document.width)/2)) {
-                    left = mouseEvent.clientX - mouseEvent.offsetX-2;
-                    top = mouseEvent.currentTarget.clientHeight/2+mouseEvent.clientY - mouseEvent.offsetY;
-                        } else {
-                    left = mouseEvent.clientX - mouseEvent.offsetX+parseInt(mouseEvent.currentTarget.clientWidth);
-                    top = mouseEvent.clientY - mouseEvent.offsetY;
-                        }
-                }
-            
-            this._showMaterialPopup(left + 'px', top + 'px', 'right', 'top', tNode.id);
+            this._showMaterialPopup(tNode.id);
         }
     },
 
@@ -113,93 +101,34 @@ exports.MaterialsLibraryPanel = (require("montage/core/core").Montage).create(re
         enumerable:true,
         value:null
     },
+
+    _materialInfo: {
+        enumerable:true
+    },
     
     _showMaterialPopup: {
         enumerable: false,
-        value: function (x, y, side, align, materialID) {
-                if (this._materialPopup && this._materialPopup.opened) {
-                        if (this._materialPopup.popup.position.x === x && this._materialPopup.popup.position.y === y) {
-                                        this._hideMaterialPopup();
-                        } else {
-                                this._materialPopup.popup.position = {x: x, y: y};
-                                this._materialPopup.popup.tooltip = {side: side, align: align};
-                    this._materialPopup.popup.base.loadMaterials(materialID);
-                                //TODO: Tooltip needs to be fixed to allow aligning to change on fly
-                                //this._materialPopup.popup.drawTooltip();
-                        }
-                } else {
-                        ////////////////////////////////////////////////////
-                        //Creating popup from m-js component
-                        var popup = document.createElement('div');
-                        var content = document.createElement('div');
-                        var popupBase = MaterialsPopup.create();
-
-
-                        //TODO: Check to see if this HACK is needed
-                        //(elements needs to be on DOM to be drawn)
-                        document.body.appendChild(popup);
-                        popupBase.element = popup;
-                                popupBase.needsDraw = true;
-                                document.body.removeChild(popup);
-                        //Adding drawn element to container
-                        content.appendChild(popupBase.element);
-
-                        //Creating an instance of the popup and sending in created material popup content
-                this._materialPopup = {};
-                        this._materialPopup.popup = PopupMananger.createPopup(content, {x: x, y: y}, {side: side, align: align});
-                                this._materialPopup.popup.element.style.opacity = 0;
-                this._materialPopup.popup.base = popupBase;
-                                popupBase._material = MaterialsLibrary.getMaterial( materialID );
-                                popupBase._materialName = materialID;
-                                //TODO: Fix this animation/draw HACK (Move to didDraw callback)
-                                setTimeout(function () {
-                                        this._materialPopup.popup.element.style.opacity = 1;
-                    this._materialPopup.popup.base.loadMaterials(materialID);
-                                }.bind(this), 150);
-
-
-
-                        //Popup was added, so it's opened
-                        this._materialPopup.opened = true;
-                        //TODO: Fix this HACK, it listens to this canvas to be clicked to close popup
-                        document.getElementById('stageAndScenesContainer').addEventListener('click', this, false);
-                }
-        }
-    },
-    ////////////////////////////////////////////////////////////////////
-    //
-    handleClick: {
-        enumerable: true,
-        value: function (e) {
-                //TODO: Fix this HACK
-                if (e._event.target.id === 'stageCanvas' && this._materialPopup.opened) {
-                        this._handleDocumentClick(e);
-                }
-        }
-    },
-    ////////////////////////////////////////////////////////////////////
-    //
-    _handleDocumentClick: {
-        enumerable: false,
-        value: function (e) {
-                this._hideMaterialPopup();
-                //TODO: Fix this HACK
-                        document.getElementById('stageAndScenesContainer').removeEventListener ('click', this, false);
+        value: function (materialID) {
+
+            if(!this._materialPopup)
+            {
+                this._materialPopup = Popup.create();
+                this._materialPopup.content = this._materialInfo;
+                this._materialPopup.modal = false;
+                this.eventManager.addEventListener("hideMaterialPopup", this, false);
+                this._materialPopup.addEventListener("show", this, false);
+            }
+            this._materialPopup.show();
+            this._materialInfo.loadMaterials(materialID);
         }
     },
-    ////////////////////////////////////////////////////////////////////
-    //
-    _hideMaterialPopup: {
+
+    handleHideMaterialPopup: {
         enumerable: false,
-        value: function () {
-                if (this._materialPopup.opened) {
-                        //
-                        PopupMananger.removePopup(this._materialPopup.popup.element);
-                        this._materialPopup.opened = false;
-                        //TODO: Fix HACK of removing popup
-                        this._materialPopup.popup.base.destroy();
-                        this._materialPopup.popup = null;
-                }
+        value: function (event) {
+            if(this._materialPopup){
+                this._materialPopup.hide();
+            }
         }
     }