You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ofbiz.apache.org by Nicolas Malin <ni...@nereide.fr> on 2016/09/18 15:21:52 UTC
Re: svn commit: r1761332 -
/ofbiz/trunk/applications/commonext/widget/SystemInfoForms.xml
Hi Michael
> <field name="taskAssignment"><check/></field>
> - <field name="updateButton"><submit/></field>
> + <field name="updateButton" title="${uiLabelMap.CommonUpdate}"><submit/></field>
> </form>
I didn't understand why you added this title. Normally the title
resolution need came from
<property key="FormFieldTitle_updateButton">. For me it's a
regression and after checked the Jira OFBIZ-8121 I didn't see a valid reason
Can you explain me more why ?
> <field name="noteInfo"><text size="50"/></field>
> - <field name="createButton"><submit/></field>
> + <field name="createButton" title="${uiLabelMap.CommonCreate}"><submit/></field>
> </form>
> </forms>
>
seem here
Nicoals
Re: svn commit: r1761332 -
/ofbiz/trunk/applications/commonext/widget/SystemInfoForms.xml
Posted by Nicolas Malin <ni...@nereide.fr>.
No worries Michael, thanks for your reactivity
Nicolas
Le 18/09/2016 � 17:36, Michael Brohl a �crit :
> Hi Nicolas,
>
> you are right, that change is also not necessary. Seems I was confused
> while reviewing the other changes in the referenced Jira and the
> related subtasks.
>
> I will revert the commit.
>
> Thanks for taking care,
>
> Michael
>
>
> Am 18.09.16 um 17:21 schrieb Nicolas Malin:
>> Hi Michael
>>> <field name="taskAssignment"><check/></field>
>>> - <field name="updateButton"><submit/></field>
>>> + <field name="updateButton"
>>> title="${uiLabelMap.CommonUpdate}"><submit/></field>
>>> </form>
>> I didn't understand why you added this title. Normally the title
>> resolution need came from
>> <property key="FormFieldTitle_updateButton">. For me it's a
>> regression and after checked the Jira OFBIZ-8121 I didn't see a valid
>> reason
>>
>> Can you explain me more why ?
>>> <field name="noteInfo"><text size="50"/></field>
>>> - <field name="createButton"><submit/></field>
>>> + <field name="createButton"
>>> title="${uiLabelMap.CommonCreate}"><submit/></field>
>>> </form>
>>> </forms>
>>>
>> seem here
>>
>> Nicoals
>
>
Re: svn commit: r1761332 -
/ofbiz/trunk/applications/commonext/widget/SystemInfoForms.xml
Posted by Michael Brohl <mi...@ecomify.de>.
Hi Nicolas,
you are right, that change is also not necessary. Seems I was confused
while reviewing the other changes in the referenced Jira and the related
subtasks.
I will revert the commit.
Thanks for taking care,
Michael
Am 18.09.16 um 17:21 schrieb Nicolas Malin:
> Hi Michael
>> <field name="taskAssignment"><check/></field>
>> - <field name="updateButton"><submit/></field>
>> + <field name="updateButton"
>> title="${uiLabelMap.CommonUpdate}"><submit/></field>
>> </form>
> I didn't understand why you added this title. Normally the title
> resolution need came from
> <property key="FormFieldTitle_updateButton">. For me it's a
> regression and after checked the Jira OFBIZ-8121 I didn't see a valid
> reason
>
> Can you explain me more why ?
>> <field name="noteInfo"><text size="50"/></field>
>> - <field name="createButton"><submit/></field>
>> + <field name="createButton"
>> title="${uiLabelMap.CommonCreate}"><submit/></field>
>> </form>
>> </forms>
>>
> seem here
>
> Nicoals