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.

Sign up for GitHub

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

Jump to bottom

[Form] fields class in fields class add extra margin-bottom #1202

Closed
dutrieux opened this issue Dec 1, 2019 · 1 comment
Closed

[Form] fields class in fields class add extra margin-bottom #1202

dutrieux opened this issue Dec 1, 2019 · 1 comment
Labels
lang/css Anything involving CSS type/bug Any issue which is a bug or PR which fixes a bug
Milestone
2.8.3

Comments

@dutrieux
Copy link
Contributor

dutrieux commented Dec 1, 2019 •
edited
Loading

Bug Report

Where there is a fields class in a fields class, a extra margin-bottom is add.

Testcase

https://jsfiddle.net/dutrieux/sbyt46a9/

Screenshot

image
image

Version

2.8.2
@dutrieux dutrieux added the state/awaiting-investigation Anything which needs more investigation label Dec 1, 2019
@dutrieux dutrieux changed the title [Form] fields in fields add extra margin-bottom [Form] field in field add extra margin-bottom Dec 1, 2019
@dutrieux dutrieux changed the title [Form] field in field add extra margin-bottom [Form] fields class in fields class add extra margin-bottom Dec 1, 2019
@lubber-de lubber-de mentioned this issue Dec 6, 2019
Merged
@lubber-de
Copy link
Member

@dutrieux Thanks. I just adopted your fix by #1208 😃

@lubber-de lubber-de added state/has-pr An issue which has a related PR open type/bug Any issue which is a bug or PR which fixes a bug lang/css Anything involving CSS and removed state/awaiting-investigation Anything which needs more investigation labels Dec 6, 2019
@lubber-de lubber-de added this to the 2.8.3 milestone Dec 6, 2019
@lubber-de lubber-de added tag/next-release/nightly Any issue which has a corresponding PR which has been merged and is available in the nightly build and removed state/has-pr An issue which has a related PR open labels Dec 22, 2019
@y0hami y0hami closed this as completed in d3c94b0 Dec 23, 2019
@y0hami y0hami removed the tag/next-release/nightly Any issue which has a corresponding PR which has been merged and is available in the nightly build label Dec 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
lang/css Anything involving CSS type/bug Any issue which is a bug or PR which fixes a bug
Projects
None yet
Milestone
2.8.3
Development

No branches or pull requests
3 participants
@dutrieux @y0hami @lubber-de