[Merge Request page] Table header background and border colors in Code Owners widget are not using design tokens

Background

This issue represents a finding from our work to audit features belonging to groupcode review. See the analysis in Figma for more information. See the overview video for guidance on how to review this analysis.

Problem to solve

  • Border colors in Code Owners widget are not using design tokens
  • Header background colors in Code Owners widget are not using design tokens
  • This will prevent the Merge Request page view from adapting properly to dark mode.

Proposal

  • Address the finding on the Merge Request page view. The specific solution may require discussion between design and engineering.
  • Designers from groupdesign system have the following guidance:
    • These borders aren't using a design token and could use border.color.default instead.
    • The backgrounds for these rows aren't using design tokens.
    • Based on what other rows do in the context of an MR widget these might not require a background color assigned.
    • The text color should probably be text.color.strong instead of subtle since these are also elements.
  • View finding yellow#15 in Figma
  • View finding yellow#14 in Figma

Screenshots

Screenshot_2025-03-03_at_11.43.24_AM

Screenshot_2025-03-03_at_11.43.08_AM

Edited by Scott de Jonge