When clicking on the delete image button in the profile page, it uses the default Windows alert. This replaces it with Material UI using the ConfirmationModal component.
It also adds a notification upon successful uploading of a profile image and not only when there's an error. This makes it more consistent.
Screenshots (if applicable):
Additional Context (optional):
Add any other context about the PR here.
Future Steps/PRs Needed to Finish This Work (optional):
Anything I may have overlooked, ways to consolidate code, etc.
Issues needing discussion/feedback (optional):
1. Adding the notification status for successful upload isn't strictly necessary, but I do think it helps maintain consistency and is useful if, for instance, the user has already moved away from the profile page.
This PR:
Resolves #463
When clicking on the delete image button in the profile page, it uses the default Windows alert. This replaces it with Material UI using the
ConfirmationModal
component.It also adds a notification upon successful uploading of a profile image and not only when there's an error. This makes it more consistent.
Screenshots (if applicable):
Additional Context (optional):
Add any other context about the PR here.
Future Steps/PRs Needed to Finish This Work (optional):
Anything I may have overlooked, ways to consolidate code, etc.
Issues needing discussion/feedback (optional):
1. Adding the notification status for successful upload isn't strictly necessary, but I do think it helps maintain consistency and is useful if, for instance, the user has already moved away from the profile page.