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

Input component forward ref to HTML input element #296

Merged
merged 1 commit into from
Jan 11, 2021

Conversation

Jephuff
Copy link
Contributor

@Jephuff Jephuff commented Jan 11, 2021

This wraps Input in React.forwardRef so it can be used as a trigger in Popover
This enables us to do a POC of a combo box

Review with whitespace turned off.

📦 Published PR as canary version: 8.10.1-canary.296.7499.0

✨ Test out this PR locally via:

npm install @apollo/space-kit@8.10.1-canary.296.7499.0
# or 
yarn add @apollo/space-kit@8.10.1-canary.296.7499.0

@Jephuff Jephuff requested a review from a team January 11, 2021 20:17
@Jephuff Jephuff added the minor Increment the minor version when merged label Jan 11, 2021
Copy link
Member

@cheapsteak cheapsteak left a comment

Choose a reason for hiding this comment

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

lgtm
(wish this kind of change didn't require a version bump and npm install)

@Jephuff Jephuff merged commit 78dd69a into main Jan 11, 2021
@Jephuff Jephuff deleted the jeffrey/forward-ref-for-input branch January 11, 2021 20:29
@apollo-bot2
Copy link
Collaborator

🚀 PR was released in v8.11.0 🚀

@apollo-bot2 apollo-bot2 added the released This issue/pull request has been released. label Jan 11, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
minor Increment the minor version when merged released This issue/pull request has been released.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants