I like the idea, but I think I would implement it as an additional taglib that could inherit from the existing display tag. I think this feature is beyond the scope of the current displaytag, so I'd rather not jumble in a bunch of logic in the core tag. We've already got way too much of that. I'd rather see us have to refactor a few things and make it possible to derive a new tag from the existing one.
The only problem I see with this approach is that if we decide to do it again for some other functionality, now you won't be able to do both at the same time. Maybe there's some way to create more of a pluggable architecture that would allow us to add the chunks of functionality we need without adding in any unecessary overhead. John On Wed, 29 Oct 2003, [EMAIL PROTECTED] wrote: > I've actually had a better idea. Rather than changing the ColumnDecorator > interface, how about we add another Decorator type - a ControlDecorator? The > ControlDecorator would create the form control, using the result of the > ColumnDecorator as the display? > > Quoting [EMAIL PROTECTED]: > > > I have been looking at adding some simple support to the Display Tag > > foreditable > > tables. My current idea is to extend the ColumnDecorator class's decorate > > method. Currently, the method just takes the object value. My proposed > > alteration would have the column name and row number passed in. This would > > allow > > a decorator to create a properly named form control. > > <snip> > > > sam > http://www.magpiebrain.com/ > > > > ------------------------------------------------------- > This SF.net email is sponsored by: SF.net Giveback Program. > Does SourceForge.net help you be more productive? Does it > help you create better code? SHARE THE LOVE, and help us help > YOU! Click Here: http://sourceforge.net/donate/ > _______________________________________________ > displaytag-devel mailing list > [EMAIL PROTECTED] > https://lists.sourceforge.net/lists/listinfo/displaytag-devel > -- John York Software Engineer CareerSite Corporation ------------------------------------------------------- This SF.net email is sponsored by: SF.net Giveback Program. Does SourceForge.net help you be more productive? Does it help you create better code? SHARE THE LOVE, and help us help YOU! Click Here: http://sourceforge.net/donate/ _______________________________________________ displaytag-devel mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/displaytag-devel
