From 8bea1e0807f36595d762592c030d4810396ada85 Mon Sep 17 00:00:00 2001
From: Eric Guzman
Date: Wed, 2 May 2012 15:30:27 -0700
Subject: CSS Panel - Add focus management to styles view delegate

---
 js/panels/css-panel/style.reel/style.css  |  2 +-
 js/panels/css-panel/style.reel/style.html |  8 +++
 js/panels/css-panel/style.reel/style.js   | 83 ++++++++++++++++++++++---------
 3 files changed, 69 insertions(+), 24 deletions(-)

(limited to 'js/panels/css-panel/style.reel')

diff --git a/js/panels/css-panel/style.reel/style.css b/js/panels/css-panel/style.reel/style.css
index c6bb070c..6864e245 100644
--- a/js/panels/css-panel/style.reel/style.css
+++ b/js/panels/css-panel/style.reel/style.css
@@ -111,7 +111,7 @@
     right: -1px;
 }
 .style-item .hintable-hint {
-    color: #CCC;
+    color: #bdbdbd;
 }
 
 /* ------------------------
diff --git a/js/panels/css-panel/style.reel/style.html b/js/panels/css-panel/style.reel/style.html
index acc169be..9830a21e 100644
--- a/js/panels/css-panel/style.reel/style.html
+++ b/js/panels/css-panel/style.reel/style.html
@@ -58,6 +58,10 @@ No rights, expressed or implied, whatsoever to this software are provided by Mot
                     {
                         "type": "change",
                         "listener": {"@": "owner"}
+                    },
+                    {
+                        "type": "stop",
+                        "listener": {"@": "owner"}
                     }
                 ]
             },
@@ -80,6 +84,10 @@ No rights, expressed or implied, whatsoever to this software are provided by Mot
                     {
                         "type": "change",
                         "listener": {"@": "owner"}
+                    },
+                    {
+                        "type": "stop",
+                        "listener": {"@": "owner"}
                     }
                 ]
             }
diff --git a/js/panels/css-panel/style.reel/style.js b/js/panels/css-panel/style.reel/style.js
index c73da6e5..1d5a11a5 100644
--- a/js/panels/css-panel/style.reel/style.js
+++ b/js/panels/css-panel/style.reel/style.js
@@ -8,27 +8,14 @@ var Montage = require("montage/core/core").Montage,
     TreeNode = require("js/components/treeview/tree-node").TreeNode;
 
 exports.Style = Montage.create(TreeNode, {
-    delegate : {
-        value: null
-    },
-    disabledClass : {
-        value: 'style-item-disabled'
-    },
-    editingStyleClass : {
-        value: 'edit-style-item'
-    },
-    editNewEmptyClass : {
-        value: 'edit-empty-style'
-    },
-    invalidStyleClass : {
-        value: "style-item-invalid"
-    },
-    propertyText : {
-        value: "property"
-    },
-    _valueText : {
-        value: "value"
-    },
+    delegate          : { value: null },
+    disabledClass     : { value: 'style-item-disabled' },
+    editingStyleClass : { value: 'edit-style-item' },
+    editNewEmptyClass : { value: 'edit-empty-style' },
+    invalidStyleClass : { value: "style-item-invalid" },
+
+    propertyText : { value: "property" },
+    _valueText : { value: "value" },
     valueText : {
         get: function() {
             return this._valueText;
@@ -158,6 +145,27 @@ exports.Style = Montage.create(TreeNode, {
         }
     },
 
+    getSiblingStyle : {
+        value: function(which) {
+            var styles = this.parentComponent.parentComponent.childComponents.map(function(sub){
+                    return sub.childComponents[0];
+                }),
+                index = styles.indexOf(this);
+
+            switch (which) {
+                case "first":
+                    return styles[0];
+                case "last":
+                    return styles[styles.length-1];
+                case "next":
+                    return (index+1 < styles.length) ? styles[index+1] : null;
+                case "prev":
+                    return (index-1 >= 0) ? styles[index-1] : null;
+            }
+        }
+    },
+
+
     handleEvent : {
         value: function(e) {
             console.log(e);
@@ -206,11 +214,40 @@ exports.Style = Montage.create(TreeNode, {
     handleStart : {
         value: function(e) {
             this.editing = true;
+
+            if(this.empty) {
+                this.editingNewStyle = true;
+            }
         }
     },
 
     //// Handler for both hintable components
-    handleStop : {
+    handlePropertyStop : {
+        value: function(e) {
+            var event = e;
+            ///// Function to determine if an empty (new) style should return
+            ///// to showing the add button, i.e. the fields were not clicked
+            function fieldsClicked() {
+                var clicked;
+                if(e._event.detail.originalEventType === 'mousedown') {
+                    clicked = e._event.detail.originalEvent.target;
+                    return clicked === this.propertyField.element || clicked === this.valueField.element;
+                }
+                return false;
+            }
+
+            this.editing = false;
+
+            if(this.sourceObject.isEmpty && !this.dirty && !fieldsClicked.bind(this)()) {
+                ///// Show add button
+                this.editingNewStyle = false;
+            }
+
+            this.delegate.handlePropertyStop(e, this);
+        }
+    },
+    //// Handler for both hintable components
+    handleValueStop : {
         value: function(e) {
             var event = e;
             ///// Function to determine if an empty (new) style should return
@@ -231,7 +268,7 @@ exports.Style = Montage.create(TreeNode, {
                 this.editingNewStyle = false;
             }
 
-            this.delegate.handleStyleStop(e);
+            this.delegate.handleValueStop(e, this);
         }
     },
 
-- 
cgit v1.2.3