> > 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. I am for one basic theme. Most projects maintain one basic theme and leave further customization and development up to the design community. I see our task as developing and maintaining a solid basic theme. Maybe that can spawn a templating community of its own. Gavin On Wed, Jun 24, 2015 at 2:58 PM, Pierre Smits <pierre.sm...@gmail.com> wrote: > 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 > > >>>>>>> > > >>>>>>> > > >>>>>>> > > >>>>>>> > > >