Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[core] feat: new SegmentedControl component #6454

Merged
merged 11 commits into from
Nov 10, 2023
Merged

Conversation

CPerinet
Copy link
Contributor

@CPerinet CPerinet commented Oct 10, 2023

Fixes #3111

Changes proposed

  • Add SegmentedControl component and docs example

Screenshot

2023-11-08 11 22 40

@adidahiya
Copy link
Contributor

Segmented control

Build artifact links for this commit: documentation | landing | table | demo

This is an automated comment from the deploy-preview CircleCI job.

@adidahiya adidahiya changed the title Segmented control [core] feat: new SegmentedControl component Nov 8, 2023
@adidahiya
Copy link
Contributor

add docs blurb

Build artifact links for this commit: documentation | landing | table | demo

This is an automated comment from the deploy-preview CircleCI job.

@adidahiya adidahiya marked this pull request as ready for review November 8, 2023 18:22
@adidahiya adidahiya dismissed their stale review November 8, 2023 18:23

addressed review comments

@adidahiya
Copy link
Contributor

Merge remote-tracking branch 'origin/develop' into cp/segmented-controls

Build artifact links for this commit: documentation | landing | table | demo

This is an automated comment from the deploy-preview CircleCI job.

@adidahiya
Copy link
Contributor

Added support for small, regular, and large sizes (just like Buttons):

2023-11-09 18 34 06

Added support for "fill" prop:

image

Added support for "inline" prop. You can see it used in the example options panel; without this prop, the "intent" control container would appear like display: block and take up full width, which we don't want:

image

Also changed hover styling to take on the specified intent, I think this is slightly better, what do you think @CPerinet @jessperrin?

Before After
image image

@adidahiya
Copy link
Contributor

add fill, inline, small, large props

Build artifact links for this commit: documentation | landing | table | demo

This is an automated comment from the deploy-preview CircleCI job.

@adidahiya adidahiya merged commit 507f7c5 into develop Nov 10, 2023
12 checks passed
@adidahiya adidahiya deleted the cp/segmented-controls branch November 10, 2023 13:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SegmentedControl element
2 participants