search

nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Small window size unusable #4273

Open nus-pe-bot opened 6 months ago

nus-pe-bot commented 6 months ago

When I resized the window smaller, it seems to work fine for most sizes due to the scrollbars:

image.png

However, when resized to the minimum allowed size, the scrollbar disappears. The app becomes unusable in such a case as I can't see details of any Person in the app:

image.png

Would be good to increase the minimum allowed size of the app's window so that such a scenario does not occur.

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

delishad21 commented 6 months ago

Team's Response

Thank you for your report. We agree that the application being minimized to such a size does block the persons and exams list. However, this was an intentional design choice.

As the command bar is still active and the results display is still viewable. the application is still usable for entering commands.

We only defined a minimum size for the command bar and the results display as we considered a potential (niche) use case: a user who is referencing a large sheet with a lot of student data might prefer to have the application in a small form factor at the corner of his screen as they add students into the application. Error messages and command success messages are all the user needs to see in this use case, the person list is not strictly needed.

The only issue is that the application is rather unsightly, making this issue purely cosmetic. Hence, we propose a downgrade of severity to severity.veryLow. Furthermore, fixing cosmetic issues like these was of a much lower priority than the improvements and upgrades we made to the UI as elaborated on in the DG, making this issue not in scope.

Duplicate status (if any):

--