-
Notifications
You must be signed in to change notification settings - Fork 13.2k
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
fix: DatabaseConnection Modal Margin Bottom #14849
Conversation
🏷️ 2021.21 |
Codecov Report
@@ Coverage Diff @@
## master #14849 +/- ##
=======================================
Coverage 77.61% 77.61%
=======================================
Files 963 963
Lines 49246 49246
Branches 6197 6197
=======================================
Hits 38224 38224
Misses 10821 10821
Partials 201 201
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
@@ -81,7 +81,6 @@ export const antDModalNoPaddingStyles = css` | |||
.ant-modal-body { | |||
padding-left: 0; | |||
padding-right: 0; | |||
margin-bottom: 110px; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we still want this margin just for the sqlalchemy view.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just checking in, we are keeping this change or should i go back to 110px?
(cherry picked from commit 877201f)
SUMMARY
The margin bottom for the modal form was too large. This PR fixers it.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
Before:
![Screen Shot 2021-05-26 at 1 49 25 PM](https://user-images.githubusercontent.com/48933336/119707514-3c53e600-be29-11eb-9be7-e7f8dee707ab.png)
![Screen Shot 2021-05-26 at 1 48 22 PM](https://user-images.githubusercontent.com/48933336/119707528-41b13080-be29-11eb-8890-9cc5341703d3.png)
After:
TESTING INSTRUCTIONS
Go into Database, create a new database.
ADDITIONAL INFORMATION