Changeset 503 for Dev/trunk/src/client/qed-client/model/widgets/QuestionEditorPreviewItem.js

Timestamp:

03/11/14 01:00:40 (11 years ago)

Author:

hendrikvanantwerpen

Message:

Don't delete elements from certain lists if they are disabled
or readOnly. Didn't include this in ListWidget?.removeItem, since
programmatically things should keep working even when disabled, I would
say. So this needs to be done in the place of the event.

File:

• Dev/trunk/src/client/qed-client/model/widgets/QuestionEditorPreviewItem.js (modified) (2 diffs)

Dev/trunk/src/client/qed-client/model/widgets/QuestionEditorPreviewItem.js

@@ -33 +33 @@
                         lang.hitch(this, 'onToggleFold')));
             this.own(this.removeButton.on('click',
-                        lang.hitch(this, 'onDestroy')));
+                        lang.hitch(this, '_handleDestroy')));
             this.own(this.editButton.on('click',
                         lang.hitch(this, 'onToggleEdit')));
             this.showEdit();
         },
-        onDestroy: function() {},
+        _handleDestroy: function(evt) {
+            if ( !( this.disabled || this.readOnly ) ) {
+                this.emit('destroy');
+            }
+            if ( evt ) { event.stop(evt); }
+            return false;
+        },
         _getValueAttr: function(value) {
             if ( this._editing ) {
@@ -102 +108 @@
                 this.innerWidget = newWidget;
                 this.addChild(this.innerWidget);
+                this.innerWidget.set('readOnly',this.readOnly);
+                this.innerWidget.set('disabled',this.disabled);
                 this.titleNode.innerHTML = this.value.type+" [editing]";
                 domClass.replace(this.editButton.iconNode, "rftIconAccept", "rftIconEdit");