Closed Bug 1183854 Opened 9 years ago Closed 9 years ago

Switch order of Logviewer header/failure steps containers

Categories

(Tree Management :: Treeherder, defect, P3)

defect

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: jfrench, Unassigned)

References

Details

Attachments

(1 file)

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.
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)
Attached image 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
Closed: 9 years ago
Resolution: --- → WONTFIX
Component: Treeherder: Log Viewer → TreeHerder
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: