-
Notifications
You must be signed in to change notification settings - Fork 348
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
[Bug] Camera mode #89
Comments
I think it's ok with camera app too, just camera app rotate image also so camera image appear to be twice rotated but not the camera app UI |
With my S10e I have noticed that:
I'm not very familiar with Android API but from what I read in OpenCamera's Manifet for Main Activity:
it seems to handle runtime configuration changes on their own as described in Android Developers documentation -> Handle configuration changes -> Handling the configuration change yourself. I guess there is no way for ScreenStream to detect rotation if (camera) app doesn't rotate its image? |
Yes, ScreenStream has no way of detecting what other apps do with rotation. |
I think autorotation in ScreenStream is not a problem here. Rather the other way around - if ScreenStream would be able to rotate output depending on device sensors rather than activity in the foreground? |
I am not a programmer, but can "screen-stream" know that which app is in foreground? |
@scrool It do rotate base on current device display rotation. |
What about a button to let us rotate manully? |
There is an option for that already. |
I'm still not sure if I have explained my issue correctly. So I have made a video with ScreenStream running on a phone and stream displayed on a monitor in the background. Background color of the stream is grey so you can see outline of actual image. On the monitor you can also see logcat displaying only messages that contain Then:
This is the ideal behaviour that I'd love to see also for other apps. Let's continue:
And finally:
Please see video here: https://youtu.be/xRXof3bDlGw Have we been on the same page before? To recap - I'd like ScreenStream to rotate the image based on device sensors rather than what oritentation active activity reports to the system. |
@scrool Rotate based on device sensors does not help much. I can lock into portrait mode and rotate phone. This will case wrong rotation detection. |
@scrool Why not just launch the application as per your requirement of "landscape" or "portrait" mode? |
I'm not sure, if I understand your point here. I don't want to lock into any mode.
I have one phone that I make a photo but I don't see application's viewfinder / preview. So I run ScreenStream there. On another (client) screen I display the stream. Rotation of phone with a camera and secondary screen are physicall in sync. Let's say I open Samsung's camera app. If I shoot in portrait mode everything is fine. But whenever I rotate phone to landscape mode I see small, 90 degree rotatated image on the (client) secondary screen where preview from the camera is rather hard to see and rotate in my brain to understand directions. So I (often) have to unmount phone camera, go to ScreenStream settings, choose correct rotation before sending to client so I can see image on the secondary display correctly rotated. With this feature I'd like to make faster switch between landscape and portrait mode.
My intention was to proove the point that this beahavior depends on (camera) application rather than ScreenStream wouldn't rotate image at all. |
See above - I rotate camera phone. On the top of that I most often use Samsung phone because it has different features and is simpler to use. |
@scrool The only thing I can propose is to add option in app settings to disable rotation detection at all. Will it make you happy? |
I'm afraid that disable of rotation detection wouldn't help me. |
I like that the screen and video autorotates but It would be nice to define it on settings because the auto rotation works fine everywhere except for the camera app? I see the logos changing rotation but the video streaming isn't
Build: 3.4.6
Android: 10
Phone: Samsung Galaxy S10
The text was updated successfully, but these errors were encountered: