ShenyiCui / pe

0 stars 0 forks source link

Docs: No breakdown of the different types of tooltips available #12

Open ShenyiCui opened 1 year ago

ShenyiCui commented 1 year ago
soc-pe-bot commented 1 year ago

Team's Response

We decided to reject this report due to the following reasons.

image.png

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: >>> It would be very unlikely that the user is confused by “common mistake” and “caution” as they have very different definitions.

  • Common mistakes can be a cause for caution.
  • Caution can be to avoid common mistakes.

The claim that they have very different definitions is false. It, in fact, is very confusing what the distinction is between a Common Mistake and Caution. Clarifying the distinction would be helpful to the user.

As for the severity of caution vs warning, we believe that based on their symbols and colours, it is intuitive to know the hierarchy between them.

  • Point is well taken, fair enough.

:question: Issue severity

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

Reason for disagreement: >>> Only cosmetic problems should have this label.

This isn't a purely cosmetic problem, as it could confuse the reader.