commit with a proper log message. Sorry about this.
> Sorry I should have snipped to the relevant portion, it's done below.
>
> Regards
> Scott
>
> On 7/04/2010, at 11:29 PM, Scott Gray wrote:
>
> > Hi Bruno,
> >
> > I know it was a while ago but in this commit you removed the geoLocation
> screen and made no mention of it in the commit log, was it intentional? The
> screen is still referenced from the common-controller.
> >
> > Thanks
> > Scott
> >
> > HotWax Media
> >
http://www.hotwaxmedia.com> >
> > On 29/12/2009, at 2:22 AM,
[hidden email] wrote:
> >
> >> Author: buscob
> >> Date: Tue Dec 29 09:22:45 2009
> >> New Revision: 894330
> >>
> >> URL:
http://svn.apache.org/viewvc?rev=894330&view=rev> >> Log:
> >> [OFBIZ-3274] - Using decorator sections to control the left-bar
> >> The leftbar content is now defined using the "left-column"
> ApplicationDecorator section instead of setting the variables
> "leftbarScreenName", "leftbarScreenLocation" and "MainColumnStyle".
> >> The logic that checks if the "left-column" section has content, and thus
> if a left column must be rendered, has been moved from the GlobalDecorator
> to the ApplicationDecorator.
> >>
> >> Modified:
> [snip]
> >> ofbiz/trunk/framework/common/widget/CommonScreens.xml
> >> ofbiz/trunk/framework/example/widget/example/CommonScreens.xml
> >> Modified: ofbiz/trunk/framework/common/widget/CommonScreens.xml
> >> URL:
>
http://svn.apache.org/viewvc/ofbiz/trunk/framework/common/widget/CommonScreens.xml?rev=894330&r1=894329&r2=894330&view=diff> >>
> ==============================================================================
> >> --- ofbiz/trunk/framework/common/widget/CommonScreens.xml (original)
> >> +++ ofbiz/trunk/framework/common/widget/CommonScreens.xml Tue Dec 29
> 09:22:45 2009
> >> @@ -198,20 +198,11 @@
> >>
> >> <!-- render messages -->
> >> <platform-specific><html><html-template
> location="${messagesTemplateLocation}"/></html></platform-specific>
> >> +
> >> + <!-- render content area -->
> >> <container style="contentarea">
> >> - <!-- may be used by sight disabled persons to skip
> directly to content area-->
> >> <decorator-section-include name="pre-body"/>
> >> - <!-- Please don't change the column-container id :
> it's referenced to allow skipping navigation-->
> >> - <container id="column-container">
> >> - <!-- by default will render left-bar only if
> leftbarScreen value not empty -->
> >> - <include-screen name="${leftbarScreenName}"
> location="${leftbarScreenLocation}"/>
> >> -
> >> - <container id="content-main-section"
> style="${MainColumnStyle}">
> >> - <!-- render messages -->
> >> - <decorator-section-include name="body"/>
> >> - </container>
> >> - <container style="clear"></container>
> >> - </container>
> >> + <decorator-section-include name="body"/>
> >> </container>
> >>
> >> <!-- render footer -->
> >> @@ -413,7 +404,7 @@
> >> <screen name="forgotPassword">
> >> <section>
> >> <widgets>
> >> - <decorator-screen name="main-decorator"
> location="${parameters.mainDecoratorLocation}">
> >> + <decorator-screen name="login-decorator"
> location="${parameters.mainDecoratorLocation}">
> >> <decorator-section name="body">
> >> <platform-specific>
> >> <html><html-template
> location="component://common/webcommon/forgotPassword.ftl"/></html>
> >> @@ -498,27 +489,6 @@
> >> </section>
> >> </screen>
> >>
> >> - <screen name="geoLocation">
> >> - <section>
> >> - <actions>
> >> - <set field="titleProperty"
> value="PageTitleCommonGeoLocation"/>
> >> - <entity-one entity-name="GeoPoint"
> value-field="geoPoint">
> >> - <field-map field-name="geoPointId"
> from-field="parameters.geoPointId"/>
> >> - </entity-one>
> >> - <script
> location="component://common/webcommon/WEB-INF/actions/includes/GeoLocation.groovy"/>
> >> - </actions>
> >> - <widgets>
> >> - <decorator-screen name="SimpleDecorator">
> >> - <decorator-section name="body">
> >> - <platform-specific>
> >> - <html><html-template
> location="component://common/webcommon/WEB-INF/geolocation.ftl"/></html>
> >> - </platform-specific>
> >> - </decorator-section>
> >> - </decorator-screen>
> >> - </widgets>
> >> - </section>
> >> - </screen>
> >> -
> >> <screen name="geoChart">
> >> <section>
> >> <actions>
>
Locked
