Skip to content
This repository has been archived by the owner on Mar 4, 2020. It is now read-only.

feat(segment): color prop #632

Merged
merged 5 commits into from
Dec 20, 2018
Merged

feat(segment): color prop #632

merged 5 commits into from
Dec 20, 2018

Conversation

bmdalex
Copy link
Collaborator

@bmdalex bmdalex commented Dec 18, 2018

feat(segment): color prop

Description

This PR:

  • adds color prop to Segment component
  • creates Segment color examples

API

<Segment color={COLOR} .../>

where COLOR is one of 'primary' | 'secondary' | 'blue' | 'green' | 'grey' | 'orange' | 'pink' | 'purple' | 'teal' | 'red' | 'yellow' | string

screenshot 2018-12-18 at 16 03 20

Known issues:

Segment text color should be dark shade of gray on segments with light background color, tracked by #629

@bmdalex bmdalex force-pushed the feat/segment-color-prop branch 2 times, most recently from 420d71f to e1448bd Compare December 19, 2018 15:50
@alinais alinais added this to Bugaa92 in Core Team Dec 19, 2018
Copy link
Member

@layershifter layershifter left a comment

Choose a reason for hiding this comment

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

LGTM

@kuzhelov kuzhelov added the needs author feedback Author's opinion is asked label Dec 20, 2018
@bmdalex bmdalex force-pushed the feat/segment-color-prop branch 2 times, most recently from 7466bb9 to 0048e09 Compare December 20, 2018 13:42
@bmdalex bmdalex added ready for merge and removed needs author feedback Author's opinion is asked labels Dec 20, 2018
@bmdalex bmdalex merged commit d5f5dc1 into master Dec 20, 2018
@bmdalex bmdalex deleted the feat/segment-color-prop branch December 20, 2018 14:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
No open projects
Core Team
  
Bugaa92
Development

Successfully merging this pull request may close these issues.

None yet

5 participants