[ http://jira.magnolia.info/browse/MAGNOLIA-1361?page=comments#action_13438 ] Chris Miner commented on MAGNOLIA-1361: ---------------------------------------
Hopefully these changes suffice for http://jira.magnolia.info/browse/MAGNOLIA-1119 as well. No problems with localization, the alert panel goes away, multiple invalid fields are shown at once. I think this is probably more like one might expect from the ability to make a dialog field required. Would be nice to show a per field error message, and extend the model to include min and max field lengths, and finally have it all run on the client side. > validation check for missing required field stops after first field > ------------------------------------------------------------------- > > Key: MAGNOLIA-1361 > URL: http://jira.magnolia.info/browse/MAGNOLIA-1361 > Project: Magnolia > Issue Type: Bug > Reporter: Chris Miner > Assigned To: Boris Kraft > Attachments: CssConstants.patch, DialogBox.java.patch, > DialogControlImpl.java.patch, dialogs.css.patch > > > If you mark more than one field in a dialog as required and enter empty > values for all fields, only the first field is caught and reported as a > problem to the user. It would be better to hightlight all invalid fields. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.magnolia.info/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira ---------------------------------------------------------------- for list details see http://www.magnolia.info/en/magnolia/developer.html ----------------------------------------------------------------
