-
Notifications
You must be signed in to change notification settings - Fork 529
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
Icon overlapping with text in text-box of Doctor's Notes. #7495 #7496
Icon overlapping with text in text-box of Doctor's Notes. #7495 #7496
Conversation
@Sulochan-khadka is attempting to deploy a commit to the Open Healthcare Network Team on Vercel. A member of the Team first needs to authorize it. |
✅ Deploy Preview for care-egov-staging ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
👋 Hi, @Sulochan-khadka, This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there. |
Hey @Sulochan-khadka |
Yeah sure... I will do... |
Hi, This pr has been automatically marked as stale because it has not had any recent activity. It will be automatically closed if no further activity occurs for 7 more days. Thank you for your contributions. |
❌ Deploy Preview for care-net failed.
|
@rithviknishad have solved merge conflicts... Kindly have a look. |
@Sulochan-khadka the UI doesn't look good, can you try to implement Ui similar to slack |
@nihal467 sure... But then I need to make changes to another issue that was solving that of patient's notes. Shall I go ahead and make your suggested changes on both? |
Should I add a rich text editor ? something like react-quill? Or you just want to correct the alignment . I think adding something like react quill will obviously be a firm step towards improving the doctor's notes functionality for example, adding markdown feature and file upload functionality. Let me know your thoughts on this. |
@rithviknishad can you comment on this issue |
@Sulochan-khadka I'd suggest creating a separate issue for that as that would be a larger change and the related issue simply needs to solve the icon overlapping issue and have a good UI. |
LGTM |
@Sulochan-khadka Your efforts have helped advance digital healthcare and TeleICU systems. 🚀 Thank you for taking the time out to make CARE better. We hope you continue to innovate and contribute; your impact is immense! 🙌 |
Proposed Changes
@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers
Merge Checklist