Simplify milestone summary
Resources
PM @victorwu | UX @pedroms | FE @mikegreiling
Description
The summary line about issues and merge requests on milestone pages are a bit hard readable. Take a look at this:
Proposal
Change the summary into a sidebar, like in issues and merge requests. For now, pressing the “Edit” link for the sidebar due date links to the milestone edit screen — no need for inline edit for this first iteration.
Sidebar expanded | Sidebar collapsed |
---|---|
![]() |
![]() |
Original Proposal
Simplify summary information:
- Remove
Progress
title. It is obvious that this information is about current milestone. - Group information about issues and merge requests into blocks. Something like this:
Issues
Total: 251
Open: 113
Closed: 138
Merge requests
Total: 251
Open: 113
Closed: 138
3 Add links to all that open
, closed
, etc items just to quickly open issues/merge requests pages with preset filters.
Links / references
Designs
- Show closed items
Activity
-
Newest first Oldest first
-
Show all activity Show comments only Show history only
- Author Developer
Another thought about how to make this information a bit easily readable.
If those vertical space proposal is not acceptable, why not to make open/closed information more colorful?/cc @dzaporozhets, @awhildy
Hi @blackst0ne,
I agree that the milestone information can be presented in a more readable way. There is already a great visual design in context of adding burndown charts at the Milestones page for GitLab EE that takes care of this, see in https://gitlab.com/gitlab-org/gitlab-ee/issues/91#note_16966039. Currently planned for 8.14. It would make sense to ideally combine that with an UI update of the page for CE as well.
/cc @pedroms
Edited by Andreas Kämmerle- Author Developer
@andreaskaemmerle, wow, that's look great!
- Mark Fletcher Added ~19492 ~157398 labels
Added ~19492 ~157398 labels
- Developer
/cc @awhildy
1 - Pedro Moreira da Silva Mentioned in issue #23704 (closed)
Mentioned in issue #23704 (closed)
- Pedro Moreira da Silva Reassigned to @pedroms
Reassigned to @pedroms
- Allison Whilden Milestone changed to %29
Milestone changed to %29
- Allison Whilden Added ~897282 label
Added ~897282 label
- Developer
@blackst0ne @andreaskaemmerle I've was thinking about how we can simplify it and still keep it usable when tackling https://gitlab.com/gitlab-org/gitlab-ce/issues/23704 and https://gitlab.com/gitlab-org/gitlab-ee/issues/91. Also, we have to keep in mind that people may add a description to the milestone… so, many things to consider. The following proposals also take https://gitlab.com/gitlab-org/gitlab-ce/issues/23602 into consideration (it's a very small tweak).
1 - Pedro Moreira da Silva Mentioned in issue #23602 (closed)
Mentioned in issue #23602 (closed)
- Pedro Moreira da Silva Mentioned in commit gitlab-design@7b5df7c2
Mentioned in commit gitlab-design@7b5df7c2
- Developer
This is an improvement for sure. I am wondering if we should just create a sidebar for milestone views, though. The styling matches the sidebar but I think it is easier to digest when stacked.
- Pedro Moreira da Silva Mentioned in issue #22398 (closed)
Mentioned in issue #22398 (closed)
- Developer
@tauriedavis I don't oppose to having a sidebar, at all. I'm going to make a mockup with the sidebar
- Developer
Thanks @tauriedavis for your suggestion. It works much better with the sidebar, especially when considering the future changes to the milestones. Tell me what you think about the icons when it's collapsed.
4 - Pedro Moreira da Silva Mentioned in commit gitlab-design@543dcc96
Mentioned in commit gitlab-design@543dcc96
- Developer
Awesome! That looks great @pedroms
3 - Taurie Davis Added ~127616 and removed ~19492 labels
Added ~127616 and removed ~19492 labels
- Taurie Davis Assignee removed
Assignee removed
- Contributor
for sidebar from me - Contributor
@blackst0ne @tauriedavis @pedroms Where is this standing? Can I attempt to implement it?
- Developer
@leibo yes! You most certainly can. Thank you!
- Contributor
@tauriedavis Thanks! Who can support the frontend part of this?
- Developer
@ClemMakesApps Are you able to support Ido with his community contribution?
- Pedro Moreira da Silva Mentioned in commit gitlab-design@4e91fef1
Mentioned in commit gitlab-design@4e91fef1
- Pedro Moreira da Silva Mentioned in commit gitlab-design@9d8ef511
Mentioned in commit gitlab-design@9d8ef511
- Developer
Edited issue description with updated designs (original due date icon and issue weight info) and added link for spec previews.
- Contributor
Sure, I can help @tauriedavis @leibo
- Developer
Edited description: formatted designs in a table for better issue viewing.
- Contributor
Hi, I'm currently stuck on some other issues, I won't get to this issue in the next few weeks.
- Developer
Thanks for letting us know @leibo!
- Allison Whilden mentioned in issue #26259 (closed)
mentioned in issue #26259 (closed)
- Allison Whilden changed milestone to %8.17
changed milestone to %8.17
- Allison Whilden added Stretch label
added Stretch label
- Allison Whilden mentioned in issue #27199 (closed)
mentioned in issue #27199 (closed)
- Pedro Moreira da Silva added ~974571 label
added ~974571 label
- Pedro Moreira da Silva removed Stretch label
removed Stretch label
- Pedro Moreira da Silva changed milestone to %29
changed milestone to %29
- Pedro Moreira da Silva mentioned in commit gitlab-design@4f83153b
mentioned in commit gitlab-design@4f83153b
- Author Developer
Any progress here?
- Developer
@blackst0ne I don't believe so. This issue isn't explicitly scheduled yet. We are probably favoring https://gitlab.com/gitlab-org/gitlab-ee/issues/91 according to this comment.
However, I think @leibo was going to pick this up. @leibo did you have the time?
Edited by Pedro Moreira da Silva - Contributor
Sorry, no. I won't have the capacity for this in the short term.
- Developer
@victorwu I think this greatly improves the readability of the milestone summary and it appears to me that it's ~Frontend only. Would you consider scheduling this?
@jschatz1 : Let's do this for 9.1. It will go well with our burn down chart. https://gitlab.com/gitlab-org/gitlab-ee/issues/91
3- Developer
Yayy, happy to see this going in with burn down charts
@pedroms : For the sidebar due date, there's a link to edit it. For now, I assume we just want that to go to the milestone edit screen right? No need for inline edit for this first iteration.
If you look at this: https://gitlab.com/gitlab-org/gitlab-ee/milestones/21, it says "Past due" in two places. In the new design, you only have it one place. Is that intended? (I like it. Just want to confirm.)
- Developer
@victorwu Yes and yes
I've updated the description with that note. - Victor Wu removed ~897282 label
removed ~897282 label
- Mike Greiling assigned to @mikegreiling
assigned to @mikegreiling
- Taurie Davis mentioned in merge request !9698 (closed)
mentioned in merge request !9698 (closed)
- Developer
@pedroms Do we need the browse buttons in the sidebar? Does clicking the number of open/closed issues/mrs go to the browse view already?
1 1 - Pedro Moreira da Silva mentioned in commit gitlab-design@37b857e8
mentioned in commit gitlab-design@37b857e8
- Developer
@tauriedavis indeed, we don't need those… :slap_in_the_forehead:
I've updated the designs accordingly. Thanks! 1 - Victor Wu added ~874211 label
added ~874211 label
- Contributor
@pedroms should we use the merge request icon instead of the
!
in the collapsed sidebar view?https://gitlab.com/gitlab-org/gitlab-ce/issues/29584#note_25546849
- Mike Greiling created branch
23674-simplify-milestone-summary
created branch
23674-simplify-milestone-summary
- Mike Greiling mentioned in merge request !10096 (merged)
mentioned in merge request !10096 (merged)
- Developer
@mikegreiling good point, thanks for mentioning that! Let's use the proposed icon https://gitlab.com/gitlab-org/gitlab-ce/issues/29584 as you suggest.
- Contributor
Here are the two MRs for this issue. Both are going through CI and should be ready for review.
gitlab-ce!10096
gitlab-ee!1511 - Jacob Schatz closed via merge request !10096 (merged)
closed via merge request !10096 (merged)
- Jacob Schatz mentioned in commit f7fefe82
mentioned in commit f7fefe82
- Taurie Davis mentioned in issue #28481 (closed)
mentioned in issue #28481 (closed)
- 🤖 GitLab Bot 🤖 added devopsplan label
added devopsplan label