Hi folks, I been using the displaytag for a couple of months now, and there is one outstanding request I have to complete for fully compliance with our corporate application style guides. The problem is that our style guide requires that the sortable column indicators (little arrows in the column headers) must be the only clickable element in the header.
This causes two problems: 1) I can't put an image in the header that is clickable (not without pushing html into the title attribute 2) I can't stop the title text being part of the sorting anchor tag. So I have been doing a little work into modifying 1.0 to allow me to do this. My proposal is to incorporate a new decorator interface, ColumnTitleDecorator. This defines one method: String decorateTitle(TableModel tableModel, HeaderCell headerCell, Href sortingHref, PageContext pageContext); The title decorator is set on a per table basis, i.e. one table one column title decorator. I've integrated this into the TableTag.writeHeaders() which delegate the construction of the header cell contents to the decorator if present. All seems well, all the unit tests run and coverage is pretty much the same as before. Is anybody interested in me submitting this as a patch in the issue tracking system? I have to update the examples and the documentation first but my employer is keen for this to be accepted and has given me the time to ensure that all the required elements (unit test/documentation etc) are complete to your satisfaction. Gareth Davis ------------------------------------------------------- The SF.Net email is sponsored by: Beat the post-holiday blues Get a FREE limited edition SourceForge.net t-shirt from ThinkGeek. It's fun and FREE -- well, almost....http://www.thinkgeek.com/sfshirt _______________________________________________ displaytag-devel mailing list displaytag-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/displaytag-devel
