nus-cs2103-AY2122S1 / pe-dev-response

0 stars 0 forks source link

App hides class information when window size is reduced significantly. #5590

Open nus-se-bot opened 2 years ago

nus-se-bot commented 2 years ago

Issue description

App hides class information when window size is reduced significantly.

Steps to reproduce the issue

  1. Resize TeachBook to as small as possible.

What's the expected result?

Either

What's the actual result?

image.png

Additional details


[original: nus-cs2103-AY2122S1/pe-interim#5581] [original labels: severity.Low type.FunctionalityBug]

Wilfredwongkc commented 2 years ago

Team's Response

Thanks for the issue. We reject this bug because it is both an expected behaviour and a correct one, and our team spent quite a lot of time to implement it. If you take a more detailed look at the app, you will realise the app has a minimum width for both the student list panel and the class list panel, which means regardless of the window size, you cannot drag the middle bar to resize the two panels to be narrower than this minimum width. Then, the minimum width of the TeachBook window is set to be exactly the sum of the two minimum widths, which means once you resize the window to the smallest, you won’t be able to drag the middle bar anymore. However, you can still drag the bar at the bottom of the class list panel if you intend to see the number of students in each class. If you wonder why the design is like this, it is because we need to “hide” some information when the window size is small, and we decided to hide the number of students because it may not always be as useful as the class name (which is used to identify a class) and the student information. This is not really related to the bug but if you look at it even more closely, the minimum width of the student list panel and the class list panel is also carefully chosen. For example, the longest class name (19 ‘M’s) can just be properly displayed when the class list panel is smallest in width.

Duplicate status (if any):

--