[jira] [Commented] (WICKET-6348) extract #onSelectionChange() from choices to behavior

["ASF subversion and git services (JIRA)" <ji...@apache.org>]

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

ASF subversion and git services commented on WICKET-6348:
---------------------------------------------------------

Commit 15573cabcf44768fe2631bb9b4b6dcf5e2b2e494 in wicket's branch refs/heads/WICKET-6348-selection-change from [~svenmeier]
[ https://git-wip-us.apache.org/repos/asf?p=wicket.git;h=15573ca ]

WICKET-6348 use submitter instead of hidden field


> extract #onSelectionChange() from choices to behavior
> -----------------------------------------------------
>
>                 Key: WICKET-6348
>                 URL: https://issues.apache.org/jira/browse/WICKET-6348
>             Project: Wicket
>          Issue Type: Improvement
>          Components: wicket
>    Affects Versions: 8.0.0-M4
>            Reporter: Sven Meier
>            Assignee: Sven Meier
>            Priority: Minor
>
> Several choice components support notification via normal HTTP request when their value changes in the browser:
> - DropDownChoice
> - RadioChoice
> - CheckGroup/Check
> - RadioGroup/Radio
> I'd propose to move support for this feature into a new behavior.
> This has the following advantages:
> - having to override #wantOnSelectionChangedNotifications() for #onSelectionChanged() to be triggered wasn't very intuitive anyway
> - we minimize the API of these components
> - we can simplify these components by removing from them this non-core concern (a legacy from the pre-Ajax era)
> - to use the feature users can add a behavior instead to have a notification triggered on *that* behavior (similar to AjaxFormComponentUpdatingBehavior/AjaxFormChoiceComponentUpdatingBehavior)
> I reused IFormSubmitter (for SubmitLink too) so we can now simplify Form:
> - no need for the hidden field "_hf_0" (the form's action is changed instead)
> - no need for #dispatchEvent()
> - #getJsForInterfaceUrl() (now #getJsForListenerUrl()) is greatly simplified



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)