Sounds good. In fact on second thought I don't understand why you don't
like "Value" and do like "Param." They're all LocParams, what's going on
here is that this LocParam provides snippets that handle the Loc value.


On Sun, May 12, 2013 at 6:31 AM, Antonio Salazar Cardozo <
notifications@github.com> wrote:

> In web/webkit/src/main/scala/net/liftweb/sitemap/Loc.scala:
>
> >     */
> > -  class Snippet(val name: String, _func: => NodeSeq => NodeSeq) extends AnyLocParam {
> > +  trait ValueSnippets[A] extends LocParam[A] {
>
> Hm. I think for a larger change like that maybe a separate pull request to
> lift_30_dev? I'd also be interested in hearing which ones, but we can see
> that in the PR or in a post to the list, either way. I do think it may be a
> bigger change than we might should be making to the 2.x series, though.
>
> Anyway, if this isn't so much a change of an existing thing, I say +1 on
> the PR and let's deal with naming separately, what do you think?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/lift/framework/pull/1410/files#r4185222>
> .
>