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)