[jbossseam-issues] [JBoss JIRA] Commented: (JBSEAM-547) s:button generates multiple onclick attributes

[Norman Richards (JIRA)]

    [ http://jira.jboss.com/jira/browse/JBSEAM-547?page=comments#action_12348043 ] 
            
Norman Richards commented on JBSEAM-547:
----------------------------------------

Not sure how we are prioritizing changes so close to 1.1 final, so ...

Index: HtmlButton.java
===================================================================
RCS file: /cvsroot/jboss/jboss-seam/src/ui/org/jboss/seam/ui/HtmlButton.java,v
retrieving revision 1.2
diff -r1.2 HtmlButton.java
193c193
<       HTML.renderHTMLAttributes(writer, this, HTML.BUTTON_PASSTHROUGH_ATTRIBUTES_WITHOUT_DISABLED);
---
>       HTML.renderHTMLAttributes(writer, this, HTML.BUTTON_PASSTHROUGH_ATTRIBUTES_WITHOUT_DISABLED_AND_ONCLICK);

> s:button generates multiple onclick attributes
> ----------------------------------------------
>
>                 Key: JBSEAM-547
>                 URL: http://jira.jboss.com/jira/browse/JBSEAM-547
>             Project: JBoss Seam
>          Issue Type: Bug
>          Components: JSF
>            Reporter: Norman Richards
>            Priority: Minor
>
> <s:button onclick="alert(hi);" action="#{some.action} /> generates  multiple onclick attributes.  On my browser the second one is ignored, but I don't know if that works everywhere. 
> <input type="button" id="_id19" onclick="alert(hi);location.href='/seam-pay/count.seam?actionMethod=count.xhtml%3Asome.action'" onclick="alert(hi);" value="test" />

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