-
Notifications
You must be signed in to change notification settings - Fork 6
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
(feat): updated the functionality of the imaging order #12
Conversation
…info on the imaging panel
Thanks so much, @its-kios09. I am wondering whether we can make the report body showing the report and other details a bit bigger and centralized? I just googled and got the below sample. Do you think we can achieve something similar or close? |
Yes, let me design the report then i will share it with you |
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.
Kapture.2024-12-17.at.10.25.29.webm
Am getting this error on running this PR
packages/esm-imaging-orders-app/src/form/review-form/review-imaging-form.scss
Show resolved
Hide resolved
packages/esm-imaging-orders-app/src/form/review-form/review-imaging-form.scss
Show resolved
Hide resolved
ping @its-kios09 |
cc @its-kios09 see the conflicts |
Do we really need the background colors for order status and no instructions? Could we just adopt a UI that's not very shouting? Disclaimer: Am not a UI expert :) |
Sure @ojwanganto, Merging this in |
Requirements
Summary
Screenshots
None.
Related Issue
None.
Other
None.