-
Notifications
You must be signed in to change notification settings - Fork 352
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
Moves help text in horiz. forms under the control #454
Conversation
Here's an example of a CHANGELOG.md entry (place it immediately under the * [#454](https://github.com/bootstrap-ruby/bootstrap_form/pull/454): Moves help text in horiz. forms under the control - [@tekkla](https://github.com/tekkla). Generated by 🚫 Danger |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch, and nice first contribution! Thanks very much for this PR. It looks good to me. I hope we will be merging this in the next few days. @mattbrictson could you please take a quick look at this?
Looks great, thanks! 🙌 I'll rebase and merge this once #455 is in. |
Updates test to reflect the changes made in form_group method
When using horizontal forms the help text is placed inside the created row but as sibling of the cols for label and control. This codechange moves the help text inside the col of the control so it gets aligned with/under the control.
Travis complains about this.
Do I have to change something or is it a problem on your side? |
This has been merged and the master build is passing, so there is nothing more we need to do. We're all good. Thanks for checking! |
* Updates help messages for horizontal forms test Updates test to reflect the changes made in form_group method * Moves help text in horiz. forms under the control When using horizontal forms the help text is placed inside the created row but as sibling of the cols for label and control. This codechange moves the help text inside the col of the control so it gets aligned with/under the control.
* Updates help messages for horizontal forms test Updates test to reflect the changes made in form_group method * Moves help text in horiz. forms under the control When using horizontal forms the help text is placed inside the created row but as sibling of the cols for label and control. This codechange moves the help text inside the col of the control so it gets aligned with/under the control.
When using horizontal forms the help text is placed inside the created row but as sibling of the cols for label and control. This codechange moves the help text inside the col of the control so it gets aligned with/under the control.
Before:
![from](https://user-images.githubusercontent.com/578661/37559878-e27c8c0a-2a2d-11e8-8eb6-f035dbafa297.png)
After:
![to](https://user-images.githubusercontent.com/578661/37559879-eaefff7a-2a2d-11e8-81a5-cc435ff91cf2.png)