search

jonjamz / blaze-forms

Dead easy reactive forms with validation (Meteor).
https://atmospherejs.com/templates/forms
MIT License
113 stars 11 forks source link

Example templates should not be included in package.js by default #49

Open vladshcherbin opened 9 years ago

vladshcherbin commented 9 years ago

Correct me if I am wrong, now the form templates come bundled with the forms package. If someone would create custom templates - then what is the point of having default templates for him ?

Maybe it is a good idea to have a skeleton and to move the templates to a different package ?

jonjamz commented 9 years ago

The templates in the current package are just simple examples to help people get started on their own custom components.

I don't think they should be sourced by default and shipped to the client, but I do think they should exist as part of the core package for convenience.

I plan on moving them into an "examples" folder and will not source them in package.js.

vladshcherbin commented 9 years ago

I wrote about the package because there may be some lazy or people, who are satisfied with the default ones, so that they can add 2 packages and are ready to go.

sure, examples would be perfect as well

keithn commented 9 years ago

perhaps the "example" templates should be in their own package? that way you could add example plain, bootstrap, ionic, or whatever you want to demo