search

nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

UI of GUI cannot be adjustable #3858

Open nus-se-script opened 2 days ago

nus-se-script commented 2 days ago

Steps to reproduce:

  1. Enter addlsn 2 l/tue 1400-1600 l/thu 1400-1600
  2. Enter addlsn 2 l/tue 1400-1600 l/thu 1400-1600 again

Expected output: To see the entire list of lessons that are clashes with the input, but the GUI currently does not allow users to expand the output box, thus only showing the first lesson that is clashing with the input, even though there are two lessons that clash.

Actual Output:

`None of the lessons could be added due to clashes. [TUESDAY 1400-1600] - Clashes with: •Betsy Crower: [TUESDAY 1400-1600]

[THURSDAY 1400-1600] - Clashes with: •Betsy Crower: [THURSDAY 1400-1600] `

Screenshots:

image.png

image.png

image.png

Potential problematic behaviour: Users might think there is only one lesson that crashes with their given input

[original: nus-cs2103-AY2425S1/pe-interim#3640] [original labels: type.FunctionalityBug severity.VeryLow]

weijianwong commented 1 day ago

Team's Response

We determined that this feature is not in scope for two reasons:

  1. Users can scroll to view other clashing lessons in the result box: The ability to scroll allows users to access all relevant information without needing additional functionality.

  2. Preventing the addition of students with multiple clashing lessons: The system already ensures that users cannot add students with scheduling conflicts, addressing the primary functionality requirement.

While adjusting the height of this feature could enhance usability, it is considered a non-essential improvement at this stage. Users can still achieve the intended functionality by scrolling to view the clashing students, making this adjustment a lower priority for the current implementation.

Duplicate status (if any):

--