-
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
refactor: icon to icons for listviewcomponent #15408
Conversation
Codecov Report
@@ Coverage Diff @@
## master #15408 +/- ##
==========================================
- Coverage 77.23% 77.12% -0.11%
==========================================
Files 975 974 -1
Lines 50610 50817 +207
Branches 6212 6727 +515
==========================================
+ Hits 39090 39195 +105
- Misses 11313 11407 +94
- Partials 207 215 +8
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
/testenv up |
@michael-s-molina Ephemeral environment creation is currently limited to committers. |
😢 @rusackas? |
/testenv up |
@rusackas Ephemeral environment spinning up at http://52.35.136.255:8080. Credentials are |
@pkdotson The alignment is a little bit up after the change. This problem is happening in more than one place when the new icon component is being used. Maybe we can find a global solution for this? |
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. Thanks for the alignment tweak.
Ephemeral environment shutdown and build artifacts deleted. |
* initial commit * fix dashboard cypress tests * fix more tests * fix more tests * fix listviews
* initial commit * fix dashboard cypress tests * fix more tests * fix more tests * fix listviews
* initial commit * fix dashboard cypress tests * fix more tests * fix more tests * fix listviews
SUMMARY
this pr refactors the listview icon and and cardview icon to the new icons component
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
before
![Screen Shot 2021-06-27 at 5 26 08 AM](https://user-images.githubusercontent.com/17326228/123545875-347caf80-d70f-11eb-83af-d127e3a76a8a.png)
after
![Screen Shot 2021-06-30 at 4 33 35 AM](https://user-images.githubusercontent.com/17326228/123953675-59199700-d95c-11eb-9d4b-0e325d790a8b.png)
TESTING INSTRUCTIONS
got to any of the listviews to view card and list icons
ADDITIONAL INFORMATION