[Laszlo-dev] [Fwd: For Review: Change 20070905-maxcarlson-z Summary: Update replicator with suggestions from a.bargull@intensis.de]

[P T Withington]

That certainly seems right to me.

On 2007-09-06, at 16:27 EDT, Henry Minsky wrote:

> On a related note, in the replicator __adjustVisibleClones, it tries
> to find the layout
> of the parent, to lock the layout:
>
>     function __adjustVisibleClones () {
>         var p = this.parent;
>         for (var l in p.layouts) {
>             Debug.info("__adjustVisibleClones layout", p.layouts[l]);
>             p.layouts[l].lock();
>         }
>
>
> Shouldn't this be the immediateparent (which is bound to
> this.container in the construct method)?
>
>     function __adjustVisibleClones () {
>    ==>     var p = this.container;
>         for (var l in p.layouts) {
>             Debug.info("__adjustVisibleClones layout", p.layouts[l]);
>             p.layouts[l].lock();
>         }