I know what your mean, this is need to add more code to this class. if I want to add dojo enabled, I have to add if(dojoEnable) and will let the this class become hard to understand
2009/3/2 Adrian Crum <[email protected]> > > --- On Sun, 3/1/09, guo weizhan <[email protected]> wrote: > > We are creating the "theme" for the ofbiz widget > > form, why we create this? > > We found it's very difficultty to extend the current > > form widget, all the > > widget generation code is hard code in HtmlFormRenderer, we > > need to change > > this file if we want to add some function to the form, like > > ajax. > > > > And yes, we have some ajax function in ofbiz now, but the > > code of > > HtmlFormRenderer with ajax is much harder for > > understanding. > > like the code: "if(ajaxEnabled)". In fact , we > > want to implement the dojo > > function in ofbiz without mess. > > The reason the widget code checks to see if Ajax is enabled is because some > users will have JavaScript disabled - in which case, plain HTML needs to be > rendered. > > -Adrian > > > > >
