-
Notifications
You must be signed in to change notification settings - Fork 220
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
Patient attachments app improvement #751
Conversation
…ng the uploaded files to change name and description
@vasharma05 looks reasonable so far 👍 |
…he file completed
…ction in the camera-upload component
…nt-chart into patient-attachments-app-improvement
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.
A bunch of small css cleanup. Otherwise looks good
packages/esm-patient-attachments-app/src/attachments/attachments-grid-overview.component.tsx
Outdated
Show resolved
Hide resolved
packages/esm-patient-attachments-app/src/attachments/attachments-grid-overview.scss
Outdated
Show resolved
Hide resolved
packages/esm-patient-attachments-app/src/attachments/attachments-grid-overview.scss
Outdated
Show resolved
Hide resolved
} | ||
|
||
.closePreviewButton svg path { | ||
fill: $ui-01 !important; |
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.
no important
} | ||
|
||
.closePreviewButton:hover svg path { | ||
fill: $ui-05 !important; |
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.
no important
packages/esm-patient-attachments-app/src/attachments/camera-upload.scss
Outdated
Show resolved
Hide resolved
display: grid; | ||
grid-template-columns: 1fr 1fr; | ||
button{ | ||
max-width: unset !important; |
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.
no important
packages/esm-patient-attachments-app/src/attachments/camera-upload.scss
Outdated
Show resolved
Hide resolved
packages/esm-patient-attachments-app/src/attachments/camera-upload.scss
Outdated
Show resolved
Hide resolved
grid-template-columns: 1fr 1fr; | ||
|
||
button { | ||
max-width: unset !important; |
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.
no important
…com/openmrs/openmrs-esm-patient-chart into patient-attachments-app-improvement
|
||
return ( | ||
<div className={styles.galleryContainer}> | ||
{attachments.map((attachment, indx) => { |
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.
5 letters for index
was too expensive? 😂
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.
Gotta save those byts
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.
😆
This reverts commit d0acf93.
Requirements
Summary
Improvement to the functionality and UI/ UX for patient attachments.
Screenshots
None yet
Related Issue
Other