Skip to content
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

Add a zero-width space after test name in HTML results #3423

Merged
merged 2 commits into from
Mar 3, 2022

Conversation

RikkiGibson
Copy link
Member

@RikkiGibson RikkiGibson commented Mar 1, 2022

This PR adjusts the HTML results generation to make it easier to highlight the name of the test with a double-click. I've attached a GIF which shows the existing behavior, and demonstrates a JavaScript snippet which achieves a similar result as this PR.

After the change, it's much easier to double-click, copy the name of the test, and ctrl+F for it in some other place, such as the file where the test is declared.

The JavaScript snippet is not part of the actual change, but included here for reference:

document.querySelectorAll(".leaf-division div:first-child span:nth-of-type(2)").forEach(elem => elem.innerHTML += "​")

zero-width-space-after-test-name

@Evangelink
Copy link
Member

@RikkiGibson Would you be able to merge/rebase main, we fixed an issue that's affecting your PR.

@RikkiGibson
Copy link
Member Author

Have merged in the latest main. Thanks!

@Evangelink Evangelink merged commit 81c415f into microsoft:main Mar 3, 2022
@RikkiGibson RikkiGibson deleted the patch-1 branch March 4, 2022 00:11
@RikkiGibson
Copy link
Member Author

Thanks very much @Haplois @Evangelink!

@Evangelink
Copy link
Member

Thank you for your contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants