Thanks for the response, Yes, your implementation looks Better than mine and the idea of turning 'fields in error red' is also nice, that's what I wanted to know if there Is already anything like this going, Hope this tag is added to Struts soon.
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.
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.
Erik
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
