Skip to content
This repository has been archived by the owner on Jun 16, 2023. It is now read-only.

WIP: Implement video stabilization mode property for ios #1606

Merged

Conversation

n1ru4l
Copy link
Collaborator

@n1ru4l n1ru4l commented Jun 4, 2018

This pull request allows setting the property videoStabilizationfor iOS devices.

This is required for smoother video recordings.

@n1ru4l n1ru4l requested a review from jgfidelis June 4, 2018 07:33
@jgfidelis
Copy link
Collaborator

@n1ru4l good job, can you resolve the conflicts please?

@sibelius sibelius merged commit a090faa into react-native-camera:master Jun 5, 2018
@jgfidelis
Copy link
Collaborator

@n1ru4l you did not set a defaultProp, isn't this required?

@@ -441,6 +441,13 @@ - (void)record:(NSDictionary *)options resolve:(RCTPromiseResolveBlock)resolve r
[self updateSessionAudioIsMuted:!!options[@"mute"]];

AVCaptureConnection *connection = [self.movieFileOutput connectionWithMediaType:AVMediaTypeVideo];
if (self.videoStabilizationMode != nil) {
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@jgfidelis I think this should handle the case?

luugiathuy added a commit to luugiathuy/react-native-camera that referenced this pull request Jun 17, 2018
* master: (188 commits)
  1.1.5
  build(change-log): v1.1.5-2
  Mock module and test a custom component using Jest (react-native-camera#1635)
  fix(rn-camera): fix codec backwards compat
  update zxing lib version
  [BUG, ANDROID] width prop not works correctly when fixOrientation is enabled (react-native-camera#1629)
  Update GradleUpgradeGuide.md (react-native-camera#1625)
  Docs around pause/resume preview (react-native-camera#1623)
  Added pause/resume preview functions to RNCamera (react-native-camera#1622)
  fix(picture-size): create None default value
  fix(rn-camera): fix types, conversions and casts
  WIP: Implement video stabilization mode property for ios (react-native-camera#1606)
  fix(picture-size): export method and change default value
  docs(expo): explain how to migrate to and from expo camera module (react-native-camera#1605)
  feat(preview): add android code
  feat(preview): add new props to JS
  ios-get-video-orientation-from-device-2
  feat(preview): add preview methods and more fixes
  Update index.d.ts
  Bug/save to capture when app goes on pause (react-native-camera#1578)
  ...
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.

None yet

3 participants