mozilla / scanjs

[DEPRECATED] Static analysis tool for javascript code.
Other
429 stars 39 forks source link

checklist: add comment/hide/delete button in output view #173

Closed mozfreddyb closed 2 years ago

mozfreddyb commented 9 years ago

As a reviewer, I want to flag findings that I consider false positives and hide them from my view.

The current output view shows Filename/Line/Rule. I suggest that we add a column which allows to remove (or hide?) the specific finding from the result set. Let's do this by establishing a finding.hidden property. A

Alternatively we could also provide a few more buttons that allow classifying a finding in a more detailed way. In that case I'd suggest the button shows a speech bubble and when being clicked gives the option to fill out a quick textarea (for comments) and then one or more of the following tags (or free form too, which some well known ratings highlighted, e.g. high=red, etc.)