Fix the color of field section labels for diffs in dark mode.

Review Request #13823 — Created May 3, 2024 and submitted

Information

Review Board
release-7.x

Reviewers

The Diff Revision, Commits, and Files section headers in the diff viewer
page had a dark grey background, which conflicted with the background
and didn't match the styling of the rest of the page.

We unfortunately don't have any standard classes for these that we can
depend on. This should be addressed, but not for this release, as it'd
introduce too many structural changes that aren't worth fixing up right
now.

In the meantime, we're now referencing the field label styling via CSS
variables. We're also using the proper font size for section labels,
which is smaller than we had before, but more consistent with the rest
of the page.

Tested the review request and diff viewer pages in light and dark mode,
and compared the styling to the other section headers.

Summary ID
Fix the color of field section labels for diffs in dark mode.
The Diff Revision, Commits, and Files section headers in the diff viewer page had a dark grey background, which conflicted with the background and didn't match the styling of the rest of the page. We unfortunately don't have any standard classes for these that we can depend on. This should be addressed, but not for this release, as it'd introduce too many structural changes that aren't worth fixing up right now. In the meantime, we're now referencing the field label styling via CSS variables. We're also using the proper font size for section labels, which is smaller than we had before, but more consistent with the rest of the page.
be7322bb2d615933f36947b7f5149f6bcc9fea45

maubin
  1. Ship It!
  2. 
      
david
  1. Ship It!
  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-7.x (dc96ca4)
Loading...