Change -= name =- to {{ name }}

[dbankier]

a limitation of alloy at the moment.
will raise JIRA and/or PR soon.

[FokkeZB]

From @jhaynie:

i think this might be the regular expression here FYI:
https://github.com/appcelerator/alloy/blob/323f0bfd034ccd99912390ae2e12924793d9c1fc/Alloy/commands/compile/styler.js#L19

In order for Alloy not to interfere with {{name}} I think the regexp on the above line should be:

/(?:[^\{]|^)\{([^:}]+)\}(?:[^\}]|$)/

[dbankier]

Thanks. There is another BINDING_REGEX in Ti.UI.Switch.

[rblalock]

I like {{}} or <% %>

[grantges]

+1 <% %>

[FokkeZB]

+1 {{ }} 😉

[netoramalho]

+1 {{ }}

[nicka]

+1 <% %>

[iotashan]

What about ColdFusion style? #name#

[TheBrousse]

{{ }} would make more sense in my opinion since it has been adopted a lot of Javascript frameworks.

[jhaynie]

+1 {{ }}

[jhaynie]

@FokkeZB that looks about right to me. if you can submit a PR to alloy we can fix it ASAP.

[dbankier]

I prefer {{ }} too.
<% %> can be implemented now.
# # I don't really like. It needs a a clear opening and closing since the expression can be long.

I will likely make it {{ }} by default but also make the syntax configurable (much like _ templates).

[rblalock]

Sounds good to me. Ive been using handlebars on a new site for Appc , getting pretty comfortable with it. You going to just allow use of handlebars completely? So people can write plugins to it? Just an idea.

[dbankier]

I haven't really used handlebars and not sure how/if it works outside the dom.
I'll have a look.

[rblalock]

It doesn't work off the dom. It's all string manipulation.

[dbankier]

OK done. It defaults to {{ }} now.
using nano.syntax(/\-\=(.+?)\=\-/gi); to use the old syntax, which you need to do till alloy is fixed.

A warning about using <% %> is that it won't work if you used it as text content in xml, e.g.

<Label> Hi <% name %> </Label>

You will get this:

[ERROR] Error parsing XML file.
[ERROR] element parse error: Error: invalid tagName:%
[ERROR] @#[line:8,col:21]
[ERROR] Alloy compiler failed

nano works are run time and the xml parser that runs at precompile doesn't like it.

Leaving this open until we don't need to use the nano.syntax command anymore.

[FokkeZB]

@dbankier you're doing a ticket+PR for Alloy?

[dbankier]

@FokkeZB, just implemented your change and tested it. It work well. Thanks.
Here is the PR: tidev/alloy#648

[dbankier]

Here is the JIRA ticket: https://jira.appcelerator.org/browse/TC-5179

[nicka]

Good job guys!

[grantges]

Nice job @FokkeZB ~

On Tue, Jan 6, 2015 at 2:35 AM, Nick den Engelsman <[email protected]

wrote:

Good job guys!

—
Reply to this email directly or view it on GitHub
#1 (comment).

[dbankier]

tidev/alloy#648 was merged.
We are good to go when the change is part of an official Alloy release.
I will update the README.md and close the issue then.

[jhaynie]

sweet.