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

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

Commit 21f42085a02ebd1f82cf08425ea3645354ad6131 in wicket's branch 
refs/heads/master from Viliam Repan
[ https://gitbox.apache.org/repos/asf?p=wicket.git;h=21f42085a0 ]

WICKET-7123 default form submit can now be set one per form (#971)

* WICKET-7123 default form submit can now be set one per form (not form 
hierarchy)

* WICKET-7123 fixed missing license header

* WICKET-7123 removed jquery from form script

* WICKET-7123 added example for default form submit used in form hierarchy

* WICKET-7123 replaced wrong element (td)

* WICKET-7123 Minor improvements to the example/demo page

Signed-off-by: Martin Tzvetanov Grigorov <[email protected]>

---------

Signed-off-by: Martin Tzvetanov Grigorov <[email protected]>
Co-authored-by: Martin Tzvetanov Grigorov <[email protected]>

> Form.setDefaultButton() supports only one button per form hierarchy
> -------------------------------------------------------------------
>
>                 Key: WICKET-7123
>                 URL: https://issues.apache.org/jira/browse/WICKET-7123
>             Project: Wicket
>          Issue Type: Improvement
>          Components: wicket-core
>    Affects Versions: 10.1.0
>            Reporter: Viliam Repan
>            Priority: Major
>
> Currently Form.setDefaultButton() support only one button per whole form 
> hierarchy, which is a problem when there are multiple forms on page of for 
> example popup. It complicates even further when those default submit buttons 
> have custom visibility/enabled state.
> Current implementation puts hidden field in root form which stores name of 
> submit button and then creates client JS click() via jquery.
> This should be easy doable directly on form where the default button is 
> located without the need to pass it to root form.
> What do you think about this?
> PR available here https://github.com/apache/wicket/pull/968



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to