svenmeier commented on pull request #467:
URL: https://github.com/apache/wicket/pull/467#issuecomment-816178510


   IMHO OnFileSelectedBehavior deserves to be a top-level class. Maybe with its 
own JS resource, since the script to collect dynamicExtraParameters is getting 
very long now. 
   
   With a static factory method users could use a lambda expression (like we 
have for several other behaviors, e.g. AjaxFormSubmitBehavior#onSubmit):
   
       file.add(OnFileSelectedBehavior.onSelected((AjaxRequestTarget target, 
List<OnFileSelectedBehavior.FileDescription> fileDescriptions) -> {
           // ...
       }));
        
   Maybe drop the "On"-prefix too, since almost all current behaviors don't 
have it - are you doing Android development? ;)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to