Skip to content

Pt 2: Add outlined chip option to chip component #1153

Open
@natdink

Description

@natdink
Contributor

Problem (Why)

Once the chip component updates have been made to the filled version we should work on adding back in the outlined chip.

Impact (Who)

Designers and devs, eventually end-users.

Existing

Pooja - you can pull the originals from this file though you will have to go in and make sure to update the colors to our color variables.

https://www.figma.com/design/5Mscyan2i1AHe68jzY1W8m/MUI-Core-v5.4.0?m=auto&node-id=6588-47646&t=jB0TmQFrvXzeQGm5-1

Requirements (What)

What are more specific requirements that needs to take to consider this issue resolved.

  • Create a new branch
    Add to chip page as a second option - the component itself should be called Chip/Outlined

Activity

added theissue type on May 15, 2025
natdink

natdink commented on May 15, 2025

@natdink
ContributorAuthor

to be assigned to @Pooja-1801 once part 1 is complete #1146

changed the title [-]Add outlined chip option to chip component[/-] [+]P2: Add outlined chip option to chip component[/+] on May 15, 2025
changed the title [-]P2: Add outlined chip option to chip component[/-] [+]Pt 2: Add outlined chip option to chip component[/+] on May 15, 2025
Pooja-1801

Pooja-1801 commented on May 29, 2025

@Pooja-1801

Created the outline chip component and its variants for both light and dark modes, including default and hover states.

Image
Image

Pooja-1801

Pooja-1801 commented on Jun 2, 2025

@Pooja-1801

hover effect does not look different from the default, can you play around with these colors?

Image

As discussed on the call, I will be working on the chip component to try out different color shades."
Or slightly more concise:

Pooja-1801

Pooja-1801 commented on Jun 2, 2025

@Pooja-1801

@bfekadu21 I’ve tried a similar shade of dark blue to match the others. For Default and Hover. Let me know your feedback on this.

Default:

Image

Hover:

Image

bfekadu21

bfekadu21 commented on Jun 2, 2025

@bfekadu21

@Pooja-1801 Can you update the hover state to be filled, similar to other hover chip designs?

Pooja-1801

Pooja-1801 commented on Jun 3, 2025

@Pooja-1801

@bfekadu21 Updated Hover chip design

Image

Pooja-1801

Pooja-1801 commented on Jun 3, 2025

@Pooja-1801

@bfekadu21 For the default chip, a high-contrast version exists, but not a low-contrast one. Do we need to create it?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

Projects

No projects

Milestone

No milestone

Relationships

None yet

    Development

    No branches or pull requests

      Participants

      @natdink@Pooja-1801@bfekadu21

      Issue actions

        Pt 2: Add outlined chip option to chip component · Issue #1153 · SEMOSS/semoss-ui