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

Fixes #1910 - Entities Margin correction #1913

Merged
merged 1 commit into from
Oct 29, 2018
Merged

Conversation

zsarnett
Copy link
Contributor

@zsarnett zsarnett commented Oct 29, 2018

Div around Rows applies the margin correctly

#1910

Verified

This commit was signed with the committer’s verified signature.
tisonkun tison
@ghost ghost assigned zsarnett Oct 29, 2018
@ghost ghost added the in progress label Oct 29, 2018
@balloob
Copy link
Member

balloob commented Oct 29, 2018

Make your PR description Fixes #XXXX and it will auto-close the issue when we merge this PR.

Do you know why a div around it fixes it? It's because a div has a display: block, while custom elements by default are display: inline. inline elements can't have margin.

@zsarnett
Copy link
Contributor Author

Oh gotcha didnt realize that would auto close.

Gotcha. That makes sense,

@zsarnett zsarnett changed the title Adding div around each row to deal with spacing Fixes #1910 - Entities Margin correction Oct 29, 2018
@zsarnett zsarnett merged commit 6dac48e into dev Oct 29, 2018
@ghost ghost removed the in progress label Oct 29, 2018
@zsarnett zsarnett deleted the entities-padding-fix branch October 29, 2018 19:45
@balloob balloob added this to the 0.81.5 milestone Nov 2, 2018
@github-actions github-actions bot locked and limited conversation to collaborators Jul 7, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants