Skip to content

Use badge component for "busy" status

Annabel Dunstone Gray requested to merge busy-badge into master

What does this MR do and why?

  • Use badge component for busy status
  • Why? It looks cleaner and draws attention to the status

What does this MR not do?

  • Update when/where the busy status shows up (for example, it doesn't show up on the Reviewers section in MR sidebar)
  • Update the width of the user dropdown. If the username is on the longer side, you may not see the Busy badge at all (existing functionality)
  • Change tooltips- they'll still say Username (Busy)

TODO:

  • MR/issue dropdown
  • Issue board dropdown
  • User popover
  • Review areas where the badge doesn't show up (for example, why Assignee and notReviewers in MRs? Why does it not show up in the Issue sidebar at all? Separate issue
  • Is warning the correct variant?

Screenshots or screen recordings

Before After
Screenshot_2023-04-07_at_2.46.50_PM Screenshot_2023-04-07_at_2.43.51_PM
Screenshot_2023-04-07_at_2.47.01_PM Screenshot_2023-04-07_at_2.43.28_PM
Screenshot_2023-04-07_at_2.47.13_PM Screenshot_2023-04-07_at_2.43.41_PM
Screenshot_2023-04-10_at_10.28.36_AM Screenshot_2023-04-10_at_10.13.12_AM
Screenshot_2023-04-10_at_11.41.46_AM Screenshot_2023-04-10_at_11.41.23_AM
Screenshot_2023-04-11_at_3.38.00_PM Screenshot_2023-04-11_at_3.35.58_PM

How to set up and validate locally

#394879 (closed)

#321466 (closed)

Edited by Annabel Dunstone Gray

Merge request reports

Loading