Hi Gonzalo and David, I've just committed something to separate YUI instantiation and event handlers from the portal.js. Event handlers for toolbox portlet are now defined in toolbox.jsp. Also, the ids are not duplicated any more by using portlet namespace and portlet unique names. By the way, I see the following problems now: - The portlets list table contains table element for each portlet item. I think it should have tr and td before adding the table element. FYI, we cannot leverage data table with YUI3 yet because it's not ported yet. - When search button is clicked, it should remove existing rows. - Preview link - ...
Cheers, Woonsan --- On Fri, 11/20/09, Woonsan Ko <[email protected]> wrote: > From: Woonsan Ko <[email protected]> > Subject: Re: Progress... > To: "Jetspeed Developers List" <[email protected]> > Date: Friday, November 20, 2009, 1:44 PM > Hi Gonzalo, > > --- On Fri, 11/20/09, Gonzalo Aguilar Delgado <[email protected]> > wrote: > > > From: Gonzalo Aguilar Delgado <[email protected]> > > Subject: Re: Progress... > > To: "Jetspeed Developers List" <[email protected]> > > Date: Friday, November 20, 2009, 9:46 AM > > > > > > El jue, 19-11-2009 a las 19:46 -0800, David Sean > Taylor > > escribió: > > > > > > On Nov 19, 2009, at 4:06 PM, Gonzalo > Aguilar > > Delgado wrote: > > > > > > > >> I've done several changes to the code. > I'm > > near form the whole > > > >> change. > > > >> > > > >> But, I have a problem with the panel. I > don't > > know what's the best > > > >> way to put > > > >> portlet information on web. > > > >> > > > >> It's supposed that I have to use the > table > > already created but I > > > >> don't know how > > > >> te set information via YUI3: > > > >> > > > >> +<table id="portlet-body-html" > > style="border-collapse: collapse; > > > >> width: 100%; margin-top: 0px; > margin-bottom: > > 0px; float: left;"> > > > >> + <tr> > > > >> + <td > > rowspan="2" width="20%" > class="portlet-section-body"> > > > >> + > > <img src='${portalContextPath}' /> > > > >> + </td> > > > >> + <td > > id="name" width="80%" > > class="portlet-section-body">#</td> > > > > > > We should be careful about node ids, id='name' is > not > > the best choice, > > > could use some namespace or prefixing > > > Also we will be getting a whole set of nodes with > the > > same id, not > > > sure if that was the desired effect or not... > > > > > > I know, I cannot handle this correctly because not > enough > > knowledge of > > YUI. That's why I > > requested help. I suppose that we should pass context > in a > > variable and > > change ids via YUI. > > > > Any other way to do it? > > I think we need to have separate YUI global object instance > in toolbox.jsp without mixing the custom event handlers for > toolbox into the global portal.js. > It says, "a page can share a single YUI instance or can use > different, insular instances for each piece of functionality > on the page." [1] > In a portlet, the second approach seems better. > Anyway, if we go this way, then it would be easier to pass > context namespace or attaching custom events. > I'll try to find a solution around this. > > [1] http://developer.yahoo.com/yui/3/yui/ > > > Regards, > > Woonsan > > > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [email protected] > For additional commands, e-mail: [email protected] > > --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
