Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: [FX-377] Proxy ref from forwardRef to FileInput #621

Merged
merged 1 commit into from
Aug 23, 2019

Conversation

denieler
Copy link
Collaborator

@denieler denieler commented Aug 23, 2019

FX-377

Description

Make ref to be proxied from forwardRef for FileInput and merged with the currently used ref inside the component.

Just will leave it here - facebook/react#13029
it was difficult to find this info

@denieler denieler requested a review from a team August 23, 2019 10:19
Copy link
Contributor

@havenchyk havenchyk left a comment

Choose a reason for hiding this comment

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

LGTM

@MilosMosovsky MilosMosovsky force-pushed the fx-move-to-v4 branch 2 times, most recently from ab83602 to 3c59056 Compare August 23, 2019 10:37
@denieler denieler changed the title feat: [FX-377] Add proxying ref from forwardRef to FileInput feat: [FX-377] Proxy ref from forwardRef to FileInput Aug 23, 2019
@toptal-devbot
Copy link
Collaborator

🎉 Last commit is successfully deployed 🎉

Demo is available on:

Your davinci-bot 🚀

@denieler denieler merged commit 9519e5e into fx-move-to-v4 Aug 23, 2019
@denieler denieler deleted the fx-377-proxy-ref-file-input branch August 23, 2019 11:12
conf pushed a commit that referenced this pull request Aug 23, 2019
# [3.0.0](v2.27.1...v3.0.0) (2019-08-23)

### Bug Fixes

* **Accordion:** fix Accordion styles ([#592](#592)) ([c2636b9](c2636b9))
* **Accordion:** fix an issue with no children rendered ([b9d198e](b9d198e))
* **CssBaseline:** [FX-368] Use our own css baseline ([#595](#595)) ([bf9052c](bf9052c))
* **Grid:** divide Picasso grid spacing according to MUI ([#586](#586)) ([41b9783](41b9783))
* **paper:** fix default text color ([#594](#594)) ([5522976](5522976))
* **Select:** issues after upgrade to MUI v4 ([#590](#590)) ([7c34698](7c34698))
* **Slider:** [FX-378] Move Slider from lab to core ([#596](#596)) ([58e33b9](58e33b9))
* **Table:** adjust cell paddings ([#591](#591)) ([550d901](550d901))
* **Tabs:** after upgrade to v4 ([#593](#593)) ([07dcef8](07dcef8))
* fix disabled state for Checkbox and Radio ([#587](#587)) ([bc73cea](bc73cea))
* **Tooltip:** issues after upgrade to v4 ([#581](#581)) ([65676e0](65676e0))
* **Typography:** fix typography, baseline visual issues ([#585](#585)) ([68e8aec](68e8aec))

### chore

* **TextField:** remove deprecated props and rename to Input ([#606](#606)) ([1d63602](1d63602))
* rename `justifyContent` prop in Grid to match Container ([#600](#600)) ([8da9a02](8da9a02))
* **Icon:** delete deprecated icon components ([#598](#598)) ([b37be08](b37be08))
* **Icon:** remove deprecated prop `size` from icons ([#599](#599)) ([033a5ac](033a5ac))

### Features

* [FX-377] Forward refs for all components ([#612](#612)) ([677af2e](677af2e))
* [FX-377] Proxy ref from forwardRef to FileInput ([#621](#621)) ([54668a3](54668a3))
* upgrade MUI to v4 ([29d14bd](29d14bd))

### BREAKING CHANGES

* **TextField:** TextField component to Input
* **Slider:** Slider component has been moved from `@toptal/picasso/lab` to `@toptal/picasso`
* rename `justify` prop in `Grid`  to `justifyContent`
* **Icon:** remove deprecated prop `size` from icons
* **Icon:** icon components deprecated in v2 have been fully removed in v3. To obtain support migrating these components, check: https://toptal-core.atlassian.net/wiki/x/boACG
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants