-
Notifications
You must be signed in to change notification settings - Fork 348
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
remove MUI circularProgress & grid #5047
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
❌ Deploy Preview for care-egov-staging failed.
|
@ayushjnv1 Thank you for the PR Could you please, merge the upstream to resolve the conflicts & update the PR description and link to the Issues you are solving |
#4930 this is the issue that I fixed |
@ayushjnv1 @rithviknishad @gigincg
|
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.
- The grid classes are unnecessary.
- Make button
w-full
Co-authored-by: Rithvik Nishad <rithvikn2001@gmail.com>
Co-authored-by: Rithvik Nishad <rithvikn2001@gmail.com>
Kudos, SonarCloud Quality Gate passed!
|
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.
@ayushjnv1 LGTM.
@nihal467 the language selector issue is a different issue not related to this.
What happens is, if the user has never selected a language in care, it'll be blank and English is not highlighted. Because only when a user selects a language, the preference is stored in the local storage.
In your staging, English is highlighted because you've selected English at some point in time.
This issue is easily noticeable in all deploy previews.
In my fresh staging (see image) I've never selected a language, so it's not highlighted. Will create an issue for this.
LGTM |
Proposed Changes
@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers
Merge Checklist