search

Silvernitro / pe

0 stars 0 forks source link

Cannot sort by type first, followed by completion status #2

Open Silvernitro opened 3 years ago

Silvernitro commented 3 years ago

Expected: If a user sorts projects by a type (e.g. deadline), he may realise that he has multiple projects due on the same date. He might then want to sort by completition status so he can ignore the completed ones. He should be able to do "sort type/deadline" followed by "sort c/" in thsis case. But, now he must do "sort c/" before "sort type/deadline"

Actual: After "sort c/" is executed, the sort type is defaulted back to alphabetical.

Steps to reproduce:

  1. Type in "sort type/deadline"
  2. Type in "sort c/"
  3. This is displayed:

Screenshot 2020-11-13 at 4.38.10 PM.png

nus-se-bot commented 3 years ago

Team's Response

This is the intended behaviour as if the sort order and sort type is not stated in the command, they are assumed to be default sort type and order (alphabetical, ascending), as stated in our UG. We want this behaviour as we want to provide convenience for users by allowing them to revert back to the default sort order with just a simple command without having to specify the sort type and order.

image.png

If you would like to toggle the completion status order while remaining on the deadline sort type view, you can run sort type/deadline order/asc c/.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]

:question: Issue severity

Team chose [severity.Low] Originally [severity.Medium]

Reason for disagreement: [replace this with your explanation]