portals-jetspeed-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From David Sean Taylor <d.tay...@onehippo.com>
Subject Re: Progress...
Date Fri, 20 Nov 2009 03:23:22 GMT
I committed this to trunk (after modifying some things) so you can  
continue to work on it. From the commit message:

Reviewed. I've modified and fixed a few things...
- fixed the display names to display
- rewrote the row population script to use YUI Node api replacing  
direct DOM manipulation
- dont' use a onclick handler, simply call the first load of portlets  
directly inline
  im actually not sure what should be populated first, perhaps a  
"Whats New" category
- added some TODOs for you, see source code for details

Im also thinking that it might be better to create the nodes directly  
from javascript, and not pass in the hidden table (I added a  
display:none attribute on it) although the hidden table is definitely  
easier. Optionally you may want to consider loading from a string,  
something like:

var clone = Y.Node.create("<table><tr>....etc....");

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>
> +    </tr>
> +    <tr>
> +       <td class="portlet-section-body">
> +           <a href="#">Preview</a>
> +           <a href="#">Add</a>
> +       </td>
> +    </tr>
> +</table>
> +
> I only managed to set the images. Normally a custom widget should be  
> created but I have
> not enough knowledge about YUI.
> The rest works well... The only change needed is to do the filter to  
> be able to load on category click
> but this will be trivial.
> If you help me to put information in each field I will finish this.
> The rest of the changes:
> Prepatch-j2-admin-JS2-1057:
> * Removed server retrieve of portlets (famous two lines)
> * Modified a bit the jsp code.
> Prepatch-jetspeed-2-JS2-1057:
> * portal.js - added ajax request and load using JSON
> * modified getportlets to handle json output.
> * GetPortletsAction - added support for JSON
> * Fixed a bug that caused xml and json fail when bad escaped texts  
> found in description or name of the portlet.
> It's not much but it took me a lot of time figure out how everything  
> glue, and setting dev environment caused some trouble <face-smile- 
> big.png>
> Your ideas will be welcome...
> Tnx
> <Prepatch-j2-admin-JS2-1057.txt><Prepatch-jetspeed-2-JS2-1057.txt>

To unsubscribe, e-mail: jetspeed-dev-unsubscribe@portals.apache.org
For additional commands, e-mail: jetspeed-dev-help@portals.apache.org

View raw message