You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@shindig.apache.org by li...@apache.org on 2008/02/21 03:16:51 UTC

svn commit: r629675 - in /incubator/shindig/trunk: features/ features/ifpc/ javascript/ javascript/container/

Author: lindner
Date: Wed Feb 20 18:16:49 2008
New Revision: 629675

URL: http://svn.apache.org/viewvc?rev=629675&view=rev
Log:
move everything over to gadgets.rpc instead of ifpc

Removed:
    incubator/shindig/trunk/features/ifpc/
    incubator/shindig/trunk/javascript/container/ifpc_relay.html
Modified:
    incubator/shindig/trunk/features/features.txt
    incubator/shindig/trunk/javascript/README
    incubator/shindig/trunk/javascript/container/sample2.html
    incubator/shindig/trunk/javascript/container/sample4.html
    incubator/shindig/trunk/javascript/container/sample5.html
    incubator/shindig/trunk/javascript/container/sample6.html
    incubator/shindig/trunk/javascript/container/sample7.html

Modified: incubator/shindig/trunk/features/features.txt
URL: http://svn.apache.org/viewvc/incubator/shindig/trunk/features/features.txt?rev=629675&r1=629674&r2=629675&view=diff
==============================================================================
--- incubator/shindig/trunk/features/features.txt (original)
+++ incubator/shindig/trunk/features/features.txt Wed Feb 20 18:16:49 2008
@@ -4,7 +4,6 @@
 features/core/feature.xml
 features/dynamic-height/feature.xml
 features/flash/feature.xml
-features/ifpc/feature.xml
 features/minimessage/feature.xml
 features/opensocial-reference/feature.xml
 features/opensocial-samplecontainer/feature.xml

Modified: incubator/shindig/trunk/javascript/README
URL: http://svn.apache.org/viewvc/incubator/shindig/trunk/javascript/README?rev=629675&r1=629674&r2=629675&view=diff
==============================================================================
--- incubator/shindig/trunk/javascript/README (original)
+++ incubator/shindig/trunk/javascript/README Wed Feb 20 18:16:49 2008
@@ -18,7 +18,7 @@
 
    A) Create an HTML file including the following <head> boilerplate:
       <script type="text/javascript" src="json.js"></script>
-      <script type="text/javascript" src="ifpc.js"></script>
+      <script type="text/javascript" src="../../js/rpc.js?c=1"></script>
       <script type="text/javascript" src="cookies.js"></script>
       <script type="text/javascript" src="gadgets.js"></script>
 

Modified: incubator/shindig/trunk/javascript/container/sample2.html
URL: http://svn.apache.org/viewvc/incubator/shindig/trunk/javascript/container/sample2.html?rev=629675&r1=629674&r2=629675&view=diff
==============================================================================
--- incubator/shindig/trunk/javascript/container/sample2.html (original)
+++ incubator/shindig/trunk/javascript/container/sample2.html Wed Feb 20 18:16:49 2008
@@ -16,7 +16,6 @@
   'http://www.labpixies.com/campaigns/todo/todo.xml'
 ];
 
-var containerParentUrl = 'ifpc_relay.html';
 
 // This container lays out and renders gadgets itself.
 
@@ -32,7 +31,6 @@
 };
 
 my.init = function() {
-  gadgets.container.setParentUrl(containerParentUrl);
   gadgets.container.layoutManager = new my.LayoutManager();
 };
 

Modified: incubator/shindig/trunk/javascript/container/sample4.html
URL: http://svn.apache.org/viewvc/incubator/shindig/trunk/javascript/container/sample4.html?rev=629675&r1=629674&r2=629675&view=diff
==============================================================================
--- incubator/shindig/trunk/javascript/container/sample4.html (original)
+++ incubator/shindig/trunk/javascript/container/sample4.html Wed Feb 20 18:16:49 2008
@@ -14,7 +14,6 @@
   gadgets.container.layoutManager =
       new gadgets.FloatLeftLayoutManager('gadget-parent');
 
-  gadgets.container.setParentUrl('ifpc_relay.html');
   gadgets.container.addGadget(
       gadgets.container.createGadget({specUrl: specUrl0}));
 };

Modified: incubator/shindig/trunk/javascript/container/sample5.html
URL: http://svn.apache.org/viewvc/incubator/shindig/trunk/javascript/container/sample5.html?rev=629675&r1=629674&r2=629675&view=diff
==============================================================================
--- incubator/shindig/trunk/javascript/container/sample5.html (original)
+++ incubator/shindig/trunk/javascript/container/sample5.html Wed Feb 20 18:16:49 2008
@@ -14,7 +14,6 @@
   gadgets.container.layoutManager =
       new gadgets.FloatLeftLayoutManager('gadget-parent');
 
-  gadgets.container.setParentUrl('ifpc_relay.html');
   var gadget = gadgets.container.createGadget({specUrl: specUrl0});
   gadgets.container.addGadget(gadget);
 };

Modified: incubator/shindig/trunk/javascript/container/sample6.html
URL: http://svn.apache.org/viewvc/incubator/shindig/trunk/javascript/container/sample6.html?rev=629675&r1=629674&r2=629675&view=diff
==============================================================================
--- incubator/shindig/trunk/javascript/container/sample6.html (original)
+++ incubator/shindig/trunk/javascript/container/sample6.html Wed Feb 20 18:16:49 2008
@@ -14,7 +14,6 @@
   gadgets.container.layoutManager =
       new gadgets.FloatLeftLayoutManager('gadget-parent');
 
-  gadgets.container.setParentUrl('ifpc_relay.html');
   var gadget = gadgets.container.createGadget({specUrl: specUrl0});
   gadgets.container.addGadget(gadget);
 };

Modified: incubator/shindig/trunk/javascript/container/sample7.html
URL: http://svn.apache.org/viewvc/incubator/shindig/trunk/javascript/container/sample7.html?rev=629675&r1=629674&r2=629675&view=diff
==============================================================================
--- incubator/shindig/trunk/javascript/container/sample7.html (original)
+++ incubator/shindig/trunk/javascript/container/sample7.html Wed Feb 20 18:16:49 2008
@@ -14,7 +14,6 @@
   gadgets.container.layoutManager =
       new gadgets.FloatLeftLayoutManager('gadget-parent');
 
-  gadgets.container.setParentUrl('ifpc_relay.html');
   var gadget = gadgets.container.createGadget({specUrl: specUrl0});
   gadgets.container.addGadget(gadget);
 };



Re: svn commit: r629675 - in /incubator/shindig/trunk: features/ features/ifpc/ javascript/ javascript/container/

Posted by Cassie <do...@apache.org>.
On Wed, Feb 20, 2008 at 8:52 PM, Kevin Brown <et...@google.com> wrote:

> On Wed, Feb 20, 2008 at 6:37 PM, Paul Lindner <pl...@hi5.com> wrote:
>
> > I don't see ifpc module support listed at
> > http://code.google.com/apis/gadgets/docs/reference/
> >
> > I do see rpc listed there.  Is this for type URL gadgets?  I don't
> > know of any spec that required a <Require feature="ifpc"/> anywhere...
>
>
> rpc is an attempt to address the shortcomings of IFPC; ultimately we want
> to
> phase out IFPC entirely, but if we change all of shindig from using ifpc
> to
> using rpc today, we will wind up breaking anyone that was in the process
> of
> migrating from using gmodules.com to shindig. ifpc *will* be removed, but
> it
> needs to stick around for a bit longer.
>
> gadgets.ifpc was actually implemented before the gadgets.rpc spec was
> created, in an attempt to emulate what current gmodules.com is doing.
> While
> gadgets.rpc is definitely the "right" architectural model to go with at
> this
> point in time (and Zhen's work is excellent), we do have to worry about
> compatibility. In short, it's the same reason why we have the legacy.jsfile
> -- it's a big hack to make adopting opensocial (and in particular,
> shindig's
> implementation) easier.
>
> In any case what's it going to be?  Lots of feature requests have been
> > shot down because they were not listed in the spec.
>
>
> We have to look at it on a case by case basis. If something is very
> popular
> today (a significant percentage of the existing gadgets use it), we have
> to
> provide these sorts of unfortunate hacks to deal with it (see: ifpc,
> legacy.js, various legacy mappings, the entire "gadgets.Prefs" library).
> At
> the same time, we don't want to get distracted with trying to support
> every
> minor feature that gmodules.com supports today just to maintain perfect
> parity (especially not when the feature inherently depends on some
> google-proprietary service).  We really don't want to encourage even more
> gadgets to be developed using these non-standard libraries if we can avoid
> it.
>
>  If this is
> > important for gmodules.com, but is not part of the gadgets spec then
> > perhaps it's something that can be maintained separately.
>
>
> It isn't important for gmodules.com itself (igoogle continues to use ifpc
> today, and when igoogle starts using shindig they will be using
> gadgets.rpcbecause it's a lot more efficient) -- it's important for
> containers that
> render on gmodules.com today and are relying on ifpc working that are
> trying
> to migrate to using shindig as their rendering mechanism. Before google
> decided to turn the gadgets platform into an open standard, many sites
> were
> using gmodules.com to render gadgets. Now that we've got an open standard,
> and shindig is around, many organizations have expressed a desire to move
> off of gmodules and onto their own gadget servers using shindig, but in
> order to ease this transition, we need to make sure that some existing
> things continue to work.
>

So this is a very interesting comment you have. How many containers out
there are really trying to do this? I*f any container on this list is trying
to migrate from gmodules to shindig please let us know.*

Alternatively, perhaps we can ask some other lists, or put a question on the
shindig website asking for container users to announce themselves.

I sorta agree with Paul here in that we should be moving faster and not
waiting around for these legacy things. If google needs to maintain its own
stuff separately, that's fine, but shindig shouldn't be hindered.

I also have the same sort of feeling with the php stuff. The only reason we
have out of date js files in javascript/* is because we were waiting for the
php stuff to catch up. I know Chris is working on completely new stuff and
besides that, the code is abandoned. I would really like to see us keep
refactoring on the js/java side without having the completely out of date
code hold us back. People are starting to get confused about how io.js is
different than io.js :)

Just my two cents..

- Cassie


>
> That being said, if we can get everyone who is in this situation to agree
> to
> migrate to using rpc instead of ifpc, we can probably remove the feature
> sooner rather than later.
>
> Personally, I want to get rid of IFPC right now -- it's slow, bloated, and
> really hacky, but if we wind up with most of these migrated sites needing
> to
> maintain their own local copy of IFPC anyway, it just makes adopting
> shindig
> harder.
>

Re: svn commit: r629675 - in /incubator/shindig/trunk: features/ features/ifpc/ javascript/ javascript/container/

Posted by Kevin Brown <et...@google.com>.
On Wed, Feb 20, 2008 at 6:37 PM, Paul Lindner <pl...@hi5.com> wrote:

> I don't see ifpc module support listed at
> http://code.google.com/apis/gadgets/docs/reference/
>
> I do see rpc listed there.  Is this for type URL gadgets?  I don't
> know of any spec that required a <Require feature="ifpc"/> anywhere...


rpc is an attempt to address the shortcomings of IFPC; ultimately we want to
phase out IFPC entirely, but if we change all of shindig from using ifpc to
using rpc today, we will wind up breaking anyone that was in the process of
migrating from using gmodules.com to shindig. ifpc *will* be removed, but it
needs to stick around for a bit longer.

gadgets.ifpc was actually implemented before the gadgets.rpc spec was
created, in an attempt to emulate what current gmodules.com is doing. While
gadgets.rpc is definitely the "right" architectural model to go with at this
point in time (and Zhen's work is excellent), we do have to worry about
compatibility. In short, it's the same reason why we have the legacy.js file
-- it's a big hack to make adopting opensocial (and in particular, shindig's
implementation) easier.

In any case what's it going to be?  Lots of feature requests have been
> shot down because they were not listed in the spec.


We have to look at it on a case by case basis. If something is very popular
today (a significant percentage of the existing gadgets use it), we have to
provide these sorts of unfortunate hacks to deal with it (see: ifpc,
legacy.js, various legacy mappings, the entire "gadgets.Prefs" library). At
the same time, we don't want to get distracted with trying to support every
minor feature that gmodules.com supports today just to maintain perfect
parity (especially not when the feature inherently depends on some
google-proprietary service).  We really don't want to encourage even more
gadgets to be developed using these non-standard libraries if we can avoid
it.

 If this is
> important for gmodules.com, but is not part of the gadgets spec then
> perhaps it's something that can be maintained separately.


It isn't important for gmodules.com itself (igoogle continues to use ifpc
today, and when igoogle starts using shindig they will be using
gadgets.rpcbecause it's a lot more efficient) -- it's important for
containers that
render on gmodules.com today and are relying on ifpc working that are trying
to migrate to using shindig as their rendering mechanism. Before google
decided to turn the gadgets platform into an open standard, many sites were
using gmodules.com to render gadgets. Now that we've got an open standard,
and shindig is around, many organizations have expressed a desire to move
off of gmodules and onto their own gadget servers using shindig, but in
order to ease this transition, we need to make sure that some existing
things continue to work.

That being said, if we can get everyone who is in this situation to agree to
migrate to using rpc instead of ifpc, we can probably remove the feature
sooner rather than later.

Personally, I want to get rid of IFPC right now -- it's slow, bloated, and
really hacky, but if we wind up with most of these migrated sites needing to
maintain their own local copy of IFPC anyway, it just makes adopting shindig
harder.

Re: svn commit: r629675 - in /incubator/shindig/trunk: features/ features/ifpc/ javascript/ javascript/container/

Posted by Paul Lindner <pl...@hi5.com>.
I don't see ifpc module support listed at 
http://code.google.com/apis/gadgets/docs/reference/

I do see rpc listed there.  Is this for type URL gadgets?  I don't
know of any spec that required a <Require feature="ifpc"/> anywhere...

In any case what's it going to be?  Lots of feature requests have been
shot down because they were not listed in the spec.  If this is
important for gmodules.com, but is not part of the gadgets spec then
perhaps it's something that can be maintained separately.

We probably need to chew through this and other issues on the phone or
in a chat room.  Perhaps I just don't understand the use cases...

Are you free tomorrow?  I'll adjust my schedule to yours.


On Wed, Feb 20, 2008 at 06:22:45PM -0800, Kevin Brown wrote:
> You can't remove the ifpc files yet -- some existing code still depends on
> it (notably most existing containers that use shindig), which is why we have
> the useLegacyProtocol flag. The changes to the sample pages are fine,
> though. Please revert the other part of the change. This workaround is
> critical for making the transition of existing sites that used google
> infrastructure (gmodules.com) to using shindig less painful. Without it, we
> wind up requiring all the existing sites to upgrade to rpc immediately,
> which isn't viable at the moment.
> 
> On Wed, Feb 20, 2008 at 6:16 PM, <li...@apache.org> wrote:
> 
> > Author: lindner
> > Date: Wed Feb 20 18:16:49 2008
> > New Revision: 629675
> >
> > URL: http://svn.apache.org/viewvc?rev=629675&view=rev
> > Log:
> > move everything over to gadgets.rpc instead of ifpc
> >
> > Removed:
> >    incubator/shindig/trunk/features/ifpc/
> >    incubator/shindig/trunk/javascript/container/ifpc_relay.html
> > Modified:
> >    incubator/shindig/trunk/features/features.txt
> >    incubator/shindig/trunk/javascript/README
> >    incubator/shindig/trunk/javascript/container/sample2.html
> >    incubator/shindig/trunk/javascript/container/sample4.html
> >    incubator/shindig/trunk/javascript/container/sample5.html
> >    incubator/shindig/trunk/javascript/container/sample6.html
> >    incubator/shindig/trunk/javascript/container/sample7.html
> >
> > Modified: incubator/shindig/trunk/features/features.txt
> > URL:
> > http://svn.apache.org/viewvc/incubator/shindig/trunk/features/features.txt?rev=629675&r1=629674&r2=629675&view=diff
> >
> > ==============================================================================
> > +++ incubator/shindig/trunk/features/features.txt Wed Feb 20 18:16:49 2008
> > @@ -4,7 +4,6 @@
> >  features/core/feature.xml
> >  features/dynamic-height/feature.xml
> >  features/flash/feature.xml
> > -features/ifpc/feature.xml
> >  features/minimessage/feature.xml
> >  features/opensocial-reference/feature.xml
> >  features/opensocial-samplecontainer/feature.xml
> >
> > Modified: incubator/shindig/trunk/javascript/README
> > URL:
> > http://svn.apache.org/viewvc/incubator/shindig/trunk/javascript/README?rev=629675&r1=629674&r2=629675&view=diff
> >
> > ==============================================================================
> > +++ incubator/shindig/trunk/javascript/README Wed Feb 20 18:16:49 2008
> > @@ -18,7 +18,7 @@
> >
> >    A) Create an HTML file including the following <head> boilerplate:
> >       <script type="text/javascript" src="json.js"></script>
> > -      <script type="text/javascript" src="ifpc.js"></script>
> > +      <script type="text/javascript" src="../../js/rpc.js?c=1"></script>
> >       <script type="text/javascript" src="cookies.js"></script>
> >       <script type="text/javascript" src="gadgets.js"></script>
> >
> >
> > Modified: incubator/shindig/trunk/javascript/container/sample2.html
> > URL:
> > http://svn.apache.org/viewvc/incubator/shindig/trunk/javascript/container/sample2.html?rev=629675&r1=629674&r2=629675&view=diff
> >
> > ==============================================================================
> > +++ incubator/shindig/trunk/javascript/container/sample2.html Wed Feb 20
> > 18:16:49 2008
> > @@ -16,7 +16,6 @@
> >   'http://www.labpixies.com/campaigns/todo/todo.xml'
> >  ];
> >
> > -var containerParentUrl = 'ifpc_relay.html';
> >
> >  // This container lays out and renders gadgets itself.
> >
> > @@ -32,7 +31,6 @@
> >  };
> >
> >  my.init = function() {
> > -  gadgets.container.setParentUrl(containerParentUrl);
> >   gadgets.container.layoutManager = new my.LayoutManager();
> >  };
> >
> >
> > Modified: incubator/shindig/trunk/javascript/container/sample4.html
> > URL:
> > http://svn.apache.org/viewvc/incubator/shindig/trunk/javascript/container/sample4.html?rev=629675&r1=629674&r2=629675&view=diff
> >
> > ==============================================================================
> > +++ incubator/shindig/trunk/javascript/container/sample4.html Wed Feb 20
> > 18:16:49 2008
> > @@ -14,7 +14,6 @@
> >   gadgets.container.layoutManager =
> >       new gadgets.FloatLeftLayoutManager('gadget-parent');
> >
> > -  gadgets.container.setParentUrl('ifpc_relay.html');
> >   gadgets.container.addGadget(
> >       gadgets.container.createGadget({specUrl: specUrl0}));
> >  };
> >
> > Modified: incubator/shindig/trunk/javascript/container/sample5.html
> > URL:
> > http://svn.apache.org/viewvc/incubator/shindig/trunk/javascript/container/sample5.html?rev=629675&r1=629674&r2=629675&view=diff
> >
> > ==============================================================================
> > +++ incubator/shindig/trunk/javascript/container/sample5.html Wed Feb 20
> > 18:16:49 2008
> > @@ -14,7 +14,6 @@
> >   gadgets.container.layoutManager =
> >       new gadgets.FloatLeftLayoutManager('gadget-parent');
> >
> > -  gadgets.container.setParentUrl('ifpc_relay.html');
> >   var gadget = gadgets.container.createGadget({specUrl: specUrl0});
> >   gadgets.container.addGadget(gadget);
> >  };
> >
> > Modified: incubator/shindig/trunk/javascript/container/sample6.html
> > URL:
> > http://svn.apache.org/viewvc/incubator/shindig/trunk/javascript/container/sample6.html?rev=629675&r1=629674&r2=629675&view=diff
> >
> > ==============================================================================
> > +++ incubator/shindig/trunk/javascript/container/sample6.html Wed Feb 20
> > 18:16:49 2008
> > @@ -14,7 +14,6 @@
> >   gadgets.container.layoutManager =
> >       new gadgets.FloatLeftLayoutManager('gadget-parent');
> >
> > -  gadgets.container.setParentUrl('ifpc_relay.html');
> >   var gadget = gadgets.container.createGadget({specUrl: specUrl0});
> >   gadgets.container.addGadget(gadget);
> >  };
> >
> > Modified: incubator/shindig/trunk/javascript/container/sample7.html
> > URL:
> > http://svn.apache.org/viewvc/incubator/shindig/trunk/javascript/container/sample7.html?rev=629675&r1=629674&r2=629675&view=diff
> >
> > ==============================================================================
> > +++ incubator/shindig/trunk/javascript/container/sample7.html Wed Feb 20
> > 18:16:49 2008
> > @@ -14,7 +14,6 @@
> >   gadgets.container.layoutManager =
> >       new gadgets.FloatLeftLayoutManager('gadget-parent');
> >
> > -  gadgets.container.setParentUrl('ifpc_relay.html');
> >   var gadget = gadgets.container.createGadget({specUrl: specUrl0});
> >   gadgets.container.addGadget(gadget);
> >  };
> >
> >
> >
> 
> 

-- 
Paul Lindner
hi5 Architect
plindner@hi5.com

Re: svn commit: r629675 - in /incubator/shindig/trunk: features/ features/ifpc/ javascript/ javascript/container/

Posted by Kevin Brown <et...@google.com>.
You can't remove the ifpc files yet -- some existing code still depends on
it (notably most existing containers that use shindig), which is why we have
the useLegacyProtocol flag. The changes to the sample pages are fine,
though. Please revert the other part of the change. This workaround is
critical for making the transition of existing sites that used google
infrastructure (gmodules.com) to using shindig less painful. Without it, we
wind up requiring all the existing sites to upgrade to rpc immediately,
which isn't viable at the moment.

On Wed, Feb 20, 2008 at 6:16 PM, <li...@apache.org> wrote:

> Author: lindner
> Date: Wed Feb 20 18:16:49 2008
> New Revision: 629675
>
> URL: http://svn.apache.org/viewvc?rev=629675&view=rev
> Log:
> move everything over to gadgets.rpc instead of ifpc
>
> Removed:
>    incubator/shindig/trunk/features/ifpc/
>    incubator/shindig/trunk/javascript/container/ifpc_relay.html
> Modified:
>    incubator/shindig/trunk/features/features.txt
>    incubator/shindig/trunk/javascript/README
>    incubator/shindig/trunk/javascript/container/sample2.html
>    incubator/shindig/trunk/javascript/container/sample4.html
>    incubator/shindig/trunk/javascript/container/sample5.html
>    incubator/shindig/trunk/javascript/container/sample6.html
>    incubator/shindig/trunk/javascript/container/sample7.html
>
> Modified: incubator/shindig/trunk/features/features.txt
> URL:
> http://svn.apache.org/viewvc/incubator/shindig/trunk/features/features.txt?rev=629675&r1=629674&r2=629675&view=diff
>
> ==============================================================================
> --- incubator/shindig/trunk/features/features.txt (original)
> +++ incubator/shindig/trunk/features/features.txt Wed Feb 20 18:16:49 2008
> @@ -4,7 +4,6 @@
>  features/core/feature.xml
>  features/dynamic-height/feature.xml
>  features/flash/feature.xml
> -features/ifpc/feature.xml
>  features/minimessage/feature.xml
>  features/opensocial-reference/feature.xml
>  features/opensocial-samplecontainer/feature.xml
>
> Modified: incubator/shindig/trunk/javascript/README
> URL:
> http://svn.apache.org/viewvc/incubator/shindig/trunk/javascript/README?rev=629675&r1=629674&r2=629675&view=diff
>
> ==============================================================================
> --- incubator/shindig/trunk/javascript/README (original)
> +++ incubator/shindig/trunk/javascript/README Wed Feb 20 18:16:49 2008
> @@ -18,7 +18,7 @@
>
>    A) Create an HTML file including the following <head> boilerplate:
>       <script type="text/javascript" src="json.js"></script>
> -      <script type="text/javascript" src="ifpc.js"></script>
> +      <script type="text/javascript" src="../../js/rpc.js?c=1"></script>
>       <script type="text/javascript" src="cookies.js"></script>
>       <script type="text/javascript" src="gadgets.js"></script>
>
>
> Modified: incubator/shindig/trunk/javascript/container/sample2.html
> URL:
> http://svn.apache.org/viewvc/incubator/shindig/trunk/javascript/container/sample2.html?rev=629675&r1=629674&r2=629675&view=diff
>
> ==============================================================================
> --- incubator/shindig/trunk/javascript/container/sample2.html (original)
> +++ incubator/shindig/trunk/javascript/container/sample2.html Wed Feb 20
> 18:16:49 2008
> @@ -16,7 +16,6 @@
>   'http://www.labpixies.com/campaigns/todo/todo.xml'
>  ];
>
> -var containerParentUrl = 'ifpc_relay.html';
>
>  // This container lays out and renders gadgets itself.
>
> @@ -32,7 +31,6 @@
>  };
>
>  my.init = function() {
> -  gadgets.container.setParentUrl(containerParentUrl);
>   gadgets.container.layoutManager = new my.LayoutManager();
>  };
>
>
> Modified: incubator/shindig/trunk/javascript/container/sample4.html
> URL:
> http://svn.apache.org/viewvc/incubator/shindig/trunk/javascript/container/sample4.html?rev=629675&r1=629674&r2=629675&view=diff
>
> ==============================================================================
> --- incubator/shindig/trunk/javascript/container/sample4.html (original)
> +++ incubator/shindig/trunk/javascript/container/sample4.html Wed Feb 20
> 18:16:49 2008
> @@ -14,7 +14,6 @@
>   gadgets.container.layoutManager =
>       new gadgets.FloatLeftLayoutManager('gadget-parent');
>
> -  gadgets.container.setParentUrl('ifpc_relay.html');
>   gadgets.container.addGadget(
>       gadgets.container.createGadget({specUrl: specUrl0}));
>  };
>
> Modified: incubator/shindig/trunk/javascript/container/sample5.html
> URL:
> http://svn.apache.org/viewvc/incubator/shindig/trunk/javascript/container/sample5.html?rev=629675&r1=629674&r2=629675&view=diff
>
> ==============================================================================
> --- incubator/shindig/trunk/javascript/container/sample5.html (original)
> +++ incubator/shindig/trunk/javascript/container/sample5.html Wed Feb 20
> 18:16:49 2008
> @@ -14,7 +14,6 @@
>   gadgets.container.layoutManager =
>       new gadgets.FloatLeftLayoutManager('gadget-parent');
>
> -  gadgets.container.setParentUrl('ifpc_relay.html');
>   var gadget = gadgets.container.createGadget({specUrl: specUrl0});
>   gadgets.container.addGadget(gadget);
>  };
>
> Modified: incubator/shindig/trunk/javascript/container/sample6.html
> URL:
> http://svn.apache.org/viewvc/incubator/shindig/trunk/javascript/container/sample6.html?rev=629675&r1=629674&r2=629675&view=diff
>
> ==============================================================================
> --- incubator/shindig/trunk/javascript/container/sample6.html (original)
> +++ incubator/shindig/trunk/javascript/container/sample6.html Wed Feb 20
> 18:16:49 2008
> @@ -14,7 +14,6 @@
>   gadgets.container.layoutManager =
>       new gadgets.FloatLeftLayoutManager('gadget-parent');
>
> -  gadgets.container.setParentUrl('ifpc_relay.html');
>   var gadget = gadgets.container.createGadget({specUrl: specUrl0});
>   gadgets.container.addGadget(gadget);
>  };
>
> Modified: incubator/shindig/trunk/javascript/container/sample7.html
> URL:
> http://svn.apache.org/viewvc/incubator/shindig/trunk/javascript/container/sample7.html?rev=629675&r1=629674&r2=629675&view=diff
>
> ==============================================================================
> --- incubator/shindig/trunk/javascript/container/sample7.html (original)
> +++ incubator/shindig/trunk/javascript/container/sample7.html Wed Feb 20
> 18:16:49 2008
> @@ -14,7 +14,6 @@
>   gadgets.container.layoutManager =
>       new gadgets.FloatLeftLayoutManager('gadget-parent');
>
> -  gadgets.container.setParentUrl('ifpc_relay.html');
>   var gadget = gadgets.container.createGadget({specUrl: specUrl0});
>   gadgets.container.addGadget(gadget);
>  };
>
>
>


-- 
~Kevin

If you received this email by mistake, please delete it, cancel your mail
account, destroy your hard drive, silence any witnesses, and burn down the
building that you're in.