> Even if its not added to Struts (and there is a reason not to - it > relies on naming conventions FormName.fieldName - which seem reasonable > > to me, but aren't mandated by Struts), its all yours to use. It makes > sense to put it in some kind of sandbox or "goodies"/contrib area of > the Struts codebase though.
Erik, I really like the idea of the tag and I've added my suggestions for improvement to the bug report. http://nagoya.apache.org/bugzilla/show_bug.cgi?id=18015 > > > Also, just a query, Does your tag extend MessageTag ? > > Is it a good idea to do that ? > > > > No, it doesn't. See here: > > http://cvs.sourceforge.net/cgi-bin/viewcvs.cgi/*checkout*/struts/ > struts-resume/src/web/org/appfuse/webapp/taglib/ > LabelTag.java?rev=HEAD&content-type=text/plain > > I personally do not find it worthwhile to extends Struts tags. They > are not designed as I would have designed them (subclassses not > exposing some parent class attributes in the TLD is *bad* IMHO) and the > attributes on the MessageTag are not needed for my label tag. All I > need is key, nothing else. I made some effort recently to refactor the tags into more appropriate chunks. While they're still not great for subclassing, they have improved. David > > Erik > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > __________________________________ Do you Yahoo!? SBC Yahoo! DSL - Now only $29.95 per month! http://sbc.yahoo.com --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
