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

Add select input component #7

Merged
merged 6 commits into from
Apr 15, 2015
Merged

Add select input component #7

merged 6 commits into from
Apr 15, 2015

Conversation

synbot
Copy link

@synbot synbot commented Apr 14, 2015

Acceptance Criteria

  • Add base select form element
  • add .select to input mixin
  • add props to have 'custom select' with customizable icon arrow and container styles (see trutankless-crm select component)
  • add mixin for custom select

@natestormer natestormer self-assigned this Apr 13, 2015
@natestormer natestormer added this to the Sprint ∞ milestone Apr 13, 2015
@natestormer
Copy link
Contributor Author

Ⓜ️

@kylecesmat
Copy link
Contributor

🌟

@synbot synbot added S-Review and removed S-Open labels Apr 14, 2015
@natestormer
Copy link
Contributor Author

✋

natestormer
natestormer reviewed Apr 14, 2015
View reviewed changes
components/form/inputs/select.jsx
value : null,
onFocus : null,
onBlur : null,
onChange : null,
Copy link
Contributor

Choose a reason for hiding this comment

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

add componentIcon propType

@natestormer
Copy link
Contributor Author

⁉️ just a few small things

@kylecesmat
Copy link
Contributor

🌟

@synbot synbot added S-Review and removed S-Open labels Apr 15, 2015
@natestormer
Copy link
Contributor Author

👍

natestormer added a commit that referenced this pull request Apr 15, 2015
@natestormer
Merge pull request #7 from synapsestudios/design/7-add-select-input
8b4258a 
Add select input component
@natestormer natestormer merged commit 8b4258a into master Apr 15, 2015
@natestormer natestormer deleted the design/7-add-select-input branch April 15, 2015 00:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@kylecesmat kylecesmat

Labels
C-Low P-Medium S-Accepted T-Design
Projects
None yet
Milestone
Sprint ∞
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@synbot @natestormer @kylecesmat @bobeagan