Skip to content

Update styling of pending comment

Annabel Dunstone Gray requested to merge pending-comment into master

What does this MR do and why?

  • Adds border to pending comments to help draw attention to them
    • The current background color doesn't stand out and may not even be visible to some users depending on vision/screen/etc
  • Updates selected line background/border to be blue instead of orange

Screenshots or screen recordings

Before After
Screenshot_2023-03-29_at_10.50.45_AM Screenshot_2023-03-29_at_10.50.02_AM
Before After
Screenshot_2023-03-28_at_2.21.48_PM Screenshot_2023-03-28_at_2.21.23_PM

How to set up and validate locally

  • Go to MR -> Changes tab
  • Start a review on a line of code

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Annabel Dunstone Gray

Merge request reports