As Matt already pointed out, you can use DOM to add listeners for
column/row events, you can find other examples at
http://www.alistapart.com/articles/domtricks2/

However, I think we should add the attribute to the column tag. I think
column tag should support at least all the attributes valid in xhtml
STRICT (while I'd like to remove all the deprecated attributes from the
transitional dtd in later versions). I don't think it's a problem adding
html specific attributes, since html is always a "preferred" output
(link, paging, sorting, styles, are html specific).

Instead, I'm against in adding attribute to the main table tag to output
row attributes. If we want to let the user specify attributes for a row
probably the best way is to expand the current tag structure with
something like:

<display:table>
    <display:row (table row attributes here)>
        <display:column>
            <display:columnhead>header using
resources</display:columnhead>
            <display:columnbody>if not using
property</display:columnbody>
        </display:column>
    </display:row>
    <display:footer>
          ... maybe a footer to add totals or other 
          without the need of a tabledecorator?
    </display:footer>
</display:table>

I think this could lead to a more flexible structure for future
implementations, but it could be difficult to mantain compatibility with
the current version...

I don't really like the idea of using the onMouseXxx handlers to output
"filtered" javascript functions too. Better let them output directly the
user defined attribute, maybe in future with le expression support.

my 0.02 $
fabrizio


-------------------------------------------------------
This SF.net email is sponsored by: The SF.net Donation Program.
Do you like what SourceForge.net is doing for the Open
Source Community?  Make a contribution, and help us add new
features and functionality. Click here: http://sourceforge.net/donate/
_______________________________________________
displaytag-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/displaytag-devel

Reply via email to