Skip to content

adding bootstrap class support #486

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

Merged
merged 1 commit into from
Sep 17, 2018

Conversation

averagejoe77
Copy link

Modified the scss in the formGroup.vue file to target elements that do not have a class that starts with 'col-' and set their width to 100%. Removed 'width:100%' from the form-group selector.

  • Please check if the PR fulfills these requirements
  • The commit message follows our guidelines
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)
  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
  • What is the current behavior? (You can also link to an open issue here)
  • What is the new behavior (if this is a feature change)?
  • Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
  • Other information:

Modified the scss in the formGroup.vue file to target elements that do not have a class that starts with 'col-' and set their width to 100%. Removed 'width:100%' from the form-group selector.
@lionel-bijaoui
Copy link
Member

Hello @averagejoe77 , thank you for your PR !

@lionel-bijaoui lionel-bijaoui merged commit 43cac83 into vue-generators:master Sep 17, 2018
@lionel-bijaoui lionel-bijaoui mentioned this pull request Sep 17, 2018
3 tasks
lionel-bijaoui pushed a commit to lionel-bijaoui/vue-form-generator that referenced this pull request Sep 24, 2018
@lionel-bijaoui lionel-bijaoui mentioned this pull request Sep 24, 2018
3 tasks
lionel-bijaoui pushed a commit to lionel-bijaoui/vue-form-generator that referenced this pull request Sep 25, 2018
lionel-bijaoui pushed a commit to lionel-bijaoui/vue-form-generator that referenced this pull request Sep 25, 2018
lionel-bijaoui pushed a commit to lionel-bijaoui/vue-form-generator that referenced this pull request Sep 28, 2018
lionel-bijaoui added a commit that referenced this pull request Sep 28, 2018
@averagejoe77
Copy link
Author

Hello @averagejoe77 , thank you for your PR !

I am sorry I thought you guys would have tested and run the build before this got merged. I only updated the .vue file but did not run the build.

The latest branch (2.3.0) with this PR merged in does not contain the actual change in the css file.

Do you want me to create a new PR with the actual css files getting updated?

@zoul0813
Copy link
Member

@averagejoe77 Travis CI should have handled that - looks like something went wrong.

@lionel-bijaoui have any thoughts?

@lionel-bijaoui
Copy link
Member

@zoul0813 See #472 (comment)

@zoul0813
Copy link
Member

zoul0813 commented Oct 2, 2018

Yep - I’ll correct that with 2.3.1, sorry about that.

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.

3 participants