On 29 December 2014 at 13:18, Jeroen van der Wal <jer...@stromboli.it> wrote:
> Looks good to me too! Some comments: > > regexPatternReplacement: > Is this the input placeholder? In that case I would choose to add a > "placeholder" attribute on @PropertyLayout and @ParameterLayout [1] > > No, it's to provide a user-friendly message in the validation message. eg: rather than say "the input didn't match '.+@.+' the message can say "the input isn't a valid email address (This is Rob Matthew's original design for @RegEx. Looking at the current implementation I think it's slightly broken, but I think it's worth resurrecting/fixing). > regexPatternFlags: > this is obsolete if you add "(?i)" to the pattern. > > True, but the flags parameter to compile covers more than this, see: http://docs.oracle.com/javase/7/docs/api/java/util/regex/Pattern.html#compile(java.lang.String,%20int) Thx Dan > Cheers, > > Jeroen > > > [1] http://www.w3schools.com/tags/att_input_placeholder.asp > > On Mon, Dec 29, 2014 at 11:45 AM, Martin Grigorov <mgrigo...@apache.org> > wrote: > > > Hi, > > > > Looks good to me! > > > > Martin Grigorov > > Wicket Training and Consulting > > https://twitter.com/mtgrigorov > > > > On Mon, Dec 29, 2014 at 12:36 PM, Dan Haywood < > > d...@haywood-associates.co.uk> > > wrote: > > > > > Hi folks, > > > > > > have done some further tidy up on the new annotations for domain > > semantics, > > > ie @DomainObject, @Property, @Collection, @Action and @Parameter. > > > > > > Nothing yet implemented in terms of facet factories, but think I have > the > > > annotations themselves pretty much finalized [1]. > > > > > > Would appreciate a review from anyone interested, save me reworking > stuff > > > > > > Cheers > > > Dan > > > > > > [1] https://issues.apache.org/jira/browse/ISIS-970 > > > > > >
