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

Use getPhysicalDisplayToken if getBuiltInDisplay is not found #587

Merged
merged 1 commit into from
Jun 14, 2019

Conversation

schwabe
Copy link
Contributor

@schwabe schwabe commented Jun 12, 2019

This makes the -S (screen off) parameter work on Android Q beta 4

Closes #586

@schwabe
Copy link
Contributor Author

schwabe commented Jun 12, 2019

I also tested it on a Android P Device to check for regressions

@rom1v
Copy link
Collaborator

rom1v commented Jun 12, 2019

Cool, I'm glad you implemented it 👍

Would checking Build.VERSION.SDK_INT work instead? But anyway, it's maybe more reliable by checking if the method exists like you did.

@schwabe
Copy link
Contributor Author

schwabe commented Jun 12, 2019

Yes checking Build.VERSION.SDK_INT should work too. I can update the patch tomorrow with that.

GulajavaMinistudio pushed a commit to javascript-indonesias/scrcpy that referenced this pull request Jun 12, 2019
@schwabe schwabe force-pushed the fix_586_screen_off_qbeta branch 2 times, most recently from 26ace0e to e46fe51 Compare June 13, 2019 09:15
@schwabe
Copy link
Contributor Author

schwabe commented Jun 13, 2019

Okay, updated commit und tested with SDK INT. Still works fine on my Pixel device (Android P) and my Pixel 3 (Q beta 4).

This might on betas between but who cares?!

This makes the -S (screen off) parameter work on Android Q beta 4

Closes Genymobile#586
@rom1v rom1v merged commit b29a568 into Genymobile:master Jun 14, 2019
@rom1v
Copy link
Collaborator

rom1v commented Jun 14, 2019

Thank you 👍

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.

screen off function -S does not work Android Q beta 4 devices
2 participants