You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by "Sven Meier (Jira)" <ji...@apache.org> on 2019/12/16 09:20:00 UTC

[jira] [Commented] (WICKET-6718) AjaxFormChoiceComponentUpdatingBehavior not working as expected with RadioGroup/RadioChoice

    [ https://issues.apache.org/jira/browse/WICKET-6718?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16997110#comment-16997110 ] 

Sven Meier commented on WICKET-6718:
------------------------------------

Internet Exporer up till version 8 didn't reliably fire the "change" event when you select a radio.

For Wicket 9 we could change the event to "change", since older IE versions are no longer supported anyway.
A second constructor for 8.x would be fine by me.

> AjaxFormChoiceComponentUpdatingBehavior not working as expected with RadioGroup/RadioChoice
> -------------------------------------------------------------------------------------------
>
>                 Key: WICKET-6718
>                 URL: https://issues.apache.org/jira/browse/WICKET-6718
>             Project: Wicket
>          Issue Type: Bug
>          Components: wicket-core
>    Affects Versions: 8.6.1
>            Reporter: Christian Tausendfreund
>            Priority: Major
>
> When using the AjaxFormChoiceComponentUpdatingBehavior on a RadioGroup or a RadioChoice my expectation is that the onUpdateChoice callback is only triggered if there is a change of the selected radio button. But it also triggers if you click the already selected radio button again. This is because the AjaxFormChoiceComponentUpdatingBehavior listens to the "click" event.
> To fix this I've built my own AjaxFormChoiceComponentUpdatingBehavior which listens to the "change" event. This works as expected and only triggers on a change of the selected button.
> To fix this issue, could you please change the event string in AjaxFormChoiceComponentUpdatingBehavior from "click" to "change" or provide a 2nd constructor with the event string as parameter? Thanks!



--
This message was sent by Atlassian Jira
(v8.3.4#803005)