bradleyfalzon / gopherci-web

website for gopherci.io

Home Page:https://gopherci.io

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Not possible to tell which repo an analysis applies for, when looking at heading.

dmitshur opened this issue · comments

Consider the following page:

https://gci.gopherci.io/analysis/1014

Without scrolling down and looking at the output log, it's not possible to know what repo the analysis applies for.

It's not in the URL. Not in the page title, "Analysis - GopherCI". Not in heading:

image

I think the full commit ID, 431bb22d86ad0a95db0780e27ab6221b49238464, does not need that much visibility. In fact, I didn't even realize it was the git commit ID, because it's not shortened to 7 or 8 characters. I thought it was the analysis ID.

Here's an alternative heading that would be much more clear IMO. It's just a mockup to demonstrate the idea:

image

I think it would look even better if the commit were in a monospaced font, and linked to the actual commit on GitHub.

Yes, I agree this is needed, and there were just a few cases that needed to be handled so I hadn't written the code and template for it.

Note, it's my fault that I haven't made it clear, but this repo is just for the hosted GopherCI interfaces, the analysis page is in the other repo (specifically https://github.com/bradleyfalzon/gopherci/blob/master/internal/web/templates/analysis.tmpl). Can you recreate there, else I will.

Can you recreate there, else I will.

Sure. Moved to bradleyfalzon/gopherci#125.