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

feat: Add floating label to select #73

Merged
merged 5 commits into from
Sep 10, 2019
Merged

feat: Add floating label to select #73

merged 5 commits into from
Sep 10, 2019

Conversation

cehsu
Copy link
Contributor

@cehsu cehsu commented Sep 10, 2019

These changes bring the select component up to date with other form elements that support floating labels.

Screen Shot 2019-09-10 at 10 01 34 AM

Task: https://app.asana.com/0/1127722639002160/1135318045362828/f

@cehsu cehsu requested a review from a team September 10, 2019 00:44
@ghost ghost requested review from choochootrain and schlegz September 10, 2019 00:44
@codecov
Copy link

codecov bot commented Sep 10, 2019

Codecov Report

Merging #73 into v5 will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##               v5      #73   +/-   ##
=======================================
  Coverage   66.66%   66.66%           
=======================================
  Files          20       20           
  Lines         450      450           
  Branches       91       91           
=======================================
  Hits          300      300           
  Misses        120      120           
  Partials       30       30

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 c5e680b...9731b19. Read the comment docs.

@kylealwyn
Copy link
Contributor

Could we have this go into v5, which will be landing soon, so we don’t have to worry about any conflicts?

Copy link
Contributor

@schlegz schlegz left a comment

Choose a reason for hiding this comment

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

LGTM! 👍

src/Form/Select.js Outdated Show resolved Hide resolved
@cehsu cehsu changed the base branch from master to v5 September 10, 2019 16:36
@cehsu cehsu force-pushed the feat/floatable-select branch from 96537ab to 662cd87 Compare September 10, 2019 16:55
@cehsu cehsu requested a review from choochootrain September 10, 2019 17:08
Copy link

@choochootrain choochootrain left a comment

Choose a reason for hiding this comment

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

✈️

@cehsu cehsu merged commit d8f506a into v5 Sep 10, 2019
@cehsu cehsu deleted the feat/floatable-select branch September 10, 2019 19:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants