Changeset 1446

Timestamp:

2010-03-08 15:32:42 (2 years ago)

Author:

freya

Message:

Changes regarding creation of collection controls: make use of kauri-layout and kauri-cursor to define the layout of collection element and position of element. Still unstable and work in progress.

Location:

trunk

Files:

• modules/kauri-forms/kauri-forms-framework/src/main/kauri/static-{build}.key/kauri.forms/basic-controls.js (modified) (2 diffs)
• modules/kauri-forms/kauri-forms-framework/src/main/kauri/static-{build}.key/kauri.forms/collection.js (modified) (3 diffs)
• modules/kauri-forms/kauri-forms-framework/src/main/kauri/static-{build}.key/kauri.forms/composite.js (modified) (2 diffs)
• modules/kauri-forms/kauri-forms-framework/src/main/kauri/static-{build}.key/kauri.forms/control.js (modified) (21 diffs)
• modules/kauri-forms/kauri-forms-framework/src/main/kauri/static-{build}.key/kauri.forms/form.js (modified) (2 diffs)
• samples/kauri-forms-sample/src/main/kauri/pages/collection-control.html.xml (modified) (3 diffs)
• samples/kauri-forms-sample/src/main/kauri/pages/collection-performance.html.xml (modified) (1 diff)

trunk/modules/kauri-forms/kauri-forms-framework/src/main/kauri/static-{build}.key/kauri.forms/basic-controls.js

@@ -38 +38 @@
     $.extend(InputControl.prototype.templates, kf.Control.prototype.templates);
     $.extend(InputControl.prototype.templates, {
-           input:"<input type='text'/>"
+           input:"<input type='text' kauri-role='input'/>"
     });
 
@@ -56 +56 @@
         }
         if(! $input.is(selector)){
-            $input.html(this.getTemplate('input'));
+            var $element = $(this.getTemplate('input')); 
+
+            $input.each( function() {
+                $.each(this.attributes, function(i, attrib) {
+                    var name = attrib.name;
+                    var value = attrib.value;
+                    if (!$element.attr(name)) {
+                        $element.attr(name, value);
+                    }
+                });
+            });
+            $input.before($element).remove(); 
         }
         

trunk/modules/kauri-forms/kauri-forms-framework/src/main/kauri/static-{build}.key/kauri.forms/collection.js

@@ -63 +63 @@
     $.extend(CollectionControl.prototype.templates, {
         layout: "<tr><td ><span kauri-role='label'/><span kauri-role='input'/><span kauri-role='messages'/><span kauri-role='delete'/></td></tr>",
-        container: "<table kauri-role='container'></table>"
+        container: "<table kauri-role='container'></table>",
+        input: "<table kauri-role='input'></table>"
     });
+    
+    CollectionControl.prototype.groupChildrenInLayoutIndex = true;
     
     CollectionControl.prototype.initElements = function( ) {
@@ -87 +90 @@
         }
 
-        var container = this.getElement(kf.ControlElements.REV_CONTAINER);
+        var container = this.getElement(kf.ControlElements.REV_INPUT);
         var layout = this.getElement(kf.ControlElements.REV_LAYOUT);
 
@@ -271 +274 @@
         }
 
-        this._childrenContainer.empty();
+      // TODO check  this._childrenContainer.empty();
         this._children = [];
     }

trunk/modules/kauri-forms/kauri-forms-framework/src/main/kauri/static-{build}.key/kauri.forms/composite.js

@@ -80 +80 @@
     $.extend(CompositeControl.prototype.templates, kf.AbstractContainerControl.prototype.templates);
     $.extend(CompositeControl.prototype.templates, {
-           container: "<dl kauri-role='container'></dl>",
-           layout: "<dt kauri-role='label'/><dd><span kauri-role='input'/><span kauri-role='messages'/></dd>"    
+           input: "<dl kauri-role='input'></dl>",
+           layout: "<dt kauri-role='label'/><dd><span kauri-role='input'/><span kauri-role='messages'/></dd>",
+           "kauri-layout": "<dt kauri-role='label'/><dd><span kauri-role='input'/><span kauri-role='messages'/></dd>"
     });
 
@@ -102 +103 @@
              throw "[CompositeControl#createChildElements] No container to work with.";           
         } 
-        var item = $(this.getTemplate('layout')).appendTo(container);
-        return item;
+        // TODO check why this has to happen?        
+        //var item = $(this.getTemplate('layout')).appendTo(container);
+        //return item;
     }
 

trunk/modules/kauri-forms/kauri-forms-framework/src/main/kauri/static-{build}.key/kauri.forms/control.js

@@ -39 +39 @@
         this.elementIndex = {};
         
+        // index of layout cursor elements
+        this.layoutIndex = {};
+        
     }
     
@@ -56 +59 @@
     
     /**
+     * Html attribute name used for specifying the role
+     * @static
+     * @final
+     */
+    ControlElements.ATTR_REV = "kauri-role";
+
+    /**
+     * Html attribute name used for specifying the layout relation.
+     * @static
+     * @final
+     */
+    ControlElements.ATTR_LAYOUT = "kauri-layout";
+
+    /**
+     * Html attribute name used for specifying the layout cursor position
+     * @static
+     * @final
+     */
+    ControlElements.ATTR_LAYOUT_CURSOR = "kauri-layout-cursor";
+
+    /**
+     * Html attribute name used for specifying the layout clone id
+     * @static
+     * @final
+     */
+    ControlElements.ATTR_LAYOUT_CLONE = "kauri-layout-clone";
+
+    
+    /**
+     * Html attribute name used for specifying the layout group which the element belongs to
+     * @static
+     * @final
+     */
+    ControlElements.ATTR_LAYOUT_GROUP = "kauri-layout-group";
+
+    
+    /**
      * Html attribute name used for specifying the relation. Rev is short for reverse index
      * @static
@@ -61 +101 @@
      */
     ControlElements.ATTR_REV = "kauri-role";
+
     
     /**
@@ -105 +146 @@
     ControlElements.REV_MARK = "mark";
     
-    /**
-     * Specifies that the element is a container for the control
+
+    /**
+     * TODO this one should be removed, it is remplaces by the kauri-layout attribute
      * @static
      * @final
      */
     ControlElements.REV_CONTAINER = "container";
-    
-    /**
-     * Specifies that the element should be used as a layout or template for child elements
-     * @static
-     * @final
-     */
     ControlElements.REV_LAYOUT = "layout";
+
     
     /**
@@ -163 +200 @@
             
             var rev = $this.attr(ControlElements.ATTR_REV);
-            
             store.addElementIndex(index, rev, $this);
             
@@ -172 +208 @@
         };
 
-        var wrapIndexor = function($wrap){
-            //var $first = $($eltArray[0]);
-            index = ""; 
-            var idref = $wrap.attr(ControlElements.ATTR_IDREF) || "";
-            if (idref.length == 0 || idref.charAt(0) != '/') 
-                index = $wrap.parents("[" + ControlElements.ATTR_INDEX + "]:first").attr(ControlElements.ATTR_INDEX);
-            if (idref) {
-                index = $.concatPath(index, idref);
-                
-            }
-            
-            var rev = $wrap.attr(ControlElements.ATTR_REV);
-            
-            store.addElementIndex(index, rev, $wrap.children());
-            
-            //add type property if known
-          //  store.addTypeAndControl(index, $elt.attr(ControlElements.ATTR_TYPE), $this.attr(ControlElements.ATTR_CONTROL));
-
-            $wrap.attr(ControlElements.ATTR_INDEX, index);
-        };
-        
         // find all kauri-idref and kauri-role marked elements and index them
-        var kauriSelector = "[" + ControlElements.ATTR_IDREF + "],[" + ControlElements.ATTR_REV + "]";
-
-        
+        var kauriSelector = "[" + ControlElements.ATTR_IDREF + "]";
+        var layoutSelector = "[kauri-layout]";
         
         // begin by index space itself,
-        if (space.is(kauriSelector)) 
+        if (space.is(kauriSelector) ) 
             space.each(indexor);
             // then go through matching children, but only once (even if they match multiple times)
@@ -205 +219 @@
             // TODO should we do this this way or should be traverse the tree ourselves (performance?)
             var $this = $(this);
-            var revParents = $this.parents("[" + ControlElements.ATTR_REV + "='" + ControlElements.REV_LAYOUT + "']");
-
-            return ($this.attr(ControlElements.ATTR_IDREF)!=undefined || revParents == null || revParents.length==0);
-        });   
-    
-        matching.each(indexor);
+            var revParents = $this.parents("[" + ControlElements.ATTR_LAYOUT + "]");
+
+            return (revParents == null || revParents.length==0);
+        });      
+        $.unique($(kauriSelector, space)).each(indexor);
+        
+        
+        var matching = $($.unique($("[kauri-layout-cursor]", space))).filter(function(){
+            var $this = $(this);
+            var revParents = $this.parents("[kauri-layout]");
+            return (revParents == null || revParents.length==0);
+        });  
+        
+        $.each(matching, function(pos, element){
+            var $element = $(element);
+            store.addCursorIndex($element.attr("kauri-layout-cursor"), $element);
+        });
+        
+        
+        if (space.is(layoutSelector)) 
+            store.addLayoutIndex(space.attr("kauri-layout"), space);
+        
+        var matchingLayout = $($.unique($(layoutSelector, space))).filter(function(){
+            var $this = $(this);
+            var revParents = $this.parents("[kauri-layout]");
+            return (revParents == null || revParents.length==0);
+        });  
+
+        $.each(matchingLayout, function(pos, element){
+            var $element = $(element);
+            store.clearLayoutIndex($element.attr("kauri-layout"));
+        });
+
+        $.each(matchingLayout, function(pos, element){
+            var $element = $(element);
+            store.addLayoutIndex($element.attr("kauri-layout"), $element);
+        });
+
     }
     
@@ -289 +335 @@
                 $elm = form.getElementIndex(index)[0];
             
-            
-                
                 if ($elm) {
                     $elm.attr(ControlElements.ATTR_INDEX, index);
@@ -304 +348 @@
     }
 
+     
+     ControlElements.lookupLayout = function(control) {
+
+        var form = control.getForm();
+        var index = control.getGroupId();
+
+        var $elm = form.getLayoutIndex(index); 
+
+        if ($elm)
+            control.setElement(ControlElements.ATTR_LAYOUT, $elm);
+        return $elm;
+    }
+
+    ControlElements.lookupLayoutCursor = function(control) {
+        var form = control.getForm();
+        var index = control.getGroupId();
+
+        var $elm = form.getCursorIndex(index); 
+        
+        if ($elm)
+            control.setElement(ControlElements.ATTR_LAYOUT_CURSOR, $elm);
+        return $elm;
+    }
+     
+     
     /**
      * Finds all elements associated to a certain control
@@ -367 +436 @@
         return empty;
     }
-    
+
+     ControlElements.newLayoutElements = function() {
+        var empty = [];
+        empty.layout = {};
+        empty.cursor = {};
+        return empty;
+    }
+
     
     /**
@@ -513 +589 @@
         });
     }
-    
+
+     /**
+      * Adds the specified element to the index.
+      *
+      * @param {string}
+      *            index index position to add to.
+      * @param {string}
+      *            [group] the group part we're adding a layout element to
+      * @param {$jquery}
+      *            elm element to add
+      */
+     ControlElements.prototype.addLayoutIndex = function(index, $element){
+         var elms = this.layoutIndex[index] = this.layoutIndex[index] || ControlElements.newLayoutElements();
+         var group = $element.attr("kauri-group");
+         if(!group)
+             group="1"; // TODO
+    
+         var layoutgroup = elms.layout[group] = elms.layout[group] || [];
+         layoutgroup.push($element);
+     }
+
+      ControlElements.prototype.clearLayoutIndex = function(index){
+          this.layoutIndex[index] = ControlElements.newLayoutElements();
+      }
+
+      
+     /**
+      * Gets the specified layout JQuery wrapper element from the index.
+      *
+      * @param {string}
+      *            [index] index to get. <code>undefined</code> will return the whole store.
+      * @param {group}
+      *            [group] the group layout part to get from the index
+      */
+     ControlElements.prototype.getLayoutIndex = function(index, group){
+     
+         if (index == undefined) 
+             return this.layoutIndex;
+         
+         var $elm;
+         //this.layoutIndex[index] = this.layoutIndex[index] || ControlElements.newLayoutElements();
+
+         if(this.layoutIndex[index]) {
+             
+             if(group == undefined)
+                 return this.layoutIndex[index].layout;
+             else         
+                 return this.layoutIndex[index].layout[group];
+         }else
+             return null;
+      }
+
+      /**
+       * Adds the specified element to the cursor index.
+       *
+       * @param {string}
+       *            index index position to add to.
+       * @param {$jquery}
+       *            elm element to add
+       */
+      ControlElements.prototype.addCursorIndex = function(index, $element){
+          var elms = this.layoutIndex[index] = this.layoutIndex[index] || ControlElements.newLayoutElements();
+          elms.cursor = $element;
+      }
+      
+      /**
+       * Gets the specified layout JQuery wrapper element from the cursor index.
+       *
+       * @param {string}
+       *            [index] index to get. <code>undefined</code> will return the whole store.
+       */
+      ControlElements.prototype.getCursorIndex = function(index){
+      
+          if (index == undefined) 
+              return this.layoutIndex;
+          
+         return (this.layoutIndex[index]) ? this.layoutIndex[index].cursor : null;
+       }
+
+
     
     /**
@@ -605 +760 @@
     };
     
+    
+    /**
+     * Holds the control-bound HTML templates per relation
+     * @private
+     */
+    Control.prototype.groupInLayoutIndex = false;
+    
+    
     /**
      * Initializes element configurations
@@ -667 +830 @@
         
         var $elements = this._elements[relation];
+        
+        if(relation == ControlElements.REV_MARK && !$elements)
+            $elements =  this._elements["input"];
+        
         return $elements;
     }
@@ -870 +1037 @@
         
         var $input = this.getElement();
-        $input.addClass("unfocused");
-        $input.blur(function(){
-            $(this).removeClass("focused").addClass("unfocused");
-        });
-        $input.focus(function(){
-            $(this).addClass("focused").removeClass("unfocused");
-        });
+        if($input) {
+            $input.addClass("unfocused");
+            $input.blur(function(){
+                $(this).removeClass("focused").addClass("unfocused");
+            });
+            $input.focus(function(){
+                $(this).addClass("focused").removeClass("unfocused");
+            });
+        }
         
         var $update = this.getUpdateElement();
@@ -1325 +1494 @@
         
         var mrkElm = this.getElement(ControlElements.REV_MARK);
-        if (msg) 
-            mrkElm.removeClass("valid").addClass("invalid");
-        else 
-            mrkElm.removeClass("invalid").addClass("valid");
-        
+        
+        if(mrkElm){ 
+            if (msg) 
+                mrkElm.removeClass("valid").addClass("invalid");
+            else 
+                mrkElm.removeClass("invalid").addClass("valid");
+        }
     }
     
@@ -1364 +1535 @@
         return $.concatPath(parentId, id);
     }
-    
+
+    /**
+    * Gets this controls absolute id. This means including all the ids of possible parent controls
+    * @return absolute id or id-path
+    * @type String
+    */
+   Control.prototype.getGroupId = function(){
+   
+       var parent = this.getParent();
+       var parentId = "";
+       if (parent) 
+           parentId = parent.getGroupId();
+       
+       var id = (this.getParent() && this.getParent().groupChildrenInLayoutIndex) ? "#" : this.getId();
+       return $.concatPath(parentId, id);
+   }
+
     
     // TODO support 'match-patterns' allowing to return arrays of controls on which some jqyery like each() can then execute
@@ -1454 +1641 @@
     Control.prototype.enable = function(){
         for (var rel in this._elements) {
-            var el = this._elements[rel];
+            var el = this.getElement(rel);
             if (el.is(":disabled")) {
                 el.removeAttr("disabled");
@@ -1569 +1756 @@
         input: "<div><span kauri-role='label'/><span kauri-role='input'/><span kauri-role='messages'/></div>",
         layout: "<div><span kauri-role='label'/><span kauri-role='input'/><span kauri-role='messages'/></div>",
+        "kauri-layout": "<div><span kauri-role='label'/><span kauri-role='input'/><span kauri-role='messages'/></div>",
         container: "<div kauri-role='container'></div>"
     });
@@ -1768 +1956 @@
      */
     AbstractContainerControl.prototype.initContainerElements = function(create){
-    
-        var container = ControlElements.lookup(this, ControlElements.REV_CONTAINER);
-        var $container = $(container);
-        if (!container) { 
-            var creator = this.getTemplate(ControlElements.REV_CONTAINER);
-            if (creator) {
-                $container = $(creator).appendTo($(this.getElement("input")));
-                this.setElement(ControlElements.REV_CONTAINER, $container);
-            }
-        }
-
-        if (!$container) { // default to complete input space 
-            this.setElement(ControlElements.REV_CONTAINER, this.getElement("input"));
-        }
-        
-        var layout = ControlElements.lookup(this, ControlElements.REV_LAYOUT);
-        if (!layout) { 
-            var creator = this.getTemplate(ControlElements.REV_LAYOUT);
-            if (creator) {
-                $layout = $(creator).appendTo($container);
-                this.setElement(ControlElements.REV_LAYOUT, $layout);
-            }
-        }
-
-       /* if (layout) {
-            layout.hide();
-        }*/
+         var container = ControlElements.lookup(this, ControlElements.REV_INPUT);
+         
+         
+         // check if there is a layout attached to this element
+         // if not,  switch the apply the input template from the control and append to the input container
+         var id = this.getAbsoluteId();
+         if(!this.getForm().getLayoutIndex(id)){
+             var creator = this.getTemplate(ControlElements.REV_INPUT);
+             if (creator) {
+                 var form = this.getForm();
+                 var $input = $(creator);
+                 $container = $input.appendTo(container);
+                 this.setElement(ControlElements.REV_INPUT, $input);
+             }    
+         }
+         
+         var $container = $(container);
+         if (!container) { 
+             
+         }
+
+         if (!$container) { // default to complete input space 
+             this.setElement(ControlElements.REV_INPUT, this.getElement("input"));
+         }
     }
     
@@ -1800 +1985 @@
     AbstractContainerControl.prototype.getUpdateElement = function(){
     };
+    
+    
     
     /**
@@ -1817 +2004 @@
      */
     AbstractContainerControl.prototype._createChildElements = function(id, position) {
-    
-        var newIndex = $.concatPath(this.getAbsoluteId(), id);
-        var elements = this.getForm().getElementIndex(newIndex);
+        var absoluteId = this.getAbsoluteId();
+        var newIndex = $.concatPath(absoluteId, id);
+        var form = this.getForm();
+        var elements = form.getElementIndex(newIndex);
         if (elements && (elements.length > 0 || elements.meta[ControlElements.REV_INPUT])) 
             return elements; // no need to create new elements
-        var container = this.getElement(ControlElements.REV_CONTAINER);
-        var layout = this.getElement(ControlElements.REV_LAYOUT);
-        
-        var newElements
-        if (!(container && layout)) {
-            newElements = this.createChildElements(id, container, layout);
-        }
-        else {
-            // there is a container and there is a layout
-            var childAtPosition = null;
-            if (position != undefined && (typeof position == "number")) {
-                childAtPosition = container.children(":nth-child(" + (position + 1) + ")");
-            }
-            var $layout = layout.clone();
-
-            if (childAtPosition != null && childAtPosition.length == 1) {
-                newElements = $layout.insertBefore(childAtPosition[0]).show();
-            } else {
-                newElements = $layout.appendTo(container).show();
-            }
+        
+        var newElements;
+        var newElementsArray = [];
+
+        var form = this.getForm();
+        var groupId = this.getGroupId();
+        var containerId = this.getId();
+        var $layoutGroups = form.getLayoutIndex(groupId);
+
+        
+        var copyIdRef = function(idx, elt){
+            var $elt = $(elt);
+            if($elt.attr(ControlElements.ATTR_IDREF))
+                $elt.attr(ControlElements.ATTR_IDREF, $elt.attr(ControlElements.ATTR_IDREF) + "/" +id);
+            else
+                $elt.attr(ControlElements.ATTR_IDREF, id);
+        };
+        
+        
+        if(!$layoutGroups) {
+            // get layout from template and where to put cursor?? default append to input element??
+            var container = this.getElement(ControlElements.REV_INPUT);
             
+            var $layout = $(this.getTemplate("layout"));
             
-            //newElements.attr(ControlElements.ATTR_REV, ControlElements.REV_ITEM);
-
-            if(true){
-                // hier item index berekenen en in elementindex steken
-                // daarna gewoon de elemennten indexeren
-                
-                var idref = id;
-                if (idref.length == 0 || idref.charAt(0) != '/') 
-                    index = newElements.parents("[" + ControlElements.ATTR_INDEX + "]:first").attr(ControlElements.ATTR_INDEX);
-                if (idref) {
-                    index = $.concatPath(index, idref);
-                }
-                
-                this.getForm().addElementIndex(index, ControlElements.REV_ITEM, newElements);
-                
-/*                
-                var $wrap = $("<kauri-layout/>");
-                $wrap.append(newElements.clone());
-
-                $wrap.attr(ControlElements.ATTR_REV, ControlElements.REV_ITEM);
-                $wrap.attr(ControlElements.ATTR_IDREF, id);
-                //newElements = $wrap;
-                var form = this.getForm();
-                ControlElements.index($wrap, form, true);
-                return newElements;*/
-            }
-
-        }
-        
-        newElements.attr(ControlElements.ATTR_IDREF, id);
-        var form = this.getForm();
-        ControlElements.index(newElements, form, true); // we force re-indexing cause layout-template elements will already be
-        // indexed
-        return newElements;
+            $layout.each(copyIdRef);
+            
+            $.each($("[kauri-role]", $layout), copyIdRef);
+            
+            newElements = container.append($layout);
+            ControlElements.index($layout, form, true);
+            newElementsArray.push($layout);
+        }else{
+            // clone each element in layout group
+            $.each($layoutGroups, function(layoutGroupId, eltArray){
+                $.each(eltArray, function(eltId, elt){
+                    var $elt = $(elt);
+                    var $layoutClone = $elt.clone();
+                    $layoutClone.attr("kauri-layout-clone", absoluteId + "(" + groupId + ")");
+                    // set the id of the newly created row  before the idref
+                    if($layoutClone.attr(ControlElements.ATTR_IDREF))
+                        $layoutClone.attr(ControlElements.ATTR_IDREF, $layoutClone.attr(ControlElements.ATTR_IDREF) + "/" +id);
+                    else
+                        $layoutClone.attr(ControlElements.ATTR_IDREF, id);
+                    $layoutClone.removeAttr( ControlElements.ATTR_LAYOUT );
+                    // search for cursor and insert before cursor
+                    var cursorId = groupId + "(" + layoutGroupId + ")" ;
+                    var cursor = form.getCursorIndex(cursorId);
+                    // make sure the clone is visible
+
+                    if(!cursor) 
+                        cursor = $elt;
+                    $layoutClone.show();
+                    newElements = $layoutClone.insertBefore(cursor);
+                    ControlElements.index(newElements, form, true);
+                    newElementsArray.push(newElements);
+                });
+            });
+        }
+
+
+        return newElementsArray;
     }
     

trunk/modules/kauri-forms/kauri-forms-framework/src/main/kauri/static-{build}.key/kauri.forms/form.js

@@ -142 +142 @@
         $.extend(this, new kf.ControlElements());
 
+        // hide all layout and cursor elements
+        var $layoutElts = $("[" + kf.ControlElements.ATTR_LAYOUT + "], [" + kf.ControlElements.ATTR_LAYOUT_CURSOR + "]", space);
+        $.each($layoutElts, function(index, element){
+            $(element).hide();
+          }
+        );
+
         space.attr(kf.ControlElements.ATTR_INDEX, '/');
         kf.ControlElements.index(space, this);
+        
+        
 
         this.setRootElement(space);
@@ -164 +173 @@
     Form.prototype.getAbsoluteId = function() {
 
-        return "/"
+        return "/";
     };
 
+    Form.prototype.getGroupId = function() {
+
+        return "/";
+    };
+
+    
     Form.prototype.setAllowInvalid = function(allow) {
 

trunk/samples/kauri-forms-sample/src/main/kauri/pages/collection-control.html.xml

@@ -86 +86 @@
         <label kauri-role="label" kauri-idref="countries">Countries</label>
 
-        <table kauri-idref="countries" kauri-type="collection">
+        <table kauri-idref="countries" kauri-type="collection" kauri-item="country">
           <thead>
             <tr>
@@ -99 +99 @@
 
           <tbody kauri-role="container">
-            <tr kauri-role="layout">
-              <td>
+            <tr kauri-layout="country">
+            
+              <td >
                 <input kauri-idref="name" kauri-type="string"/>
               </td>
@@ -114 +115 @@
                 <span class="action copyAction">copy</span>
               </td>
+            
             </tr>
+            
           </tbody>
 

trunk/samples/kauri-forms-sample/src/main/kauri/pages/collection-performance.html.xml

@@ -87 +87 @@
           </thead>
 
-          <tbody kauri-role="container">
-            <tr kauri-role="layout">
+          <tbody>
+            <tr kauri-layout="/performances">
               <td>
                 <input kauri-idref="task" kauri-type="string" size="10"/>