[ http://jira.codehaus.org/browse/DISPL-66?page=comments#action_53192 ] 

Emilio Dolce commented on DISPL-66:
-----------------------------------

I suggest to simply move the code:
            if (this.tableModel.getTableDecorator() != null)
            {
                String stringStartRow = 
this.tableModel.getTableDecorator().startRow();
                if (stringStartRow != null)
                {
                    write(stringStartRow, out);
                }
            }
present in TableTag class after open TR tag and let TableDecorator to decide 
what render as jscript event for TR tag such as onclick or whatever.
Actually the startRow callback function is not useful before the TR tag.

If you agree with this solution I can provide the patch for you.

> Customizable code in the <td>and <tr> tags
> ------------------------------------------
>
>          Key: DISPL-66
>          URL: http://jira.codehaus.org/browse/DISPL-66
>      Project: DisplayTag
>         Type: Wish

>   Components: Tag Library
>     Versions: 1.0 RC1
>     Reporter: fabrizio giustina
>     Priority: Minor

>
>
> ====
> imported from sf tracker
> id 825670 
> submitted by Christoph Schmidt - k-os
> http://sourceforge.net/support/tracker.php?aid=825670 
> ====
> It would be nice if i could add my own code to the <tr>
> and <td> tags, like onMouseOver, onClick Events etc.
> For example a Row Highlighting could then be done via
> Javascript...
> maybe something like this:
> <display:table name="test" class="<%=lClass%>">
> <display:row customCode="onClick="hlgtRow()"/>
> <display:column property="id" title="ID"
> class="idcol"/ customCode="....">
> <display:column property="name" />
> ....
> </display:table>
> (For higlighting i think i need a <tr id="r1" " ... >
> and then a hlgtRow('r1') to identify which dom object i
> should change the stylesheet... anyone an idea how
> this could be done via the taglib?)
> ===
> Date: 2003-10-17 21:03
> Sender: mraible
> Logged In: YES 
> user_id=226669
> You don't need id's on tr's or td's in order to do this - not
> if you're
> using modern browsers.  Here's how:
> http://tinyurl.com/rbwg
> ===
> Date: 2003-10-18 11:08
> Sender: k-os
> Logged In: YES 
> user_id=77543
> to mraible: cool .. that will do.. thank you (-:
> I noticed that the patch &quot;716857 &lt;display:row&gt; tag
> implementation + RowDecorator&quot; woukd solve my request. But
> it seems to be coded for version 0.8.6.
> Is there a chance that this featur will get into the 1.0?

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira



-------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc. Do you grep through log files
for problems?  Stop!  Download the new AJAX search engine that makes
searching your log files as easy as surfing the  web.  DOWNLOAD SPLUNK!
http://ads.osdn.com/?ad_id=7637&alloc_id=16865&op=click
_______________________________________________
displaytag-devel mailing list
displaytag-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/displaytag-devel

Reply via email to