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

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
5 messages Options
Reply | Threaded
Open this post in threaded view
|

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

Jacques Le Roux
Administrator
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/>
>
>


Reply | Threaded
Open this post in threaded view
|

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

Erwan de FERRIERES
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
Reply | Threaded
Open this post in threaded view
|

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

Jacques Le Roux
Administrator
Thanks Erwan

Jacques

From: "Erwan de FERRIERES" <[hidden email]>

> 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
>


Reply | Threaded
Open this post in threaded view
|

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

Adam Heath-2
In reply to this post by Erwan de FERRIERES
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.
Reply | Threaded
Open this post in threaded view
|

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

Erwan de FERRIERES
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