-
Notifications
You must be signed in to change notification settings - Fork 243
Only clicking the Results row show the log information #653
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
Comments
I'm not sure I follow. When you say "any of the rows" do you mean "any of the cells" for a particular row? One row == one test (not entirely true as setup/teardown can get their own rows). Regardless, please try We've changed so that instead of only the "show/hide" text being clickable, the entire Results cell is now clickable. |
Exactly. Right now only clicking the Results cell expands the logs...but my opinion is, it would be better UI if ANY of the cells (except the links) expanded the logs, even clicking the test name or the duration |
Cool, I'll see if it's an easy add. If it is - I'll add it to RC5. |
I think this is what you're after: #656 Now all cells (incl. user added, excl. Links) will be clickable to expand/collapse the row. |
Thanks @BeyondEvil ! How would I pull in that PR to my test suite with |
You can pip install master branch (it’s been merged). https://stackoverflow.com/questions/56161092/how-to-install-a-pip-module-from-its-master-branch-on-github#56161126 |
I use pipenv so github installs aren't supported, need to grab it from a release unfortunately |
You should be able to do: If not, you can do or
|
This is more of an enhancement, but, it would be a huge improvement to the UI
Right now, the only thing you can click to show/expand the logs is the Results row. However, the Results row is kind of small. This report would be much easier to use if any of the rows, when clicked, showed/expanded the logs.
How can I make this happen?
The text was updated successfully, but these errors were encountered: