carbon-design-system / carbon

A design system built by IBM
https://www.carbondesignsystem.com
Apache License 2.0
7.86k stars 1.82k forks source link

[Ghost button] Add support for the selected state (pushed) #7190

Closed cheer021 closed 3 years ago

cheer021 commented 4 years ago

Use this template if you want to request a new feature, or a change to an existing feature.

If you are reporting a bug or problem, please use the bug template instead.

Summary

Please describe your request in one or two sentences.

Clarify if you are asking for design, development, or both design and development.

Justification

Provide the business reasons for this request.

Desired UX and success metrics

Describe the full user experience for this feature. Also define the metrics by which we can measure success/failure for the user.

"Must have" functionality

Highlight any "must have" needs and functionality for the request.

This should not be a full list of functionality; the Carbon team will work with you to define functionality based on the desired UX.

Specific timeline issues / requests

Do you want this work within a specific time period? Is it related to an upcoming release?

NB: The Carbon team will try to work with your timeline, but it's not guaranteed. The earlier you make a request in advance of a desired delivery date, the better!

Available extra resources

What resources do you have to assist this effort?

Carbon is a collaborative system. We encourage teams to build components and submit them for integration as either add-ons or core components.

tw15egan commented 4 years ago

Related https://github.com/carbon-design-system/carbon/issues/7157

jnm2377 commented 3 years ago

Maybe I'm missing something, but is this not what you're looking for?

Screen Shot 2021-04-23 at 9 16 01 AM
tw15egan commented 3 years ago

@jnm2377 I believe it's pretty strict in regards to which variants it can be used for, it looks like it's only allowed on icon-only ghost buttons

@carbon-design-system/design is this something we would want to extend to other variants?

aagonzales commented 3 years ago

Yes, its only allowed for icon-only ghost button for scenarios like a tool bar.

Most of the time buttons are used to trigger actions instantly and don't required a selected state. One exception we have found is the toolbar scenario but the icon only button. @cheer021 What is the use-case for needing a regular button to have a selected state? Could you provide an in-context image?

vpicone commented 3 years ago

Going to close this one out, let us know if something changes. Also recommend checking out the Toggle component.