DarkDragoon2002 / pe

0 stars 0 forks source link

Modules with the same University not grouped together #5

Open DarkDragoon2002 opened 6 days ago

DarkDragoon2002 commented 6 days ago

Modules with the same university should be grouped together such that the name of the university only appears once and the module codes are listed out after that. For more popular mods this will cause clutter making it much harder to read and scroll through. Thanks!

Steps to replicate: filter ee313C

image.png

nus-se-script commented 3 days ago

Team's Response

Thank you for the feedback and suggestions! Currently, the filter command does fulfil its function to show all valid course mappings, and we agree that it would be a good addition to our next iteration to reduce clutter.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: The issue is valid and should not be classified as response.NotInScope because it directly impacts the usability of a core feature, the filter command, which is essential for users to retrieve and interpret results efficiently. While the command technically works, the repeated listing of university names creates unnecessary clutter, making the results harder to read and navigate, particularly for modules with multiple mappings. This is a usability flaw that aligns with the scope of improving user experience in the current version.

The dev team themselves acknowledged that grouping results "would be a good addition to reduce clutter," demonstrating that the current design is suboptimal and addressing it is valid. Furthermore, the fix is straightforward, requiring only a minor adjustment to group results by university, which significantly enhances readability with minimal effort:

image.png

Given its relevance to the feature’s usability and the low effort required to implement the fix, the issue does not meet the criteria for response.NotInScope and should be accepted as valid.