[ http://issues.apache.org/jira/browse/TOMAHAWK-463?page=comments#action_12414502 ]
Russell Loewen commented on TOMAHAWK-463: ----------------------------------------- I'm gonna be honest that from my location I am having trouble checking out from SVN so all my looking is being done remotely, and my comment here is further based on the 1.1.2 branch. Am I correct that one is used for rendering the tags produced by SelectOneRadio when you are using it for display, and the other is used by the Radio tag when you are using SelectOneRadio with a spread layout? If so, I would say that a similar change needs to be made in both places, and even that we should really be getting it changed in the myfaces core as well. That is, the problem occurs in all SelectOneRadio components regardless of whether the developer is using spread/radio for their layout or just the default rendering for SelectOneRadio. It's not particular in the tomahawk code, but for my current application I am using the spread layout so that's why I reported the issue here. > for attribute not rendered on radio buttons > ------------------------------------------- > > Key: TOMAHAWK-463 > URL: http://issues.apache.org/jira/browse/TOMAHAWK-463 > Project: MyFaces Tomahawk > Type: Bug > Components: selectOneRadio / radio > Versions: 1.1.2 > Environment: n/a > Reporter: Russell Loewen > Assignee: Grant Smith > Priority: Minor > > SelectOneRadio doesn't render the "for" attribute on the label tag it creates > beside the radio button. The net effect is that on Internet Explorer the > user is not able to click on the label text to select the radio item. It's > arguable that this is IE's problem for not supported the parent-child > relationship between the label tag surrounding the radio button, but the for > attribute is a clearer association anyhow. > I believe that the following could fix the problem. > org.apache.myfaces.shared_impl.renderkit.html.HtmlResponseWriterImpl.class > after Line 163, insert: > _writer.writeAttribute(HTML.FOR_ATTR, > uiComponent.getClientId(facesContext), null); -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira
