You can check out the tomahawk 1.1.3 source, and just apply the
t:radio patches to it. You just add one line.
I think these are the only changes I had to make to get onclick
working under facelets. Under jsp, you'd also have to update the Tag
file and the tld file.
Index:
E:/myfaces/myfaces-current-2006-08-15-17-51/tomahawk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlRadioRenderer.java
===================================================================
---
E:/myfaces/myfaces-current-2006-08-15-17-51/tomahawk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlRadioRenderer.java
(revision
431891)
+++
E:/myfaces/myfaces-current-2006-08-15-17-51/tomahawk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlRadioRenderer.java
(working
copy)
@@ -131,7 +131,9 @@
writer.startElement(HTML.LABEL_ELEM, uiSelectOne);
- renderRadio(facesContext,
+ uiSelectOne.getAttributes().put(HTML.ONCLICK_ATTR,
radio.getAttributes().get(HTML.ONCLICK_ATTR));
+
+ renderRadio(facesContext,
uiSelectOne,
itemStrValue,
selectItem.getLabel(),
On 9/25/06, Thorbjørn Ravn Andersen <[EMAIL PROTECTED]> wrote:
Mike Kienenberger skrev den 18-09-2006 01:52:
> Hey Greg,
>
> I finished doing this in my application a couple of weeks ago.
> I used t:selectOneRadio and t:radio. I had to extend t:radio to
> allow it to take an onclick attribute. I was going to hold off until
> the 1.1.4 Tomahawk branch before committing my changes, but perhaps
> I'll try committing them to trunk tomorrow.
>
I have run into the same issue, so I would appreciate this.
Unfortunately I am very close to a deadline and therefore reluctant to
bring in a cvs snapshot, so if there is another way which is supported
by myfaces 1.1.4 plus tomahawk 1.1.3 I would *love* to hear about it :)
--
Thorbjørn
