Closed Bug 1182068 Opened 6 years ago Closed 6 years ago

Load the first visible step in Logviewer by default

Categories

(Tree Management :: Treeherder: Log Viewer, defect, P3)

defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: glob, Assigned: jfrench)

References

Details

Attachments

(1 file)

46 bytes, text/x-github-pull-request
camd
: review+
Details | Review
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.
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? :)
Blocks: 1182178
OS: Unspecified → All
Priority: -- → P3
Hardware: Unspecified → All
Summary: if there's only one step in the log it should be viewed by deafult → Load the first visible step in Logviewer by default
Assignee: nobody → tojonmz
Status: NEW → ASSIGNED
Attached file PR 828
Please see above PR for status and review :)
Attachment #8643154 - Flags: review?(cdawson)
Attachment #8643154 - Flags: review?(cdawson) → review+
Comment on attachment 8643154 [details] [review]
PR 828

An additional tweak for camd to review :) Details in the PR.
Attachment #8643154 - Flags: review+ → review?(cdawson)
Attachment #8643154 - Flags: review?(cdawson) → review+
Marking fixed per above merge. Will verify on the next push to stage/prod.
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Verified fixed on stage and prod.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.