se701team3 / Forgettable

University of Auckland SOFTENG 701 Team 3 Assignment 1
MIT License
12 stars 44 forks source link

Change the delete person modal #333

Closed BraveNewWord closed 2 years ago

BraveNewWord commented 2 years ago

Description

Clicking delete on the edit person page currently shows a modal to the user saying they are attempting to delete an encounter. It should be changed to show that they are deleting a person instead, and should show that deleting the person will also delete encounters with them if they were the only person there. Button style should be changed to highlight destructive action of deleting a person

Acceptance Criteria

Implementation details

Create or modify current delete person modal to show that the user that they are deleting a person Change style of buttons to highlight destructive action

Testing notes

New features behave correctly Old feature (delete encounter) behave correctly

EDIT: It was trivial to change the text, extending scope to styling the buttons, may extend further to buttons in other components with another issue