Skip to content
This repository has been archived by the owner on Jun 21, 2023. It is now read-only.

Checks need more information #1967

Open
StanleyGoldman opened this issue Oct 2, 2018 · 5 comments · May be fixed by #2013
Open

Checks need more information #1967

StanleyGoldman opened this issue Oct 2, 2018 · 5 comments · May be fixed by #2013
Assignees

Comments

@StanleyGoldman
Copy link
Contributor

Check Runs and Statuses on GitHub.com have a secondary field which often conveys some important information.

image

Our handling of that in the pull request detail does not find a way to surface that information.

image

I'm tempted to add a tooltip, but I feel that is not very appealing from a UX standpoint.

@donokuda ideas?

@donokuda donokuda self-assigned this Oct 2, 2018
@donokuda
Copy link
Contributor

Here's a quick idea of adding a second line the list of checks, which I think works well:

I'm tempted to add a tooltip, but I feel that is not very appealing from a UX standpoint.

I'm also open to using a tooltip to keep the interface minimal since I don't think putting the supporting information would block or confuse someone's workflow.

How about we try out the 2 line approach and use a tooltip if we feel that it's too cluttered?

@StanleyGoldman
Copy link
Contributor Author

That looks good to me

@donokuda
Copy link
Contributor

I also wanted to highlight that the "Details" link is in the second row. Unfortunately I won't be able to jump in to do visual tweaks in XAML since my development environment is still broken. 😭

Hopefully this image is helpful:

@StanleyGoldman StanleyGoldman linked a pull request Oct 25, 2018 that will close this issue
1 task
@xXMrNiceXx1
Copy link

@AArnott

@AArnott
Copy link

AArnott commented May 9, 2020

@xXMrNiceXx1 was there something you needed?

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

Successfully merging a pull request may close this issue.

4 participants