1182068 - Load the first visible step in Logviewer by default

Status: VERIFIED FIXED

(Tree Management :: Treeherder, defect, P3)

Description

[:glob ✱]

for jobs that have a single log entry, such as https://treeherder.mozilla.org/logviewer.html#?job_id=135&repo=bmo-master you shouldn't have to click on the step to view the log.

Comment 1

[Jonathan French (:jfrench)]

Thanks :glob. We'd like to generalize this to loading the first visible step (whether there's one step or many) so we'll use your bug for that if that's ok? :)

Comment 2

[Jonathan French (:jfrench)]

Attachment: PR 828

Please see above PR for status and review :)

Comment 3

[Jonathan French (:jfrench)]

Comment on attachment 8643154 [details] [review]
PR 828

An additional tweak for camd to review :) Details in the PR.

Comment 4

[Treeherder GitHub Bugbot]

Commit pushed to master at https://github.com/mozilla/treeherder

https://github.com/mozilla/treeherder/commit/7fed6e575e1c3de1616ff9cfb85ab1b84c4f7961
Bug 1182068 - Load log content in Logviewer by default

Comment 5

[Jonathan French (:jfrench)]

Marking fixed per above merge. Will verify on the next push to stage/prod.

Comment 6

[Jonathan French (:jfrench)]

Verified fixed on stage and prod.