nitin42 / react-imgpro

📷 Image Processing Component for React
2.18k stars 83 forks source link

Access img ref/reference #20

Closed darkowic closed 5 years ago

darkowic commented 6 years ago

Hello!

I need to access reference of the rendered img component. It is impossible right now.

We have 2 options:

What do you think? Since this component is rendering img component directly, I think the second solution will be more appropriate.

nitin42 commented 6 years ago

Seems doable! Thanks for the suggestion. Can you create a pull request for this ? @Sunshine168 this is interesting, what do you think ?

Sunshine168 commented 6 years ago

Absolutely, ref is useful, and recommend to use forwardRef Api.

Sunshine168 commented 6 years ago

~~BTW forwardRef Api is available in React 16.3~ You can use create-react-ref for this feat

Sunshine168 commented 6 years ago

@darkowic Did you like to create a PR about this feat ~ or I will work on this .

darkowic commented 6 years ago

I will handle this in few days, ok?

sob., 15 wrz 2018, 06:52 użytkownik ZiYingMai notifications@github.com napisał:

@darkowic https://github.com/darkowic Did you like to create a PR about this feat ~ or I will work on this .

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/nitin42/react-imgpro/issues/20#issuecomment-421530871, or mute the thread https://github.com/notifications/unsubscribe-auth/AK-jNe04OPSXyF0UGVp9WUkkK2iO7N11ks5ubIeJgaJpZM4Wg8gj .

Sunshine168 commented 6 years ago

@LukasDrgon Grateful~ I just make sure you take it

Sunshine168 commented 5 years ago

Hi any news about this ~

Sunshine168 commented 5 years ago

Done #29