Add a disabled property to DOMUI lists

chrome/browser/resources/shared/js/cr/ui/list.js

 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // require: listselectionmodel.js
 
 /**
  * @fileoverview This implements a list control.
  */
 
@@ skipping 78 matching lines @@
     itemHeight_: 0,
 
     /**
      * Whether or not the list is autoexpanding. If true, the list resizes
      * its height to accomadate all children.
      * @type {boolean}
      * @private
      */
     autoExpands_: false,
 
+    /**
+     * Whether or not the list is disabled.
+     * @type {boolean}
+     * @private
+     */
+    disabled_: false,

[arv (Not doing code reviews), 2010/12/04 01:03:27]

cr.defineProperty(List, 'disabled', cr.PropertyKind.BOOL_ATTR);

[stuartmorgan, 2010/12/04 01:36:51]

Ah-ha, this is the magic I was missing; I originally tried to do it as a
property but since it wasn't a real property CSS ignored it, and I didn't
realize I could declare it to make it work. Thanks for the pointer!

+
     dataModel_: null,
 
     /**
      * The data model driving the list.
      * @type {ListDataModel}
      */
     set dataModel(dataModel) {
       if (this.dataModel_ != dataModel) {
         if (!this.boundHandleDataModelSplice_) {
           this.boundHandleDataModelSplice_ =
@@ skipping 71 matching lines @@
       return this.autoExpands_;
     },
     set autoExpands(autoExpands) {
       if (this.autoExpands_ == autoExpands)
         return;
       this.autoExpands_ = autoExpands;
       this.redraw();
     },
 
     /**
+     * Whether or not the list is disabled.
+     * @type {boolean}
+     */
+    get disabled() {
+      return this.disabled_;
+    },
+    set disabled(disabled) {
+      this.disabled_ = disabled;
+      if (disabled)
+        this.classList.add('disabled');
+      else
+        this.classList.remove('disabled');
+    },
+
+    /**
      * Convenience alias for selectionModel.selectedItem
      * @type {cr.ui.ListItem}
      */
     get selectedItem() {
       var dataModel = this.dataModel;
       if (dataModel) {
         var index = this.selectionModel.selectedIndex;
         if (index != -1)
           return dataModel.item(index);
       }
@@ skipping 78 matching lines @@
     },
 
     /**
      * Callback for mousedown and mouseup events.
      * @param {Event} e The mouse event object.
      * @private
      */
     handleMouseDownUp_: function(e) {
       var target = e.target;
 
+      if (this.disabled)

[arv (Not doing code reviews), 2010/12/04 01:03:27]

Do this before "var target"

[stuartmorgan, 2010/12/04 01:36:51]

Done.

+        return;
+
       // If the target was this element we need to make sure that the user did
       // not click on a border or a scrollbar.
       if (target == this && !inViewport(target, e))
         return;
 
       while (target && target.parentNode != this) {
         target = target.parentNode;
       }
 
       if (!target) {
         this.selectionController_.handleMouseDownUp(e, -1);
       } else {
         var cs = getComputedStyle(target);
         var top = target.offsetTop -
                   parseFloat(cs.marginTop);
         var index = Math.floor(top / this.itemHeight_);
         this.selectionController_.handleMouseDownUp(e, index);
       }
     },
 
     /**
      * Handle a keydown event.
      * @param {Event} e The keydown event.
      * @return {boolean} Whether the key event was handled.
      */
     handleKeyDown: function(e) {
+      if (this.disabled)
+        return;
+
       return this.selectionController_.handleKeyDown(e);
     },
 
     /**
      * Callback from the selection model. We dispatch {@code change} events
      * when the selection changes.
      * @param {!cr.Event} e Event with change info.
      * @private
      */
     handleOnChange_: function(ce) {
@@ skipping 231 matching lines @@
         delete this.cachedItems_[index];
         this.redraw();
       }
     }
   };
 
   return {
     List: List
   }
 });