-
Notifications
You must be signed in to change notification settings - Fork 11
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improvement in the way the package is presented #14
Comments
I think the best way to emphasize why people should use this package is to create a much better set of examples and, as you say, improve the docs. For example, check out this screenshot from Virgin America's flight ordering process: While I haven't built a replica of this using Why? Because And, you can add your own set of helpers onto any element or form block template and they'll stack with the ones this package provides--just like any Blaze template can have more than one call to A note on the real-time pricing in that screenshot: I think I'd have to make the form data context reactive and easily accessible through a helper, but it's possible that providing real-time pricing information based on a user's selections is easily achievable, even now. |
Thanks for thinking about this, and I look forward to seeing your ideas! |
Here is the draft: |
I think this is a great starting point for new docs. Very clear. I really like your examples. 👍 |
Thanks. |
An additional note about examples and package marketing: I am quite sure you alienate half of potential users if you have sample code in CoffeeScript. See this typical discussion. I think package authors who want to promote CoffeeScript without alienating anyone should go double-language. |
Sure--but my examples are all JavaScript. Unless you're talking about the live example, which indeed is CoffeeScript. I should fix that... I have no problem going double-language, it just takes up a lot of space. Perhaps the new docs could also exist on a separate website where I could use the |
Well, it was just long-term consideration and a precaution for new coming examples. I don't think it is urgent to fix the live example, and maybe it could stay as it is if more examples are provided. |
If I could add my two cents here, as a compete novice...the new draft of the readme would help a lot. I'm learning how to write code as I build an app and have been relying heavily on SimpleSchema and Autoform to do so. Everything I have read elsewhere sounds like template:form is the easier way to go, but the current docs are pretty daunting (especially for my relatively simple use case). This new doc would make it much easier for me to get on board. Is the draft accurate for me to rely on? |
Somewhat accurate, although some of the naming is off (for example, the package name is still I'm working on a pretty big update to the package this weekend so I'll adjust the docs too. |
Late to this discussion, but I here are my two cents:
Thanks! |
Now that this latest update is out, I'll spend some more time fixing up the docs and working on examples. |
Check out the in-progress docs for |
Wow, that looks great! |
I find the new documentation excellent. Did not yet have time to look into all details, but it appears now really well structured and straight forward. Very good job! |
Hey @jonjamz any update on when documentation will be officially ready? I think the Meteor community would greatly benefit from it! I find this package to be a breath of fresh air compared to the overly verbose Autoforms package. Let us know if you need any help :) |
Basically all that's left before I put up the new docs is to make sure the API methods have the right names. I also need to test out the examples, particularly some of the more complex ones. |
Brilliant news! :) I just tested AdjustableNumberOfInputs. When changing a field value it seems to output this error: The field that it was using had a schema like this:
|
👍 |
I have the feeling this package would get more users if the way it is presented was improved.
First, I think the name of the package and the name of the github repo should match. EDIT: and the name of the API entry point (currently ReactiveForms).
Second, I think the package should be renamed from 'templates:forms' to something more unique, like 'jonjamz:forms' or 'circus:forms' (use case: "I use the form package from aldeed, what form package do you use? I use xxxx").
Third, I think the documentation should emphasize more why people should use this package and how easy it is. I will post a draft here.
The text was updated successfully, but these errors were encountered: