Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[Form] Removed .mb-0 in errors. #27887

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
Jul 9, 2018

Conversation

vudaltsov
Copy link
Contributor

@vudaltsov vudaltsov commented Jul 7, 2018

Q A
Branch? 3.4
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets
License MIT
Doc PR

<span class="d-block"> will never have a margin on its own, so .mb-0 is needless.

@nicolas-grekas
Copy link
Member

lowest deps need a bump so that tests pass

@vudaltsov
Copy link
Contributor Author

Isn't it strange that we have templates in one library and abstract tests for these templates in another?

@vudaltsov
Copy link
Contributor Author

@xabbuh , now all tests pass except for deps=high.

@nicolas-grekas nicolas-grekas force-pushed the form-removed-errors-mb-0 branch from 44ce40a to 925dda1 Compare July 9, 2018 12:39
@nicolas-grekas
Copy link
Member

Thank you @vudaltsov.

@nicolas-grekas nicolas-grekas merged commit 925dda1 into symfony:3.4 Jul 9, 2018
nicolas-grekas added a commit that referenced this pull request Jul 9, 2018
This PR was merged into the 3.4 branch.

Discussion
----------

[Form] Removed .mb-0 in errors.

| Q             | A
| ------------- | ---
| Branch?       | 3.4
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets |
| License       | MIT
| Doc PR        |

`<span class="d-block">` will never have a margin on its own, so `.mb-0` is needless.

Commits
-------

925dda1 Removed .mb-0 in errors.
@vudaltsov vudaltsov deleted the form-removed-errors-mb-0 branch December 14, 2018 10:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants