Skip to content

De-emphasize required approvals in sidebar

Annabel Dunstone Gray requested to merge assign-approver into master

What does this MR do and why?

  • De-emphasize the required approvals text
    • Gray text regardless of approvals being optional or not
    • Smaller font to help with information hierarchy. This is a new section in the sidebar and it doesn't seem to quite fit with the other sidebar blocks
  • Use btn-link for assign
  • Use - between text and button to match other sidebar "assign" states
  • Use lowercase for "assign" to match other states

Screenshots or screen recordings

Before After
Screenshot_2024-10-15_at_10.09.52_AM Screenshot_2024-10-15_at_10.31.35_AM
Screenshot_2024-10-15_at_10.08.57_AM Screenshot_2024-10-15_at_10.31.28_AM

How to set up and validate locally

  • Enable :reviewer_assign_drawer
  • Project > Code > Merge requests > Merge request, look at sidebar

Links

Edited by Michael Le

Merge request reports