Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(react-native): add picture-in-picture support for Android #1133

Merged
merged 17 commits into from
Oct 20, 2023

Conversation

santhoshvai
Copy link
Member

@santhoshvai santhoshvai commented Oct 6, 2023

picture-in-picture support for Android

  • Our Native module exposes methods and events to manage PiP in Android
  • Docs for PiP
  • Expo config plugin support
  • Do not disable camera when on PiP

@santhoshvai santhoshvai marked this pull request as draft October 6, 2023 14:51
@santhoshvai santhoshvai marked this pull request as ready for review October 11, 2023 12:25

if (props.androidPictureInPicture) {
const mainActivity = getMainActivityOrThrow(androidManifest);
('keyboard|keyboardHidden|orientation|screenSize|uiMode');
Copy link
Contributor

Choose a reason for hiding this comment

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

Does this do anything?

Copy link
Member Author

Choose a reason for hiding this comment

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

What this says is.. "dont kill my screen even if these things change.. I can handle it "

it is the android docs to do this if PiP is to be enabled

# Conflicts:
#	packages/react-native-sdk/expo-config-plugin/fixtures/index.ts
#	packages/react-native-sdk/expo-config-plugin/src/common/types.ts
#	packages/react-native-sdk/expo-config-plugin/src/index.ts
@santhoshvai santhoshvai merged commit ad313cc into main Oct 20, 2023
10 of 11 checks passed
@santhoshvai santhoshvai deleted the feat/pip branch October 20, 2023 11:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants