cta-wave / dpctf-test-runner

Repo for the DPCTF Test Runner
Other
0 stars 3 forks source link

UI improvements for configuring tests to run on session configuration page #9

Closed jpiesing closed 3 years ago

jpiesing commented 3 years ago

The session configuration page UI for configuring which tests to run could be improved in several ways.

image

1) The message "select at least one API" is wrong and confusing. The test operator needs to select either one API or one test within an API. 2) It is not remotely obvious that selecting an API name or the area to the right of the API name shows all the tests within that API.

The first comment could be addressed by changing the message to "select at least one API or at least one test within an API". The second comment could be addressed by some kind of indication that the API name can be expanded to list the tests.

FritzHeiden commented 3 years ago

Modified error message and added icons as expanding indicators

jpiesing commented 3 years ago

OK.