Skip to content
This repository was archived by the owner on Feb 22, 2023. It is now read-only.

[camera] Fixes #28350#1261

Merged
bparrishMines merged 4 commits into
flutter-team-archive:masterfrom
p30arena:master
Mar 6, 2019
Merged

[camera] Fixes #28350#1261
bparrishMines merged 4 commits into
flutter-team-archive:masterfrom
p30arena:master

Conversation

@p30arena

@p30arena p30arena commented Feb 23, 2019

Copy link
Copy Markdown
Contributor

flutter/flutter#28350
Adds sensorOrientation

Adds sensorOrientation
@p30arena p30arena changed the title Fixes #28350 Fixes https://github.com/flutter/flutter/issues/28350 Feb 23, 2019
@p30arena p30arena changed the title Fixes https://github.com/flutter/flutter/issues/28350 Fixes #28350 Feb 23, 2019

@bparrishMines bparrishMines left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

Thanks for the contribution!

Your code was good, so i added a part for iOS. I believe all their sensors are rotated 90 based on this documentation.

I also added some documentation! I'll merge once the tests pass again.

@bparrishMines bparrishMines added feature submit queue The Flutter team is in the process of landing this PR. labels Mar 6, 2019
@bparrishMines bparrishMines changed the title Fixes #28350 [camera] Fixes #28350 Mar 6, 2019
@p30arena

p30arena commented Mar 6, 2019

Copy link
Copy Markdown
Contributor Author

@bparrishMines
ah I totally forgot about iOS, Thanks!

@bparrishMines bparrishMines merged commit 87e2bdc into flutter-team-archive:master Mar 6, 2019
jonasbark pushed a commit to jonasbark/plugins that referenced this pull request Mar 11, 2019
* commit '9017d6e7f867af278edd7e8e584d52524f37443f': (408 commits)
  [webview_flutter]Allow specifying a navigation delegate(Android and Dart). (flutter-team-archive#1236)
  Allow specifying a navigation delegate (iOS implementation). (flutter-team-archive#1323)
  Change build link in contributors site to cirrus (flutter-team-archive#1327)
  [image_picker] Update versioning for flutter-team-archive#1268 (flutter-team-archive#1326)
  [image_picker] remove unnecessary camera permmision (flutter-team-archive#1268)
  Exclude longPress from semantics (flutter-team-archive#1324)
  [in_app_purchase] refactoring and tests (flutter-team-archive#1322)
  [in_app_purchase] Adds Dart BillingClient APIs for loading purchases (flutter-team-archive#1286)
  [connectivity] Update README.md (flutter-team-archive#1201)
  [camera] Fixes #28350 (flutter-team-archive#1261)
  [cloud_functions] Specify version for CocoaPod and handle null regions gracefully (flutter-team-archive#1316)
  [in_app_purchase]retrieve receipt (flutter-team-archive#1303)
  [firebase_analytics] Add resetAnalyticsData method (flutter-team-archive#1311)
  trackCameraPosition is inferred from GoogleMap.onCameraMove (flutter-team-archive#1314)
  [google_maps_flutter]ChangeNotifier is replaced with granular callbacks (flutter-team-archive#1302)
  [video_player]Do not divide by zero (flutter-team-archive#793)
  [firebase_dynamic_links] Version bump for firebase_dynamic_links PR flutter-team-archive#1142 (flutter-team-archive#1309)
  [firebase_dynamic_links] fix dynamic link crash when creating shortlink if warnings are null (flutter-team-archive#1142)
  Fix typo in RewardedVideoAdd sample (flutter-team-archive#927)
  Add my name to firebase_performance and firebase_dynamic_links owners (flutter-team-archive#1300)
  ...

# Conflicts:
#	packages/camera/android/src/main/java/io/flutter/plugins/camera/CameraPlugin.java
#	packages/camera/ios/Classes/CameraPlugin.m
#	packages/firebase_auth/ios/Classes/FirebaseAuthPlugin.m
#	packages/image_picker/android/build.gradle
#	packages/video_player/android/src/main/java/io/flutter/plugins/videoplayer/VideoPlayerPlugin.java
Akachu pushed a commit to Akachu/flutter_camera that referenced this pull request Apr 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bugfix cla: yes feature submit queue The Flutter team is in the process of landing this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants