-
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: Remove Icon and align close button on DatasetModal #14902
fix: Remove Icon and align close button on DatasetModal #14902
Conversation
b04f065
to
6109cfb
Compare
Codecov Report
@@ Coverage Diff @@
## master #14902 +/- ##
==========================================
+ Coverage 77.50% 77.54% +0.03%
==========================================
Files 963 963
Lines 49306 49304 -2
Branches 6226 6228 +2
==========================================
+ Hits 38216 38232 +16
+ Misses 10889 10873 -16
+ Partials 201 199 -2
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
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.
LGTM! Just a nit
{t('Add dataset')} | ||
</> | ||
} | ||
title={<>{t('Add dataset')}</>} |
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.
Is the fragment needed here? Can't we just pass a plain string for the title?
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.
oh you know @geido , they don't need to be a fragment. Just passed in a regular string. Thank you!
* aligned DatabaseModal header * revisions
* aligned DatabaseModal header * revisions
* aligned DatabaseModal header * revisions
SUMMARY
The Dataset Modal had a ! icon that didn't make sense and was causing the header to be misaligned. This removes that, which aligns everything.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
Before:
![image](https://user-images.githubusercontent.com/48933336/120014226-ce85f680-bfaf-11eb-9fab-b7ccec6f6500.png)
After:
![image](https://user-images.githubusercontent.com/48933336/120014239-d2b21400-bfaf-11eb-9135-ec6cb52b08c2.png)
TESTING INSTRUCTIONS
Go to Datasets
Add a dataset
ADDITIONAL INFORMATION