[ 
http://issues.apache.org/jira/browse/JS2-321?page=comments#action_12317512 ] 

Scott T Weaver commented on JS2-321:
------------------------------------

Layout api completely refactored from the ground up:

-Created a ColumnLayout model object that handles the layout and movement and 
orginaztion.  This model object makes writing a testcase much easier than 
testing the logic by hand in the running portal.

-  Created an extensive test case for the ColumnLayout

- Created a formal exception hierarchy for the layout api instead of checking 
for nulls, the layout has become a "good citizen" notfiying us what exactly 
goes wrong using exceptions.

- Created an event system for the ColumnLayout.  This makes it much easier to 
accomplish things when the LayoutChanges, for instance, upfating the Page 
object with the new positions of fragments.

- Created a PageManagerLayoutEventListener that gets fired when fragments are 
moved around to save the new fragment positions.

> Customizer not correctly positoning portlets
> --------------------------------------------
>
>          Key: JS2-321
>          URL: http://issues.apache.org/jira/browse/JS2-321
>      Project: Jetspeed 2
>         Type: Bug
>   Components: Customizer, Layout
>     Versions: 2.0-M1, 2.0-M2, 2.0-M3, 2.0-M4
>     Reporter: Scott T Weaver
>     Assignee: Scott T Weaver
>      Fix For: 2.0-M4, 2.0-FINAL

>
> It appears that the positioning in the customizer is still not correctly 
> updating postions for moved fragments after returning for edit mode in a 
> layout.  Also, all the positioning code lies withing the MultiColumnPortlet 
> itself and would be better served if it were moved out into a seperate, 
> easily teastable model object.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to