search

satyrnjs / satyrn

A Markdown-based JavaScript Sandbox
Apache License 2.0
3 stars 2 forks source link

Keyword let issue in javascript editor #3

Open ghost opened 5 years ago

ghost commented 5 years ago

@wip-abramson commented on Jul 15, 2019, 1:22 PM UTC:

Using let in the js editor causes: SyntaxError: Identifier 'p5' has already been declared when you run the editor block more than once. Resetting the editor does not solve the issue.

Not using the let keyword at all seems the best approach, but not sure that is what we want.

This issue was moved by jandrieu from WebOfTrustInfo/satyrn#56.

ghost commented 5 years ago

@jandrieu commented on Jul 15, 2019, 4:49 PM UTC:

Add some explanation in the about guide.

jandrieu commented 5 years ago

This is the expected behavior, IF you understand what's going on under the hood.

Let's document in the wiki as run-once functionality and that you can reload the entire page to reset if necessary.

It's probably too hard to dynamically respond to "let" commands even to provide a warning.