-
Notifications
You must be signed in to change notification settings - Fork 437
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
Added icons to Asset page and enhanced the UI #3117
Conversation
❌ Deploy Preview for care-net failed.
|
Hey, @kunatastic I have moved the SVG icons to a component so they can be reused. |
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
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.
If you found a solid variant of the monitor then replace the current monitor icon to match other icons.
@developedBySJ I have fixed all the type issues as suggested I have added a solid variant icon |
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
tested |
@nihal467 @patelaryan7751 is this pr complete? Aren't you making this icons as pictures, right? |
@patelaryan7751 @khavinshankar I think, rather than an icon, we have to make it a picture, that is what we have discussed in the EOD call as far as I remember. Aryan, shall we discuss it in the EOD call ! |
79f7652
to
36e2e8a
Compare
✅ Deploy Preview for care-egov-staging ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Fixes #3116
Enhancement
Added icons to Asset page and enhanced the UI