sikai00 / pe

0 stars 0 forks source link

Closing results window after it popping out does not restore the results in the view panel #1

Open sikai00 opened 1 year ago

sikai00 commented 1 year ago

Steps to reproduce:

1) show 1

image.png

2) click on results graph

3) close the results graph

4) image.png

The graph is not restored into the window.

nus-pe-bot commented 1 year ago

Team's Response

While this is a clear bug and affects usage, considering the work done relative to the graph feature, where it functions with the grade and assignment command. Furthermore, although this is not specifically mentioned, it can be fixed by using the show 1 command again.

Also, the user is unable to misuse the missing graph that will cause the app to crash. While difficult to see, the graph can actually be clicked again when it disappears by clicking the small area where the graph used to be, hence its mostly a UI bug. This is to say that fixing the bug is of much lesser importance compared to the features it stems from.

Before and after outlining the missing graph border box, admittedly hard to see. Clicking the area will lead back to the graph, although the graph is not shown.

image.png

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: I believe that it is not unlikely that a user would not want to see their graph disappear after having it pop out. This should indeed be a functionality bug. After closing the window, it is likely that a user would want to see whatever changes made to the statistics reflected in the original window. It may be surprising to the user to see it disappear, and it is likely very common for a user to use this feature. While this is considered a UI bug, I believe there is high value in rectifying this bug as such and this should still be in scope.

As for the point about how the graph can be made to be shown again, it is reflected in the choice of severity of "low".

Furthermore, the point of the UI being clickable from an almost invisible (unless you really try to find it) seems irrelevant to this bug report.