[ https://issues.jboss.org/browse/DROOLS-3077?page=com.atlassian.jira.plugi... ] Michael Anstis commented on DROOLS-3077: ---------------------------------------- [~karreiro] [~uxdlc] On the whole I think it looks good. However; I think with {{Applicant.Age}} set as the input expression and the Data Type {{Age}} the _existing_ constraint of {{[18..24]}} could be overridden *without* needing to select a different or define a new Data Type. Furthermore I wonder whether the User should _set_ the constraint rather than _add_ one (as they can only have one constraint). Also, at the moment, if I change the Name or Data Type the change can be undone. However (currently) changes to the Constraint definition cannot be undone. So we have a mis-match of functionality. However we could change this so that clicking OK performs the actions necessary to change the Name, Data-type and Constraint in such a way as they can be undone. Changes made on the Data Types cannot be undone. Finally, I wonder if adding the "OK", "Cancel" buttons to the {{InputClause}} popover should be replicated to all popovers; otherwise some popovers will continue to apply changes when (a) the "Name" {{TextBox}} looses focus, (b) when a selection is made in the "Data Type" drop-down, (c) when the "OK" button is clicked... just wondering. -- This message was sent by Atlassian Jira (v7.12.1#712002)