testing: try out new 'badges' UI for test failures #233137
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
I'm exploring some UI to make tests look better in already-busy scenarios, like Copilot edits. We've long had these big inline message decorations which get very noisy quickly, especially with word wrap on. I'm exploring these smaller 'badge' type indicators. These are shown in at most one place per test and their content length is very limited. Clicking on them opens the traditional test error peek. Because I feel like they are easier to see and more inviting to click, I turned off the default behavior of opening the error peek. Making any edits in the test case will dismiss the badge. Closes #233028
alexdima
approved these changes
Nov 5, 2024
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I'm exploring some UI to make tests look better in already-busy
scenarios, like Copilot edits. We've long had these big inline message
decorations which get very noisy quickly, especially with word wrap on.
I'm exploring these smaller 'badge' type indicators. These are shown in
at most one place per test and their content length is very limited.
Clicking on them opens the traditional test error peek. Because I feel
like they are easier to see and more inviting to click, I turned off the
default behavior of opening the error peek. Making any edits in the test
case will dismiss the badge.
Closes #233028