search

nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Sorting Contacts is not Reversible #1196

Open nus-se-script opened 4 months ago

nus-se-script commented 4 months ago

Sorting contacts by name is not reversible. While it may make sense to sort the names of contacts alphabetically, users may want to revert to the previous sorted order to access contacts based on the date they were added into the contact list.

[original: nus-cs2103-AY2324S2/pe-interim#1519] [original labels: severity.Low type.FeatureFlaw]

bennyLCK commented 4 months ago

Team's Response

With reference to the description about the sort persons feature in the UG:

image.png

As well as the DG:

image.png

The proposed implementation was to make "Sorting contacts by name reversible" because "users may want to revert to the previous sorted order to access contacts based on the date they were added into the contact list".

Firstly, it was mentioned clearly in the UG that sorting persons by name was permanent (UG) and was implemented like so after some considerations on the space overhead it would occur otherwise (DG).

Furthermore, since there is already a proposed implementation of the undo/redo features in the DG:

image.png

The reversal of any commands can be made after the proposed features are implemented, which also addresses this suggestion in question.

Lastly, if the user wanted to "access contacts based on the date they were added into the contact list", I feel that a better way to do so is to simply implement time stamps attached to person or even article entries so that we can sort according to those timestamps to order those entries by recency. Thank you for pointing this out so that we can possibly include such features in our next iteration of the product!

Duplicate status (if any):

--