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 2017/08/09 12:14:21 UTC
Re: Restore common-theme with ofbiz-framework and now it's ready to
play
Hello,
I push some little correction on the common-theme branch [1] (I have
been detected that the specific-template macro used always html, you can
see here [2] all commits)
If you have some time to test it to have any feedback ! It's would be
really appreciate because I prefer don't keep the branch much longer :)
. All is green from my part so if you detect a regression I'm available
to correct it.
Nicolas
[1] https://github.com/nmalin/ofbiz-framework/tree/common-theme
[2]
https://github.com/apache/ofbiz-framework/compare/trunk...nmalin:common-theme?expand=1
Le 28/07/2017 à 22:50, Nicolas Malin a écrit :
> It's with a great pleasure that can be announce : common-theme is
> operational !
>
> Sure all isn't supported, but now
>
> can by theme surcharge :
>
> * widget properties
> * ftl template macro
> * common screen
>
> The maximum of image, js, ftl was moved from common to common-theme,
> implement a new object ModelTheme to offer the access to theme
> properties directly form context and I currently detect no regression.
> For a first step, I think it's a good step ^^
>
> So if you want to play with it, it's the time !
>
> Cheers,
>
> Nicolas
>
>
> Le 27/07/2017 à 20:37, Nicolas Malin a écrit :
>> Hi Jacques,
>>
>>
>> Le 26/07/2017 à 19:32, Jacques Le Roux a écrit :
>>> Hi Nicolas,
>>>
>>> Maybe you can create a Jira (is there one already?) and add patches
>>> explaining they should not be committed but only reviewed.
>>> Especially when you feel your work is ready enough to be
>>> reviewed/used/tested even if not committed.
>> Yes the jira is OFBIZ-9138 - Create a common theme
>> <https://issues.apache.org/jira/browse/OFBIZ-9138>
>>
>> For the patch, at the beginning I tried but it's to complicate to
>> manage and review for both.
>>
>> It's the reason that I prefer continue only on my github account [1]
>> where it's more easier to learn commit by commit the way with the
>> comparison from github apache repo [2]
>>
>>> Taher does that sometimes, it's convenient for early review and no
>>> problems (too early commit by someone else) never happened. Anyway a
>>> revert is always very easy ;)
>> Sure but before I will continue to consolidate my works and I will
>> create some diagram to help how the theme will works :)
>>
>> Cheers,
>> Nicolas
>>
>> [1] https://github.com/nmalin/ofbiz-framework/tree/common-theme
>> [2]
>> https://github.com/apache/ofbiz-framework/compare/trunk...nmalin:common-theme?expand=1
>>>
>>> Thanks
>>>
>>> Jacques
>>>
>>>
>>> Le 26/07/2017 à 00:34, Nicolas Malin a écrit :
>>>> I continue the way and my recent success :
>>>> * set the object theme always present for macro rendering
>>>> * create a theme that extends common-theme and specially the
>>>> htmlFormMacroLibrary with surchage only the renderDisplayField
>>>> macro like that
>>>>
>>>> <#include
>>>> "component://common-theme/template/macro/HtmlFormMacroLibrary.ftl"/>
>>>> <#macro renderDisplayField type imageLocation idName description
>>>> title class alert inPlaceEditorUrl="" inPlaceEditorParams="">
>>>> <#if description?has_content>
>>>> **${description?replace("\n", "<br />")}**<#t/>
>>>> <#else>
>>>> <#t/>
>>>> </#if>
>>>> </#macro>
>>>>
>>>> Next step, I will try to surchage only one decorator
>>>>
>>>> All my work has been committed on my git repo
>>>> https://github.com/nmalin/ofbiz-framework/tree/common-theme
>>>>
>>>> Nicolas
>>>>
>>>> Le 21/07/2017 à 18:26, Nicolas Malin a écrit :
>>>>> [...]
>>>>>>>
>>>>>>> On Tue, Jul 18, 2017 at 5:38 PM, Nicolas Malin
>>>>>>> <ni...@nereide.fr> wrote:
>>>>>>>> Michael,
>>>>>>>>
>>>>>>>> I finished to restore my work on common-theme from ofbiz to
>>>>>>>> ofbiz-framework
>>>>>>>> :
>>>>>>>> https://github.com/apache/ofbiz-framework/compare/trunk...nmalin:common-theme?expand=1
>>>>>>>>
>>>>>>>>
>>>>>>>> Now I will work on the java object to manage the widget theme
>>>>>>>> on user
>>>>>>>> session.
>>>>>>>>
>>>>>>>> Nicolas
>>>>>>>>
>>>>>>>> --
>>>>>>>> logoNrd <https://nereide.fr/>
>>>>>>>> Nicolas Malin
>>>>>>>> The apache way <http://theapacheway.com/> : *Openness*
>>>>>>>> Technical decisions
>>>>>>>> are made publicly
>>>>>>>> information@nereide.fr
>>>>>>>> 8 rue des Déportés 37000 TOURS, 02 47 50 30 54
>>>>>>>>
>>>>>>>> Apache OFBiz <http://ofbiz.apache.org/>|The Apache Way
>>>>>>>> <http://theapacheway.com/>|ofbiz-fr
>>>>>>>> <http://www.ofbiz-fr.org/>|réseau LE
>>>>>>>> <http://www.libre-entreprise.org/>
>>>>>>
>>>>>>
>>>>>
>>>>>
>>>>
>>>>
>>>
>>>
>>
>>
>
>