Code review comment for ~caleb-ellis/maas:suppress-single-test

Revision history for this message
Martin Storey (cassiocassio) wrote :

nice.

- The column width for the HIDE column is kind of wide for a single checkbox

- Should/could we special-case "All tests hidden" in the notification?

(scope creep)

- when you get to "Hide all tests" in the overide failed tests dialog, can you expose the same hide-all functionality with a checkbox at the top the HIDE column?

« Back to merge proposal