Switch order of Logviewer header/failure steps containers

RESOLVED WONTFIX

Status

Tree Management
Treeherder: Log Viewer
P3
normal
RESOLVED WONTFIX
3 years ago
3 years ago

People

(Reporter: jfrench, Unassigned)

Tracking

Details

Attachments

(1 attachment)

(Reporter)

Description

3 years ago
Jgriffin posed that users interact with the Logviewer failure steps container first and more frequently, so he suggested reversing the layout order of the main Logviewer containers, to be:

Failure/Success Steps | Meta/Job Header Info

This is sort of opposite to the layout style from the main Treeherder panel where the Job Details is left side and the interactive task-like 'stuff' like suggested failures are on the right, but we've no need to adhere to that consistency in Logviewer if there's a clear UX win.

Adding a few folks for feedback and opinion, and since we have so many users we want to be careful it's not perceived as an arbitrary switch.
(Reporter)

Comment 1

3 years ago
Adding a few need info's, hopefully I haven't over solicited.

A note this is just part of a broader amount of proposed work for Logviewer in meta bug 1182178.
Flags: needinfo?(wkocher)
Flags: needinfo?(ryanvm)
Flags: needinfo?(philringnalda)
Flags: needinfo?(mjzffr)
Flags: needinfo?(jmaher)
Flags: needinfo?(dbaron)
I don't really interact with the log viewer UI much. As long as my logs show up properly, I'm happy.
Flags: needinfo?(mjzffr)
Created attachment 8633853 [details]
Screenshot

Hmm, so this?

Other than being opposed to breaking muscle memory after using it the other way for a year or so, I'm mostly indifferent.
Flags: needinfo?(wkocher)
As an aside, I think it'd be okay to move those two buttons so they're under the job details section instead of the failure steps. That should hopefully free up like half an inch or so of vertical space for the actual log viewer.
personally I just use the raw logs, so moving this wouldn't affect me.
Flags: needinfo?(jmaher)
I'm having a hard time justifying breaking muscle memory over this. I can't recall ever hearing any devs complaining about it either.
Flags: needinfo?(ryanvm)
No opinion from me.
Flags: needinfo?(dbaron)
Doesn't seem worth doing to me, but if it's done I won't have much trouble adapting.
Flags: needinfo?(philringnalda)
Seems like not much love for this, let's not do it.
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.