-
Notifications
You must be signed in to change notification settings - Fork 9.8k
Camera plugin first working version (IOS and Android). #458
Conversation
…s more gracefully. Handles app going to background. Prevent creation of empty video files.
…ermission issue with MIC on iOS. Added support to record to file path sent by flutter on iOS.
…amera-dev Fixes: Removed start stop on flutter iOS and dummy calls on Android. Fixed permission issue with MIC on iOS. Added support to record to file path sent by flutter on iOS.
We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for all the commit author(s) or Co-authors. If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google. |
@googlebot I've signed the Google Corporate CLA agreement on [email protected] |
Will there be a native UI version too? |
When I run the example on a Google Pixel everything is normal. On an iPhone 6s the first video you take is not in the preview window in the bottom and the second video is. Then every other video you take shows up in the preview window otherwise it shows white after recording. |
@AppleEducate What do you mean by native UI version? |
@sigurdm Native UI as in the stock camera apps on Android and iOS. That would be the full screen camera app that takes care of the flash, camera modes and selecting the camera for you. This plugin is great In Giving you access to the raw camera but it would be nice to also provide a way to ask the user for a photo or video and they can choose on from the Library or take a new one like the image picker does. It navigates to the Stock Camera application on the phone and navigates back with the file. |
@AppleEducate I think that's outside the responsibilities of this plugin. Otherwise please file a new issue at https://github.com/flutter/flutter/issues/new to have track of the discussion in the right place. |
@sigurdm I think you're right. This may be something for a media_picker plugin. |
@sigurdm @AppleEducate I've been looking into the bug with the IOS version, I tracked it down to an issue in the video player plugin. More specifically, an |
I created a new PR with a fix for the issue: PR #557 |
Fix a bug with video recording sound in IOS
@sigurdm I was able to create the plugin I was describing. Capture and Select both Photos and Videos from the Gallery and with the camera using the stock camera apps. I took the image_picker plugin and added video capability. What should I do now? Should I create a video_picker plugin and media_picker plugin that have a dependency on image_picker and video_picker or just bring all the code into media_picker? |
as @sigurdm mentioned in his PR, the aspect ratio is wired in iOS, specially if running in fullscreen and on a iPhone X. Looks like the aspect ratio of the video controller is not correctly calculated. |
packages/camera/example/pubspec.yaml
Outdated
@@ -7,6 +7,7 @@ dependencies: | |||
sdk: flutter | |||
camera: | |||
path: ../ | |||
video_player: "0.5.1" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This needs to be 0.5.2
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Besides the need to use video_player 0.5.2 in the example pubspec which includes the fix, this LGTM
You can add yourself to the AUTHORs file if you want.
@sigurdm Awesome! I updated the pubspec and the authors. Thank you for your review :) |
Thanks for all the hard work! |
how do I use the camera plugin to record video |
Hi Norbert.
There is a big example of the plugin with video recording in the example
folder:
https://github.com/flutter/plugins/blob/master/packages/camera/example/lib/main.dart
2018-08-13 20:10 GMT+01:00 Norbert Aberor <[email protected]>:
… how do I use the camera plugin to record video
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#458 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/APGcdWOZ8WLOgjhZH4P-YS7QVf-hthBbks5uQc8hgaJpZM4TE46p>
.
|
Some backward incompabilities introduced. See CHANGELOG. Version bump to 0.2.0
Can i get to know how to mute the mic while recording? |
Can you please add some more functionality |
Ready for review.