react-native-responsive-screen icon indicating copy to clipboard operation
react-native-responsive-screen copied to clipboard

will RNRS support functional components anytime soon? this.setState undefined error is still here

Open sammysium opened this issue 4 years ago • 3 comments

I have seen two issues, at least, #65 and #82 . My component is functional but the View is not responsive to orientation change. Using the listners causes this.setState is not defined. Given a couple of PR attempts that have not yet been merged in yet, is there a plan to handle this issue please? It is sort of a deal breaker for such a great library.

sammysium avatar Dec 23 '20 13:12 sammysium

My pull request adds support for functional components and a working example: https://github.com/marudy/react-native-responsive-screen/pull/83

Greg Fenton.

On Wed., Dec. 23, 2020, 08:56 sammysium, [email protected] wrote:

I have seen two issues, at least, #65 https://github.com/marudy/react-native-responsive-screen/issues/65 and #82 https://github.com/marudy/react-native-responsive-screen/issues/82 . My component is functional but the View is not responsive to orientation change. Using the listners causes this.setState is not defined. Given a couple of PR attempts that have not yet been merged in yet, is there a plan to handle this issue please? It is sort of a deal breaker for such a great library.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/marudy/react-native-responsive-screen/issues/92, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABBUCB3PSFWBM3YVVJ5OPBLSWHZIVANCNFSM4VHATUKQ .

gregfenton avatar Dec 23 '20 15:12 gregfenton

@gregfenton i saw it. will it be merged or we gotta stick to your PR?

sammysium avatar Dec 23 '20 15:12 sammysium

I had missed that @marudy is asking for maintenance helpers: https://github.com/marudy/react-native-responsive-screen/commit/94a632049d27018ac9786cee8b5e03d6d87813a8

Greg Fenton.

On Wed., Dec. 23, 2020, 10:48 sammysium, [email protected] wrote:

@gregfenton https://github.com/gregfenton i saw it. will it be merged or we gotta stick to your PR?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/marudy/react-native-responsive-screen/issues/92#issuecomment-750358228, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABBUCB2DCPSGOIIPM5MN3WLSWIGLVANCNFSM4VHATUKQ .

gregfenton avatar Dec 23 '20 15:12 gregfenton