Closed Bug 1062825 Opened 7 years ago Closed 7 years ago

Logviewer: Page title should include job details to make tabs recognisable

Categories

(Tree Management :: Treeherder, defect, P3)

defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: emorley, Assigned: jfrench)

References

()

Details

Attachments

(1 file)

The log viewer page title is always:
"Treeherder Log Viewer"

We should make it include details about the job, so multiple open tabs can be differentiated.

eg: Log viewer - <job type/platform> - <repo or timestamp or something>
Based on what's in the artifact header, I think we could do something like this example:

"Log viewer - Ubuntu VM 12.04 x64 mozilla-central pgo test mochitest-e10s-devtools-chrome"

Or we could append the buildid or starttime to the end, if you didn't mind it being longer:

eg.
"Log viewer - Ubuntu VM 12.04 x64 mozilla-central pgo test mochitest-e10s-devtools-chrome - 20141103043009"

or:
"Log viewer - Ubuntu VM 12.04 x64 mozilla-central pgo test mochitest-e10s-devtools-chrome - Mon Nov 03 2014 08:43:07 GMT-0500 (Eastern Standard Time)"

I assume the intent is to have it readable on tab hover anyway.
Flags: needinfo?(emorley)
I like this:

> "Log viewer - Ubuntu VM 12.04 x64 mozilla-central pgo test
> mochitest-e10s-devtools-chrome"

I think anything too much longer and it will get hard to read (and we can always add more, if in the future people say they find it hard to tell tabs or bookmarked log viewer pages apart :-))
Flags: needinfo?(emorley)
Ok, thanks Ed, sounds good. I'll make the change.
Assignee: nobody → tojonmz
Status: NEW → ASSIGNED
I noticed there is also an {{angular}} flicker in the tab, as the page loads. If I can sort that out like we did with the ng-cloak for the angular body elements, I'll open a PR.
Attached file treeherder-ui-PR#269
Please see the above PR for review and status.
Attachment #8521785 - Flags: review?(cdawson)
Commits pushed to master at https://github.com/mozilla/treeherder-ui

https://github.com/mozilla/treeherder-ui/commit/ef3ffcb427ea94f8f2d66598bf85d2614bc95e1a
Bug 1062825 - Include job details in Logviewer page title

https://github.com/mozilla/treeherder-ui/commit/077a82ade8ad2f36eae15a5e37f9ead9b0817b0b
Merge pull request #269 from tojonmz/logviewer-page-title

Bug 1062825 - Include job details in Log viewer page title
Attachment #8521785 - Flags: review?(cdawson) → review+
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Verified fixed and working correctly on dev in both Firefox and Chrome.
Status: RESOLVED → VERIFIED
Commits pushed to master at https://github.com/mozilla/treeherder

https://github.com/mozilla/treeherder/commit/4f1c6852b371222e3238a32dabd1bc6f5dda2c1f
Bug 1062825 - Include job details in Logviewer page title

https://github.com/mozilla/treeherder/commit/2df8878561c6b5128fd00c783106e42aa4291c51
Merge pull request #269 from tojonmz/logviewer-page-title

Bug 1062825 - Include job details in Log viewer page title
No longer blocks: 1060482
You need to log in before you can comment on or make changes to this bug.