Closed Bug 1066377 Opened 10 years ago Closed 8 years ago

Improve the layout of the repo menu

Categories

(Tree Management :: Treeherder, defect, P4)

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: KWierso, Assigned: KWierso)

References

()

Details

Attachments

(2 files, 2 obsolete files)

Attached image Mockup (obsolete) —
Bug 1045618 changed the repo panel into a plain dropdown menu, which is much nicer for scanning than the panel ended up being. I think the dropdown menu could be further cleaned up by making it a two-column menu, so it takes up less vertical space. Attached is kind of what I was thinking this could be. On the top of the left column, we could have all of the repos from the "development" group. To the right of that, everything from "release-stabilization". Below them, the rest of the categories can be laid out in such a way that everything fits next to the "projects" group. This layout keeps most of the sheriffs' frequently accessed repositories at the top of the menu, while still separating the development and stabilization branches. On decently sized/resolutioned monitors, the entire menu's contents should be visible with minimal or no scrolling required.
I might have a look at this, and will assign myself if appropriate. Atm now having some issues pulling job data from dev/stage/prod.
The mockup looks good; Ryan/Tomcat what do you think? (Compare to current treeherder-dev http://treeherder-dev.allizom.org/ )
OS: Windows 8.1 → All
Priority: -- → P3
Hardware: x86_64 → All
Blocks: 1053985
Assignee: nobody → tojonmz
Status: NEW → ASSIGNED
No longer blocks: treeherder-dev-transition
Sorry I've been fixing a bunch of other bugs and haven't had time for this one yet. I'll un-assign myself for now. If someone else wants to take it, feel free. I will investigate it more if/when I can.
Assignee: tojonmz → administration
Status: ASSIGNED → NEW
Assignee: administration → nobody
Blocks: 1125264
Blocks: 1125352
No longer blocks: 1053985
No longer blocks: 1125264
Summary: Further cleanup of the repos menu → Improve the layout of the repo menu
Priority: P3 → P4
Assignee: nobody → wkocher
Attached image Screenshot of PR (obsolete) —
Below the fold is the rest of the 'taskcluster' and 'qa automation tests' group repos, and the entirety of the 'other' group.
Comment on attachment 8856308 [details] Screenshot of PR looks nice ! at some point i guess for community members some tooltips at the repos would be nice but i guess that can come at a later point
Attachment #8856308 - Flags: feedback?(cbook) → feedback+
I've added tooltips to the repo group headers.
Comment on attachment 8856307 [details] [review] [treeherder] KWierso:repomenu > mozilla:master Looks good! I wonder if we should also move the comm-* repos to their own group?
Attachment #8856307 - Flags: feedback?(emorley) → feedback+
Comment on attachment 8856307 [details] [review] [treeherder] KWierso:repomenu > mozilla:master I've done a major rewrite of the PR. It's now arranged into two columns with css grid, rather than just floats. There is a new 'comm-repositories' repo group, and all of the comm-* repos have been moved into this new group. Descriptions of the repository groups have been added to each repo group, and the UI shows them in the menu via tooltips. There's a new field in the repository model named 'is_try_repo' which is set to true for the try repos. I've moved the try repos out of the try repo group and into more fitting repo groups. This effectively removes the try repo group. The UI now looks for this new field rather than the try group for a number of checks. A number of repos haven't had any pushes in over a month. I've marked a bunch of them as onhold to trim down the repo menu a bit. (Happy to undo some of these if they are in fact still active.)
Attachment #8856307 - Flags: review?(emorley)
Here's what the repo menu looks like with all of these changes.
Attachment #8488348 - Attachment is obsolete: true
Attachment #8856308 - Attachment is obsolete: true
Comment on attachment 8856307 [details] [review] [treeherder] KWierso:repomenu > mozilla:master Have left some comments :-)
Attachment #8856307 - Flags: review?(emorley)
Attachment #8856307 - Flags: review?(emorley) → review+
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: