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

Restore <body> bottom padding #4252

Merged
merged 3 commits into from
Oct 17, 2019
Merged

Restore <body> bottom padding #4252

merged 3 commits into from
Oct 17, 2019

Conversation

kravets-levko
Copy link
Collaborator

What type of PR is this? (check all applicable)

  • Bug Fix

Description

Restore <body> bottom padding accidentally removed in #3697

@kravets-levko kravets-levko changed the title Restore <body> bottom padding (WIP) Restore <body> bottom padding Oct 16, 2019
@kravets-levko
Copy link
Collaborator Author

Percy caught few more issues with this fix. Need to check once more

@ranbena
Copy link
Contributor

ranbena commented Oct 16, 2019

I sent you a PR with some fixes

* No bottom padding when body.fixed-layout

* Shave off the 15px from the dashboard edit bottom padding
@kravets-levko
Copy link
Collaborator Author

@ranbena Thanks a lot!

@kravets-levko kravets-levko changed the title (WIP) Restore <body> bottom padding Restore <body> bottom padding Oct 16, 2019
@kravets-levko kravets-levko requested a review from ranbena October 16, 2019 20:46
@kravets-levko kravets-levko merged commit e0a2705 into master Oct 17, 2019
@kravets-levko kravets-levko deleted the fix/body-bottom-padding branch October 17, 2019 10:22
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.

2 participants