Lalelulilulela / pe

0 stars 0 forks source link

Visuals for edit feature not well integrated to the explanation #16

Open Lalelulilulela opened 6 months ago

Lalelulilulela commented 6 months ago

image.png For the edit feature, the image does not really show how the contact has been edited. Annotations or before and after images can be showed instead to better explain the change.

nus-pe-script commented 6 months ago

Team's Response

Command box shows the editing of the persons, but we will make it clear in future.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: The image is redundant since it is not integrated well to the explanation and it hinders the readers in the sense that readers are slowed down trying to understand the image shown. I am not sure how the "Command box shows the editing of the persons" but maybe your team meant the success/ error message box? Even then, the message does not show how the details of the client has changed and without any annotations, the image is not very useful as the message box is not obvious to the readers. Hence, this is a valid bug which hinders the readers which should be Accepted.