• 
      

    Add dark mode for infoboxes.

    Review Request #13824 — Created May 4, 2024 and submitted

    Information

    Review Board
    release-7.x

    Reviewers

    Infoboxes now work in dark mode, using a typical dark grey background
    and a light grey contrasting outline.

    This applies to all the variations of infoboxes. That involved fixing
    fade-out gradients on the review request and bug infoboxes, and the Ship
    It bar and issue status bar on the review request infoboxes.

    The Ship It and issue status areas don't use the same visual flair as on
    light mode, instead relegating the responsibility to the icon prefixes
    (of which there can be multiple for a given review request anyway).

    Tested all the infoboxes with all visual states in light and dark modes.

    Summary ID
    Add dark mode for infoboxes.
    Infoboxes now work in dark mode, using a typical dark grey background and a light grey contrasting outline. This applies to all the variations of infoboxes. That involved fixing fade-out gradients on the review request and bug infoboxes, and the Ship It bar and issue status bar on the review request infoboxes. The Ship It and issue status areas don't use the same visual flair as on light mode, instead relegating the responsibility to the icon prefixes (of which there can be multiple for a given review request anyway).
    343450bec2726a9b8a0143fdb0ded372339bc070

    Description From Last Updated

    I feel like this needs to be some kind of green to match the green of light mode. If this …

    daviddavid
    david
    1. 
        
    2. Show all issues

      I feel like this needs to be some kind of green to match the green of light mode. If this popped up I think it would be a miracle if I noticed that there were ship-its, whereas it's extremely prominent in light mode.

      1. Yeah it's not great, but I want to leave it as-is right now. I went back and forth on options and didn't settle on anything I was happy with. Even tried changing the text to say "Ship Its" and such, but then it became a problem to do that for issues, where the labels started to wrap the box in edge cases.

      2. OK, as long as we have this tracked somewhere.

    3. 
        
    david
    1. Ship It!
    2. 
        
    maubin
    1. Ship It!
    2. 
        
    chipx86
    david
    1. Ship It!
    2. 
        
    maubin
    1. Ship It!
    2. 
        
    chipx86
    Review request changed
    Status:
    Completed
    Change Summary:
    Pushed to release-7.x (45f9248)