Skip to content
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

Html helper methods #4

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Html helper methods #4

wants to merge 2 commits into from

Conversation

sparkymat
Copy link
Owner

No description provided.

@kirillrdy
Copy link
Collaborator

Sorry it took me that long to have a look at this.
Basically this looks very good and useful, the only think i would rename package to something else.
perhaps "form" mainly i was thinking of it becoming more or less equivalent of formtastic or simple_form.

That being said the name is still open for discussion

K

@sparkymat
Copy link
Owner Author

I agree. Need a better name. I agree, something more like simple_form or formtastic.

goform? :)

@kirillrdy
Copy link
Collaborator

since package is nested under websl form sounds like a good name unless you want it to be seperate propject

@sparkymat
Copy link
Owner Author

formalize right? . i'd prefer keeping it in webdsl for now

On Mon, Mar 2, 2015 at 9:55 AM kirillrdy notifications@github.com wrote:

since package is nested under websl form sounds like a good name unless
you want it to be seperate propject


Reply to this email directly or view it on GitHub
#4 (comment).

@kirillrdy
Copy link
Collaborator

sure, why not

On Mon, Mar 2, 2015 at 11:13 AM sparkymat notifications@github.com wrote:

formalize right? . i'd prefer keeping it in webdsl for now

On Mon, Mar 2, 2015 at 9:55 AM kirillrdy notifications@github.com wrote:

since package is nested under websl form sounds like a good name unless
you want it to be seperate propject

Reply to this email directly or view it on GitHub
#4 (comment).

Reply to this email directly or view it on GitHub
#4 (comment).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants