Skip to content
This repository has been archived by the owner on Aug 1, 2022. It is now read-only.

feat(ui): SegmentedControl component #377

Merged
merged 6 commits into from May 13, 2020
Merged

Conversation

cloudhead
Copy link
Contributor

@xla xla added feature Something that doesn't exist yet ui labels May 12, 2020
@xla xla changed the title Add "SegmentedControl" component feat(ui): SegmentedControl component May 12, 2020
@xla xla requested review from rudolfs and sarahscott May 12, 2020 14:44
@juliendonck
Copy link
Member

For context, this was a component I needed on both the issues page and the settings page. So I asked Alexis to pair on this with me.
Example:
Screenshot 2020-05-12 at 17 58 09
Screenshot 2020-05-12 at 17 58 01

@juliendonck juliendonck mentioned this pull request May 12, 2020
1 task
@rudolfs
Copy link
Member

rudolfs commented May 12, 2020

The tabs could use some space in between them.

Screenshot 2020-05-12 at 18 54 58

@cloudhead cloudhead marked this pull request as ready for review May 12, 2020 22:21
cloudhead and others added 2 commits May 13, 2020 10:02
@cloudhead cloudhead requested a review from rudolfs May 13, 2020 08:27
@cloudhead
Copy link
Contributor Author

The tabs could use some space in between them.

Fixed by only showing one bg at a time.

Copy link
Member

@rudolfs rudolfs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great! 🚢

@xla xla merged commit 6c92ebd into master May 13, 2020
@xla xla deleted the cloudhead/segmented-controls branch May 13, 2020 08:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature Something that doesn't exist yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants