Closed
Bug 1045618
Opened 10 years ago
Closed 10 years ago
Repo menu: The "repos" panel should use all the space available
Categories
(Tree Management :: Treeherder, defect, P2)
Tree Management
Treeherder
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: Paolo, Assigned: camd)
References
Details
Attachments
(1 file)
103.79 KB,
image/png
|
Details |
The "repos" panel often needs scrolling to find the repository you are looking for, but at the same time it overlaps with the interface, meaning that it should be closed anyways before continuing. It might as well use all the space available and require no scrolling.
Additionally, the checkboxes should be replaced with "plus" buttons after each name, since the removal is handled by the X buttons in the toolbar itself, and I don't need to use the checkboxes to see which repositories I've selected because I already see them in the toolbar.
Updated•10 years ago
|
Blocks: treeherder-sheriff-transition
Priority: -- → P3
Updated•10 years ago
|
No longer blocks: treeherder-sheriff-transition
Updated•10 years ago
|
Priority: P3 → P2
Updated•10 years ago
|
Summary: The "repos" panel should use all the space available → Repo menu: The "repos" panel should use all the space available
Comment 2•10 years ago
|
||
Hey Ed, what do you think of using a large bootstrap modal for the repos? We'd get a lot of nice behavior (clicking outside the modal dismisses it, etc) that way for free.
http://getbootstrap.com/javascript/#modals
Flags: needinfo?(emorley)
Reporter | ||
Comment 3•10 years ago
|
||
(In reply to William Lachance (:wlach) from comment #2)
> clicking outside the modal dismisses it
I don't know about the technology, it might be actually be a good idea as far as I can tell, but I would say that clicking outside the popup to close it is not the user interface I would design, as it's not discoverable, but more importantly works differently than other panels in the same application. I think the current method of toggling the panel works well.
Comment 4•10 years ago
|
||
(In reply to :Paolo Amadini from comment #3)
> (In reply to William Lachance (:wlach) from comment #2)
> > clicking outside the modal dismisses it
>
> I don't know about the technology, it might be actually be a good idea as
> far as I can tell, but I would say that clicking outside the popup to close
> it is not the user interface I would design, as it's not discoverable, but
> more importantly works differently than other panels in the same
> application. I think the current method of toggling the panel works well.
There's also an "X" on the modal to dismiss. Agreed that this would be different from the other panels (actually if we used this it would no longer really be a panel), but then again the repos panel has pretty different behaviour than the other panels (which are designed to filter stuff inside the current view).
Comment 5•10 years ago
|
||
I'm not sure. Thinking about this some more I'm not sure if making modal is the right way to go. I've explained the common workflows a bit more in bug 1053985 comment 2 (added to that bug, since that's the overall UX bug) - with some thoughts.
Flags: needinfo?(emorley)
Comment 6•10 years ago
|
||
Hey Ed, so now that we're back to using the checkboxes, I still think a modal is the way to go here, I think the way they work (easy to dismiss by clicking on X or outside) works well for the use cases you describe in bug 1053985 comment 2. I knocked up a prototype today, here's a screenshot... let me know what you think?
Assignee: nobody → wlachance
Attachment #8485255 -
Flags: feedback?(emorley)
Comment 7•10 years ago
|
||
I'll ask the other sheriffs what they think today :-)
Status: NEW → ASSIGNED
Comment 8•10 years ago
|
||
Comment on attachment 8485255 [details]
repo-modal.png
So this is a definite improvement over what we have already - though talking things through here, one of the main issues we identified for usability of the current repos menu that is still present with this mockup, is the fact that having multiple columns for each group makes it very hard to quickly find a desired repo. ie: it's hard to tell at first glance whether it's sorted alphabetically left-to-right then up-and-down, or vice versa.
To avoid this, we were wondering whether it would be best to display each group as a single vertical column, with each group next to each other.
ie:
[development] | [release-stabilization] | [foo]
mozilla-central | mozilla-aurora | foo-repo
mozilla-inbound | mozilla-aurora> | bar-repo
Attachment #8485255 -
Flags: feedback?(emorley)
Comment 9•10 years ago
|
||
(In reply to Ed Morley (At work week 6th-13th Sept) [:edmorley] from comment #8)
> Comment on attachment 8485255 [details]
> repo-modal.png
>
> So this is a definite improvement over what we have already - though talking
> things through here, one of the main issues we identified for usability of
> the current repos menu that is still present with this mockup, is the fact
> that having multiple columns for each group makes it very hard to quickly
> find a desired repo. ie: it's hard to tell at first glance whether it's
> sorted alphabetically left-to-right then up-and-down, or vice versa.
>
> To avoid this, we were wondering whether it would be best to display each
> group as a single vertical column, with each group next to each other.
>
> ie:
>
> [development] | [release-stabilization] | [foo]
> mozilla-central | mozilla-aurora | foo-repo
> mozilla-inbound | mozilla-aurora> | bar-repo
Definitely worth a try. We could do this with a table, most likely:
http://getbootstrap.com/css/#tables
Assignee | ||
Comment 10•10 years ago
|
||
this is just a drop-down menu now.
https://github.com/mozilla/treeherder-ui/commit/1ac87534a44c7ff5e8ae000752cb878800e9b272
Assignee: wlachance → cdawson
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•