Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Screenshots could provide more context #11

Open
chaaaaun opened this issue Apr 19, 2024 · 1 comment
Open

Screenshots could provide more context #11

chaaaaun opened this issue Apr 19, 2024 · 1 comment

Comments

@chaaaaun
Copy link
Owner

chaaaaun commented Apr 19, 2024

For commands that require information from both lists, it might be clearer if the screenshots provided show the entire app window. For example, for lookup:

Screenshot 2024-04-19 at 5.27.04 PM.png

While the change in person list is shown, the actual article that is being looked up is not shown. It would be helpful if users could see exactly how the article looks like such that performing the lookup would lead to the given result.

@nus-pe-script
Copy link

nus-pe-script commented Apr 22, 2024

Team's Response

Agreed, it could show the full app at the beginning to be clearer. I changed the severity to very low as this should not hinder the reader.

Items for the Tester to Verify

❓ Issue severity

Team chose [severity.VeryLow]
Originally [severity.Low]

  • I disagree

Reason for disagreement: I feel that the very fact that the team has acknowledged that the screenshots "could be clearer" shows that this is not a purely cosmetic issue, and warrants a higher severity than VeryLow.

Based on the second bullet point in the UG guidelines below, I would make the case that omitting the very list that the command is operating on shows that the visuals do not fit the explanation. For example, based on the screenshot in the original issue, readers will likely have to put in additional effort to understand the effect of looking up index 2, when the item at that index isn't even shown.

Screenshot 2024-04-23 at 3.45.39 PM.png


Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants