jblotus/Try-Handlebars.js — Issue #4: "context (JSON object)" is misleading
Author: skierpage
Created: 2012-02-26 02:11:02+00:00
State: open

In all the examples, the context is *not* a JSON object. The context data for all your examples fails http://jsonlint.com/ because the keys are not double-quoted.

It works because you eval the context, but if someone tries to use the same data with something like JSON.parse() it will fail.

The "fix" is to change the label on line 285 from "Context (json object)" to "Context (JavaScript literal or JSON)"; "Context (JavaScript literal object or JSON)" is even more correct, but wraps.

Thanks for all you do!

```diff
--- index.html.original 2012-02-25 17:56:19.000000000 -0800
+++ index.html 2012-02-25 18:02:04.000000000 -0800
@@ -282,7 +282,7 @@

- Context (json object)
+ Context (JavaScript literal or JSON)
{{context}}

```