If we, as a community, are opting for multiple themes in one component, that would sure fit the bill. But is that what we want? Add complexities and bulk to a somewhat global theme management component?
Or should we just have a strategy of one theme - one component? I am in favour of the latter option, even if it means some kind of duplication of functionality. Best regards, Pierre Smits *ORRTIZ.COM <http://www.orrtiz.com>* Services & Solutions for Cloud- Based Manufacturing, Professional Services and Retail & Trade http://www.orrtiz.com On Wed, Jun 24, 2015 at 2:42 PM, Gavin Mabie <kwikst...@gmail.com> wrote: > > > > I did not review anything but from your explanation having > > separated/specific header.ftl & appbar.ftl files in "Sunrise" makes sense > > to me (since it's a "skin") > > BTW I understand that <<Sunrise is a "skin">> because it's mostly a copy > > of basic, right? > > > That's right. If Julien is okay with it, I can move his commits to new > folder "sunrise" under bootstrap/includes. The "sunrise" folder can serve > as placeholder for templates that deviate from the "basic" templates. > > I agree with Pierre that we should try to get this into the trunk sooner > rather then later because of the massive refactoring work. New issues will > definitely emerge once in the trunk, but we can deal with that there. > > Gavin > > On Wed, Jun 24, 2015 at 12:58 PM, Jacques Le Roux < > jacques.le.r...@les7arts.com> wrote: > > > Le 24/06/2015 11:47, Gavin Mabie a écrit : > > > >> Four issues: > >> > >> > >> 1. The Bootstrap Basic and "Bootstrap Sunrise" is in fact just one > >> theme > >> with Basic as theme and Sunrise as a "skin" (implementation of > Basic). > >> Hence the location of the css for Sunrise under bootstrap/css/skins. > >> Other > >> than that, Sunrise uses the same template libraries as Basic. It is > a > >> false > >> choice. > >> 2. If we want to elevate Sunrise to theme status (i.e not just a > >> skin), > >> then we have to create separate template libraries for it. To > qualify > >> as a > >> theme, it should have its own distinct widget implementation > including > >> headers, menus, forms, tables, pagination etc. > >> 3. The commits by Julien - r1683430 (header.ftl & appbar.ftl) > affects > >> both Basic and Sunrise. I have not seen patches for these and could > >> therefore not do reviews. > >> 4. The approach with the Basic theme is to keep it as basic as > >> possible, > >> minimizing personal preferences with regards to look-and-feel and > >> leaving > >> this level of styling up to individual designers. > >> > >> I have committed patches to deal with most if not all the issues flagged > >> by > >> Adrian on 19 May 2015. New issues have cropped up as a result of > >> r1683430. To get this merge-ready my recommendation is that we revert > >> r1683430 which deals with header.ftl & appbar.ft. These are minor > issues > >> which relate mainly to personal preferences. If the changes introduced > >> with > >> r1683430 is absolutely necessary, then I recommend that separate > >> header.ftl > >> & appbar.ftl files are created and that themeResources in "Sunrise" > point > >> to locations where the new files reside. > >> > > > > I did not review anything but from your explanation having > > separated/specific header.ftl & appbar.ftl files in "Sunrise" makes sense > > to me (since it's a "skin") > > BTW I understand that <<Sunrise is a "skin">> because it's mostly a copy > > of basic, right? > > > > Jacques > > > > > > > >> Regards > >> > >> Gavin > >> > >> > >> > >> > >> > >> On Wed, Jun 24, 2015 at 12:21 AM, Pierre Smits <pierre.sm...@gmail.com> > >> wrote: > >> > >> I am running the Bootstrap Basic (somewhat modified) against trunk. > >>> > >>> Best regards, > >>> > >>> Pierre Smits > >>> > >>> *ORRTIZ.COM <http://www.orrtiz.com>* > >>> Services & Solutions for Cloud- > >>> Based Manufacturing, Professional > >>> Services and Retail & Trade > >>> http://www.orrtiz.com > >>> > >>> On Tue, Jun 23, 2015 at 9:56 PM, Adrian Crum < > >>> adrian.c...@sandglass-software.com> wrote: > >>> > >>> I evaluated the bootstrap branch as it currently exists. If there are > >>>> patches waiting to be applied to the branch, then I am not aware of > >>>> them. > >>>> > >>>> Adrian Crum > >>>> Sandglass Software > >>>> www.sandglass-software.com > >>>> > >>>> On 6/23/2015 12:43 PM, Pierre Smits wrote: > >>>> > >>>> Hi Adrian, > >>>>> > >>>>> Thanks for the feedback. That the existing patch files available in > >>>>> JIRA > >>>>> issues don't work in trunk has to do with the fact that the bootstrap > >>>>> > >>>> dev > >>> > >>>> branch is not in sync with trunk. We have to take in consideration > that > >>>>> the > >>>>> framework stack of the bootstrap branch is based (for the greater > part) > >>>>> > >>>> on > >>> > >>>> r1634810. > >>>>> > >>>>> On top of that, the Bootstrap Basic and the other one were > >>>>> co-developed. > >>>>> And shortly before the disentanglement of the two themes changes in > the > >>>>> templates were implemented for the other theme that affected the > >>>>> > >>>> Bootstrap > >>> > >>>> Basic theme negatively. > >>>>> > >>>>> Nonetheless, I am ok with whatever which way the community chooses to > >>>>> go > >>>>> with these bootstrap themes and the ones in trunk. It remains a > >>>>> personal > >>>>> preference what one likes best. > >>>>> > >>>>> Best regards, > >>>>> > >>>>> > >>>>> Pierre Smits > >>>>> > >>>>> *ORRTIZ.COM <http://www.orrtiz.com>* > >>>>> Services & Solutions for Cloud- > >>>>> Based Manufacturing, Professional > >>>>> Services and Retail & Trade > >>>>> http://www.orrtiz.com > >>>>> > >>>>> On Tue, Jun 23, 2015 at 5:20 PM, Adrian Crum < > >>>>> adrian.c...@sandglass-software.com> wrote: > >>>>> > >>>>> I don't think the new themes are ready. I updated my local copy > >>>>> > >>>> yesterday > >>> > >>>> and tried them out - many of the layout issues I reported still exist, > >>>>>> plus > >>>>>> I found another one. > >>>>>> > >>>>>> I don't mind if there are just a few minor quirks - those can be > fixed > >>>>>> after the themes are in the trunk, but right now there are too many. > >>>>>> > >>>>>> Also, we need to discuss how many themes we want to include in the > >>>>>> > >>>>> trunk. > >>> > >>>> The Bootstrap Basic theme doesn't seem to get as much attention as the > >>>>>> other one, and it shows - its layout is much worse. I suggest we > port > >>>>>> over > >>>>>> one of the themes instead of two. > >>>>>> > >>>>>> Also, it would be nice to drop one or two existing themes. > >>>>>> > >>>>>> > >>>>>> Adrian Crum > >>>>>> Sandglass Software > >>>>>> www.sandglass-software.com > >>>>>> > >>>>>> On 6/23/2015 2:08 AM, Pierre Smits wrote: > >>>>>> > >>>>>> Hi all, > >>>>>> > >>>>>>> Recently we have seen that great strides have been made with > respect > >>>>>>> > >>>>>> to > >>> > >>>> widget refactoring, theme functions disentanglement from framework ( > >>>>>>> OFBIZ-6362 <https://issues.apache.org/jira/browse/OFBIZ-6362>) and > >>>>>>> in > >>>>>>> the > >>>>>>> Bootstrap dev branch (OFBIZ-5840 > >>>>>>> <https://issues.apache.org/jira/browse/OFBIZ-5840>). > >>>>>>> > >>>>>>> Based on this all I am inclined to believe that both the Bootstrap > >>>>>>> > >>>>>> Basic > >>> > >>>> and the Bootstrap Sunrise themes are trunk ready. > >>>>>>> > >>>>>>> What do you think? > >>>>>>> > >>>>>>> Best regards, > >>>>>>> > >>>>>>> Pierre Smits > >>>>>>> > >>>>>>> *ORRTIZ.COM <http://www.orrtiz.com>* > >>>>>>> Services & Solutions for Cloud- > >>>>>>> Based Manufacturing, Professional > >>>>>>> Services and Retail & Trade > >>>>>>> http://www.orrtiz.com > >>>>>>> > >>>>>>> > >>>>>>> > >>>>>>> >