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

Revisit stacked and inline form layout options #21581

Closed
mdo opened this issue Jan 6, 2017 · 4 comments
Closed

Revisit stacked and inline form layout options #21581

mdo opened this issue Jan 6, 2017 · 4 comments

Comments

@mdo
Copy link
Member

mdo commented Jan 6, 2017

Folks often want to mix the standard form group and inline form options in the same form. I'll build some example layouts out and see what we can do to help.

@mdo mdo added this to the v4.0.0-beta milestone Jan 6, 2017
@dmxt
Copy link

dmxt commented Jan 7, 2017

(Reference post) Based on https://news.ycombinator.com/item?id=13339739

@cr101
Copy link

cr101 commented Jan 7, 2017

This looks like a great example http://forms.viewflow.io/demo/bank/ referenced in the Hacker News article above

@srvance
Copy link

srvance commented Feb 16, 2017

While I like the ability to control the spacing for inline forms, the defaults are distinctly lacking. When you revisit this, can you make inline layout have sensible, aesthetic defaults that can be overridden when desired?

@mdo
Copy link
Member Author

mdo commented May 3, 2017

I've been playing around with ideas and have been struggling to find anything more helpful and flexible than the grid system provided by Bootstrap. I still haven't gotten to the default spacing and alignment options in the inline forms, but that's a straightforward enough fix I think.

Thoughts from folks on something like this for the sizing of multiple inputs per row though? https://jsbin.com/zaloye/edit?html,output

@mdo mdo added this to In progress in v4 Beta 1 May 26, 2017
@mdo mdo moved this from In progress to Safe to merge in v4 Beta 1 May 27, 2017
@mdo mdo added the has-pr label Jun 15, 2017
@mdo mdo moved this from Safe to merge to Shipped in v4 Beta 1 Jun 15, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
v4 Beta 1
Shipped
Development

No branches or pull requests

4 participants