You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ofbiz.apache.org by Jacques Le Roux <ja...@les7arts.com> on 2010/08/03 14:52:25 UTC

Re: svn commit: r981846 - /ofbiz/branches/jquery/framework/widget/templates/htmlFormMacroLibrary.ftl

OK I will fix my comment Adam asked for and will be able to ask for a better one here :o)
Which bug is supposed to be fixed please?

Jacques

> Author: erwan
> Date: Tue Aug  3 12:41:04 2010
> New Revision: 981846
>
> URL: http://svn.apache.org/viewvc?rev=981846&view=rev
> Log:
> A patch from Sascha correcting a bug related to calendar implementation
>
> Modified:
>    ofbiz/branches/jquery/framework/widget/templates/htmlFormMacroLibrary.ftl
>
> Modified: ofbiz/branches/jquery/framework/widget/templates/htmlFormMacroLibrary.ftl
> URL: 
> http://svn.apache.org/viewvc/ofbiz/branches/jquery/framework/widget/templates/htmlFormMacroLibrary.ftl?rev=981846&r1=981845&r2=981846&view=diff
> ==============================================================================
> --- ofbiz/branches/jquery/framework/widget/templates/htmlFormMacroLibrary.ftl (original)
> +++ ofbiz/branches/jquery/framework/widget/templates/htmlFormMacroLibrary.ftl Tue Aug  3 12:41:04 2010
> @@ -30,7 +30,7 @@ under the License.
>         <#if inPlaceEditorUrl?has_content || class?has_content || alert=="true">
>             <span <#if idName?has_content>id="cc_${idName}"</#if> <@renderClass class alert />><#t/>
>         </#if>
> -
> +
>         <#if description?has_content>
>             ${description?replace("\n", "<br />")}<#t/>
>         <#else>
> @@ -54,7 +54,7 @@ under the License.
>     <#if value?has_content> value="${value}"</#if><#rt/>
>     <#if textSize?has_content> size="${textSize}"</#if><#rt/>
>     <#if maxlength?has_content> maxlength="${maxlength}"</#if><#rt/>
> -    <#if disabled?has_content && disabled> disabled="disabled"</#if><#rt/>
> +    <#if disabled?has_content && disabled> disabled="disabled"</#if><#rt/>
>     <#if id?has_content> id="${id}"</#if><#rt/>
>     <#if event?has_content && action?has_content> ${event}="${action}"</#if><#rt/>
>     <#if clientAutocomplete?has_content && clientAutocomplete=="false"> autocomplete="off"</#if><#rt/>
> @@ -376,8 +376,7 @@ ${item.description}</span>
>                 dateFormat: 'yy-mm-dd'
>               });
>       </script>
> -<#if formName?has_content>${formName}.</#if><#if name?has_content>${name}_fld0_value,</#if>'<#if 
> defaultDateTimeString?has_content>${defaultDateTimeString}</#if>');" title="${localizedIconTitle}"><#rt/>
> -</a><#rt/>
> +<#rt/>
> </#if>
> <#if titleStyle?has_content>
> <span class="${titleStyle}"><#rt/>
> @@ -412,8 +411,7 @@ ${item.description}</span>
>                 dateFormat: 'yy-mm-dd'
>               });
>       </script>
> -<#if formName?has_content>${formName}.</#if><#if name?has_content>${name}_fld1_value,'</#if><#if 
> defaultDateTimeString?has_content>${defaultDateTimeString}</#if>');" title="${localizedIconTitle}"><#rt/>
> -</a><#rt/>
> +<#rt/>
> </#if>
> <#if titleStyle?has_content>
>  <span class="${titleStyle}"><#rt/>
>
> 



Re: svn commit: r981846 - /ofbiz/branches/jquery/framework/widget/templates/htmlFormMacroLibrary.ftl

Posted by Jacques Le Roux <ja...@les7arts.com>.
Thanks Erwan

Jacques

From: "Erwan de FERRIERES" <er...@nereide.fr>
> Le 03/08/2010 14:52, Jacques Le Roux a écrit :
>> OK I will fix my comment Adam asked for and will be able to ask for a
>> better one here :o)
>> Which bug is supposed to be fixed please?
>>
> Hi Jacques,
> this is on the jquery branch, there was a display problem when date fields where on the screen (some code was displayed in the 
> user interface).
>
> Cheers,
>
>>
> -- 
> Erwan de FERRIERES
> www.nereide.biz
> 



Re: svn commit: r981846 - /ofbiz/branches/jquery/framework/widget/templates/htmlFormMacroLibrary.ftl

Posted by Erwan de FERRIERES <er...@nereide.fr>.
Le 03/08/2010 16:28, Adam Heath a écrit :
> Erwan de FERRIERES wrote:
>> Le 03/08/2010 14:52, Jacques Le Roux a écrit :
>>> OK I will fix my comment Adam asked for and will be able to ask for a
>>> better one here :o)
>>> Which bug is supposed to be fixed please?
>>>
>> Hi Jacques,
>> this is on the jquery branch, there was a display problem when date
>> fields where on the screen (some code was displayed in the user interface).
>
> So say that in the commit message.  The commit message needs to stand
> alone.  This email will not be linked to the commit.   The log will be
> read a long time from now, when the application integrator for company
> XYZ finally gets around to upgrading.
>
Ok, done. Thanks for taking time to comment !
Cheers,

-- 
Erwan de FERRIERES
www.nereide.biz

Re: svn commit: r981846 - /ofbiz/branches/jquery/framework/widget/templates/htmlFormMacroLibrary.ftl

Posted by Adam Heath <do...@brainfood.com>.
Erwan de FERRIERES wrote:
> Le 03/08/2010 14:52, Jacques Le Roux a écrit :
>> OK I will fix my comment Adam asked for and will be able to ask for a
>> better one here :o)
>> Which bug is supposed to be fixed please?
>>
> Hi Jacques,
> this is on the jquery branch, there was a display problem when date
> fields where on the screen (some code was displayed in the user interface).

So say that in the commit message.  The commit message needs to stand
alone.  This email will not be linked to the commit.   The log will be
read a long time from now, when the application integrator for company
XYZ finally gets around to upgrading.

Re: svn commit: r981846 - /ofbiz/branches/jquery/framework/widget/templates/htmlFormMacroLibrary.ftl

Posted by Erwan de FERRIERES <er...@nereide.fr>.
Le 03/08/2010 14:52, Jacques Le Roux a écrit :
> OK I will fix my comment Adam asked for and will be able to ask for a
> better one here :o)
> Which bug is supposed to be fixed please?
>
Hi Jacques,
this is on the jquery branch, there was a display problem when date 
fields where on the screen (some code was displayed in the user interface).

Cheers,

>
-- 
Erwan de FERRIERES
www.nereide.biz