On Mon, 9 Jun 2003, Matt Raible wrote:
> Rather than adding two new tags to the tag library, I think it'd be easier to
> make enhancements such as the following:
>
> 1. If the <display:column> has a body, then use that as the title.
>
> <display:column property="fullName" sort="true" width="15%"
> title='<%=resources.getMessage("userForm.fullName")%>' />
>
> could become (in struts):
>
> <display:column property="fullName" sort="true" width="15%">
> <bean:message key="userForm.fullName"/>
> </display:column>
>
> However, the problem with this is that it'd be nice to expose the object for
> use with an expression like syntax, so the following would be possible:
>
> <display:column property="fullName" sort="true" width="15%">
> <input type="checkbox" name="fullName${status.index}"
> value="${userForm.fullName}"/>
> </display:column>
I don't think this will work with the current structure of this tag.
Without using a real iterate-like structure, you'll never really be able
to customize the pieces that most users want to customize without adding
hacks like decorators.
I don't understand why you would want to be able to override the title
attribute from within the <display:column>, the table column only has 1
title, so what's wrong with using the title attribute to <display:column>?
I know I've said this a million times, but these are exactly the reasons
why I refactored all this code to begin with. I'm not even sure it's worth
adding any new functionality in the existing code base. In my opinion, you
be better off declaring a brand new 1.0 release that will have better
structure and customizabilty but not necessarily be backward compatible.
Many of the features in this tag are not needed once you can actually
customize the <display:column> bodies.
Sorry for the rant.
John
-------------------------------------------------------
This SF.net email is sponsored by: Etnus, makers of TotalView, The best
thread debugger on the planet. Designed with thread debugging features
you've never dreamed of, try TotalView 6 free at www.etnus.com.
_______________________________________________
displaytag-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/displaytag-devel
