Changeset 490 for Dev/trunk/src/client/qed-client/widgets

Timestamp:

03/08/14 22:51:23 (11 years ago)

Author:

hendrikvanantwerpen

Message:

• Mark content as dirty to prevent moving away from unsaved data.
• Better change propagation from lists and our own widgets.
• Generate notifications for errors and show correct message.
• Moved all path/url generation to the class stores, not everywhere we use it.
• Give user always a choice between Save and Save & Close.
• Better refresh behaviour on form changes and saves.
• Don't generate duplicate code error when existing object is the one you're storing.

Location:

Dev/trunk/src/client/qed-client/widgets

Files:

• LineWithActionsWidget.js (modified) (3 diffs)
• ListWidget.js (modified) (8 diffs)

Dev/trunk/src/client/qed-client/widgets/LineWithActionsWidget.js

@@ -44 +44 @@
                             onClick: lang.hitch(this, function(action, e){
                                 if ( action.callback ) { action.callback(e); }
-                                event.stop(e);
+                                if ( e ) { event.stop(e); }
                                 return false;
                             }, this.actions[action])
@@ -58 +58 @@
                             onClick: lang.hitch(this, function(action, e){
                                 if ( action.callback ) { action.callback(e); }
-                                event.stop(e);
+                                if ( e ) { event.stop(e); }
                                 return false;
                             }, this.actions[action])
@@ -73 +73 @@
             var preventDefault = this.onClick(e) === false;
             if (preventDefault) {
-                event.stop(e);
+                if ( e ) { event.stop(e); }
             }
             return !preventDefault;

Dev/trunk/src/client/qed-client/widgets/ListWidget.js

@@ -8 +8 @@
     "dojo/dnd/Source",
     "dojo/dnd/common",
-    "dojo/dom-construct"
-], function(_Container, _WidgetBase, registry, array, declare, lang, Source, dnd, domConstruct) {
+    "dojo/dom-construct",
+    "dojo/on"
+], function(_Container, _WidgetBase, registry, array, declare, lang, Source, dnd, domConstruct, on) {
     return declare([_WidgetBase,_Container],{
         name: "",
@@ -49 +50 @@
             });
             this.source = new this.container(this.domNode,sourceParams);
+            this.source.on('Drop',lang.hitch(this,'_handleChange'));
         },
         creator: function(item, hint) {
@@ -66 +68 @@
                 }
             }
+            on(nodeOrWidget,'change',lang.hitch(this,'_handleChange'));
             var node = nodeOrWidget.domNode ? nodeOrWidget.domNode : nodeOrWidget;
             if ( hint !== "avatar" && node.id !== id ) {
@@ -79 +82 @@
         createListElement: null, /* function(id,item){},*/
         _getValueAttr: function() {
-            return array.map(this.source.getAllNodes(),function(node){
+            this.value = array.map(this.source.getAllNodes(),function(node){
                 var widget = registry.byNode(node);
                 if ( widget && "value" in widget ) {
@@ -87 +90 @@
                 }
             },this);
+            return this.value;
         },
         _setValueAttr: function(value) {
@@ -110 +114 @@
             }
         },
-        appendItems: function(items) {
+        appendItems: function(items,forceEvent) {
             this.source.insertNodes(false,items);
+            if ( forceEvent ) { this._handleChange(); }
         },
-        appendItem: function(item) {
+        appendItem: function(item,forceEvent) {
             this.source.insertNodes(false,[item]);
+            if ( forceEvent ) { this._handleChange(); }
         },
-        removeItem: function(key) {
+        removeItem: function(key,forceEvent) {
             array.forEach(array.filter(this.source.getAllNodes(), function(node){
                 return node.id === key;
             }), lang.hitch(this, "_destroyNodeOrWidget"));
             this.source.delItem(key);
+            if ( forceEvent ) { this._handleChange(); }
         },
         getChildren: function() {
@@ -127 +134 @@
             }),function(widget){ return widget !== null; });
         },
-        clear: function() {
+        clear: function(forceEvent) {
             array.forEach(this.source.getAllNodes(),
                           lang.hitch(this, "_destroyNodeOrWidget"));
             this.source.clearItems();
+            if ( forceEvent ) { this._handleChange(); }
         },
         _destroyNodeOrWidget: function(node) {
@@ -149 +157 @@
             this.source.destroy();
             this.inherited(arguments);
-        }
+        },
+        _handleChange: function() {
+            this.value = this._getValueAttr();
+            this.onChange(this.value);
+        },
+        onChange: function(value) {}
     });
 });