-
Notifications
You must be signed in to change notification settings - Fork 561
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
Linked skills- to show on patient dashboard for "treating doctor" #7810
Comments
|
May you present some UI idea to that... I think since the skills may be large thus, having a dropdown with the heading as Skills would be better. |
@aparnacoronasafe Let me know if I can help you ... |
Since this is P1, can @rithviknishad @nihal467 comment on this and let me know if I can go ahead and raise a PR? |
@Sulochan-khadka The UI shown above is not suitable. @aparnacoronasafe has already mentioned a good solution on how to show the skills in the UI in the issue comment itself. Just re-explaining... Show a comma-separated text of skills with a limit of maybe 3. It could be a text in gray color, underlined and cursor pointer. If there are more than 3 skills, show two skills and rest can be shown in a tooltip. Eg:
Feel free to create a PR since you've already achieved the functionality 😄 |
@Sulochan-khadka please follow the suggestion from @rithviknishad above. We do NOT need a button to show linked skills. It must appear on the page itself. So simply adding a text with skills in (..) should suffice. When there are multiple skills show them one after the other separated by a comma |
* fixes: Linked skills- to show on patient dashboard for "treating doctor" #7810 * Update src/Components/Patient/PatientInfoCard.tsx Co-authored-by: Rithvik Nishad <rithvikn2001@gmail.com> * updated with implementing suggestions * made changes * refactor based on suggestions * show tooltip only if overflow * Update src/Components/Patient/PatientInfoCard.tsx --------- Co-authored-by: Rithvik Nishad <rithvikn2001@gmail.com> Co-authored-by: rithviknishad <mail@rithviknishad.dev>
* fixes: Linked skills- to show on patient dashboard for "treating doctor" #7810 * Update src/Components/Patient/PatientInfoCard.tsx Co-authored-by: Rithvik Nishad <rithvikn2001@gmail.com> * updated with implementing suggestions * made changes * refactor based on suggestions * show tooltip only if overflow * Update src/Components/Patient/PatientInfoCard.tsx * fixes:Maintain aspect ratio for Bed Icon in Patient Dashboard #7971
https://care.coronasafe.in/facility/0c95c7f0-e1d2-4aff-83fa-933cef60d3a8/patient/7e8fc20f-02da-47f7-8c33-f2d8c7e52dd8/consultation/edfe0bd6-0f50-4a4c-8cd0-2d4ad0e9cea1
The patient dashboard shows name of the "treating doctor". After the name of the doctor, show the skills linked to the doctor within "(_)"
The text was updated successfully, but these errors were encountered: