-
Notifications
You must be signed in to change notification settings - Fork 248
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
(refactor) O3-2709: Update placeholder camera icon used in registration form #1601
Conversation
d7426c5
to
4d75f66
Compare
packages/esm-patient-attachments-app/src/camera-media-uploader/capture-photo.scss
Outdated
Show resolved
Hide resolved
…n patient registration
4d75f66
to
92a2ec3
Compare
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.
Thanks, @psworld!
Why didn't you just use the React component like we do with most of our icons? It's something like
(but please refer to an actual usage in the codebase for the particulars). |
…on form (openmrs#1601) * (refactor): O3-2709 used Carbon Library camera icon for placeholder in patient registration * Fix indentation * Fix Camera icon import --------- Co-authored-by: Ps <[email protected]> Co-authored-by: Dennis Kigen <[email protected]>
I have made the changes and used Could you please guide on how should I push changes ? as this branch is already merged
|
Please create a new PR , @psworld. Thanks for catching this, @brandones. |
Requirements
Summary
In #1580 the icon that is used for image placeholder icon did not conform to the O3- style guidelines.
I changed the placeholder icon to use
Camera Icon
available at https://carbondesignsystem.com/guidelines/icons/library/ under Technology section and titled CameraScreenshots
Before
After
Related Issue
https://openmrs.atlassian.net/browse/O3-2709