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

feat(dropdown): inline prop #863

Merged
merged 8 commits into from
Feb 8, 2019
Merged

feat(dropdown): inline prop #863

merged 8 commits into from
Feb 8, 2019

Conversation

bmdalex
Copy link
Collaborator

@bmdalex bmdalex commented Feb 7, 2019

feat(dropdown): inline prop

Description:

This PR implements the inline prop for Dropdown component.

There will be a follow-up PR with a prototype on how to handle the inline Dropdown in the context of a rich text editor

API

<Dropdown inline items={inputItems} placeholder="Select your hero" />

Styling

screen recording 2019-02-07 at 18 36 44

@bmdalex bmdalex added question Further information is requested, concerns that require additional thought are raised 🚀 ready for review labels Feb 7, 2019
@bmdalex bmdalex self-assigned this Feb 7, 2019
@codecov
Copy link

codecov bot commented Feb 7, 2019

Codecov Report

Merging #863 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #863   +/-   ##
=======================================
  Coverage   93.54%   93.54%           
=======================================
  Files          21       21           
  Lines         728      728           
  Branches       69       73    +4     
=======================================
  Hits          681      681           
  Misses         47       47

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 404b0e3...742d81e. Read the comment docs.

@bmdalex bmdalex force-pushed the feat/dropdown-inline branch 2 times, most recently from 63d99fa to b6dd43c Compare February 8, 2019 12:04
@bmdalex bmdalex merged commit a468d8f into master Feb 8, 2019
@layershifter layershifter deleted the feat/dropdown-inline branch February 8, 2019 14:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
question Further information is requested, concerns that require additional thought are raised 🚀 ready for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants