MR widget JUnit seems strange
I noticed in https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/21307 that our JUnit presentation could be improved.
Designs
- Show closed items
Relates to
- #23962Backlog
- gitlab-foss #51004
Activity
-
Newest first Oldest first
-
Show all activity Show comments only Show history only
- Contributor
@ayufan what exactly do you mean?
- Filipa Lacerda added UX label
added UX label
- Filipa Lacerda unassigned @filipa
unassigned @filipa
- Author Maintainer
Look at paddings, try the CI job failure how it looks on gitlab.com and when you expand it.
- Elliot Rushton added 1 deleted label and removed 1 deleted label
added 1 deleted label and removed 1 deleted label
- 🤖 GitLab Bot 🤖 added [deprecated] Accepting merge requests label
added [deprecated] Accepting merge requests label
- Jason Yavorska added Category:Continuous Integration label
added Category:Continuous Integration label
- Jason Yavorska added grouppipeline execution label
added grouppipeline execution label
- 🤖 GitLab Bot 🤖 removed [deprecated] Accepting merge requests label
removed [deprecated] Accepting merge requests label
- Jason Yavorska changed milestone to %Backlog
changed milestone to %Backlog
- 🤖 GitLab Bot 🤖 added [deprecated] Accepting merge requests label
added [deprecated] Accepting merge requests label
- Jason Yavorska added Category:Code Testing and Coverage grouppipeline security labels and removed Category:Continuous Integration grouppipeline execution labels
added Category:Code Testing and Coverage grouppipeline security labels and removed Category:Continuous Integration grouppipeline execution labels
- Nadia Udalova mentioned in issue #217568 (closed)
mentioned in issue #217568 (closed)
- Juan J. Ramirez added MR-Improvements label
added MR-Improvements label
- Dimitrie Hoekstra marked this issue as related to #217568 (closed)
marked this issue as related to #217568 (closed)
- Contributor
@jj-ramirez have you taken a look at this by chance?
- Contributor
@jheimbuck_gl Seems like we need to fix some padding here. Doesn't look that complicated. I probably can do this myself.
Collapse replies - Contributor
no rush on it, just checking in to see if you had a fix stashed maybe . . ;). If you start on it pull it into the milestone but it's not a priority.
- Contributor
@jheimbuck_gl Sounds good. Just assigned this to myself.
- Juan J. Ramirez added 1 deleted label
added 1 deleted label
- Juan J. Ramirez assigned to @jj-ramirez
assigned to @jj-ramirez
- 🤖 GitLab Bot 🤖 added sectionops label
added sectionops label
- Maintainer
Thanks for working on this @jj-ramirez! We've removed the ~"Accepting merge requests" label to avoid having multiple people working on the same issue.
- 🤖 GitLab Bot 🤖 removed [deprecated] Accepting merge requests label
removed [deprecated] Accepting merge requests label
- Nadia Udalova unassigned @jj-ramirez
unassigned @jj-ramirez
- Contributor
Removing Juan as an assignee to be prioritised and assigned to the new designer when arrives.
- 🤖 GitLab Bot 🤖 added [deprecated] Accepting merge requests label
added [deprecated] Accepting merge requests label
- Nadia Udalova removed 1 deleted label
removed 1 deleted label
- Contributor
@gdoyle I think we've covered the problems covered here in some of our recent changes (Test Summary Cleanup, Modal unreadable) or will in the upcoming MR Widget work.
@ayufan - I'm closing this out based on those changes but feel free to re-open if the initial problems aren't yet addressed.
1 Collapse replies - Developer
Here's the link to the test summary MR widget redesign as well - @jheimbuck_gl agreed that it should fix those padding concerns!
- James Heimbuck closed
closed