search

tastejs / todomvc-app-template

Template used for creating TodoMVC apps
http://todomvc.com
473 stars 405 forks source link

This depends on the learn.json from the main app #10

Open ashnur opened 5 years ago

ashnur commented 5 years ago

Since https://github.com/tastejs/todomvc-common/blob/master/base.js#L248

If you want to inject some random content into every application you showcase, there are better ways than this.

sindresorhus commented 5 years ago

there are better ways than this.

Ok, what are those better ways?

ashnur commented 5 years ago

Depends, maybe you could move it above the comment, then it wouldn't be compulsory and people could delete it. Or you could hardcode with the content so people see what they need to download and run. Or just create a page where you embed the application and you control the context. Or any number of other solutions where you don't provide a "template" with runtime errors out of the box.