Skip to content
This repository has been archived by the owner on Sep 19, 2024. It is now read-only.

Update packages #2

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Update packages #2

wants to merge 6 commits into from

Conversation

KevinDoom
Copy link

Hey guys,

I made a few changes to this package. Mainly, I wanted to retrieve some values as your CCCameraOrientation values, since that's what the camera seems to mainly use.

  • Added getInitialOrientationInt() — Just like getInitialOrientation, but returns a CCOrientation value instead of a string.
  • Added getOrientationAsCCInt() — Just like getOrientation() but returns a CCOrientation value instead of a device-specific int.
  • Removed @synthesize bridge = _bridge; from Orientation.m — I kept getting errors saying that you shouldn't synthesize the bridge when extending from RCTEventEmitter.

@jasongaare we'll probably have to get Matt to look at some changes I made to Orientation.m. Not real strong on Objective-C. Let me know if you see any problems.

Attempting to fix an error on iOS that said brindge should bot be synthesized since one is inherited from RCTEventEmitter.
-- Allows async get of orientation as our enum values
@jasongaare jasongaare removed their request for review February 14, 2020 17:54
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.

2 participants