Skip to content

Global search results merge request badge icons don't match Pajamas style guide

The following discussion from !46944 (merged) should be addressed:

  • @nickbrandt started a discussion: (+1 comment)

    @DylanGriffith I saw recently that this was merged and I have a follow-up question regarding this.

    It looks like we are using a different variant of the badge component for Merged results than what we have available and documented in Pajamas (also seen here in Figma). Similar to how we mapped Closed issues to the info badge style, I think we should do the same here for Merged.

    closed merged
    status box on pages Screen_Shot_2020-11-17_at_10.59.16_AM Screen_Shot_2020-11-17_at_10.59.10_AM
    badge in global search Screen_Shot_2020-11-17_at_11.00.43_AM Screen_Shot_2020-11-17_at_10.58.48_AM

    There will be updates coming to Issues and MRs to update the current status indicators on those pages to use these badges as well (like what you see here) in the future.

Proposal

Modify the Merged badge to utilize the info variant.

Edited by Nick Brandt