-
Notifications
You must be signed in to change notification settings - Fork 372
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
Revamped Upload files buttons #3419
Conversation
✅ Deploy Preview for care-egov-staging ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
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.
- Use tailwind buttons
- Apply styles for the
browse
button
❌ Deploy Preview for care-net failed.
|
@Pranshu1902 is it good for testing |
Yes |
@Pranshu1902 test failed, check the button alignment in normal view |
@nihal467 |
in responsive view, the buttons are not having gap in between them |
@nihal467 |
in your PR buttons look bigger when compared to existing staging buttons, reduce the size of record, browse and upload buttons in to a way that, it is similar to all other buttons like discharge summary etc... |
@nihal467 |
@Pranshu1902 the icons and text seem really big, can you discuss it in EOD, regarding this |
when you compared button icon and text size with other buttons, it is big, can you discuss this in EOD, to have a better solution |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Kudos, SonarCloud Quality Gate passed!
|
Fixed in #3363 |
Fixes #3418
http://localhost:4000/sample/f01568f0-3b58-418a-96fc-89d228073462
Made buttons full width for mobile view
![image](https://user-images.githubusercontent.com/70687348/185042849-c9dc52ad-cc2c-474d-8897-c946eeb0c341.png)