-
Notifications
You must be signed in to change notification settings - Fork 302
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
Adding External Screen Support [Feature Request #11] #411
base: master
Are you sure you want to change the base?
Conversation
@@ -61,8 +90,9 @@ - (void)viewDidLoad | |||
|
|||
_controllerSupport = [[ControllerSupport alloc] initWithConfig:self.streamConfig presenceDelegate:self]; | |||
_inactivityTimer = nil; | |||
|
|||
_renderView = (StreamView*)[[UIView alloc] initWithFrame:self.view.frame]; |
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.
It's not safe to cast a UIView to a StreamView.
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.
@cgutman What would you suggest to be a better alternative? Should it be a split to a #IF os(iOS)
? Is this due to StreamView being a possible OSView for non iOS devices?
if (UIScreen.screens.count > 1) { | ||
[self prepExtScreen:UIScreen.screens.lastObject]; | ||
} | ||
else { |
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.
Looks like the indentation got a bit mangled here
Will this be merged anytime soon? |
What does this need to be merged? I hate playing in pillarbox on my TV, which is pretty much my main use case for this app |
Still waiting for this one. Steam Link already does this albeit with black bars due to the aspect ratio of iPads. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. |
Ping to keep from closing. |
Hello, I started working on this and got it to work for now. However, I've never coded in objective C before and there is some potential to optimize or better ways to solve it for sure. For now, when you connect an external Display, it will automatically move the stream to the external screen in its native resolution and the iPad only shows a black screen (or the statistics overlay if you enabled it). If you unplug the monitor, it will move to the iPad screen again. However, at the moment, i don't get it to work above 60Hz... |
@marinobantli Do you have a IPA binary available for testing? |
I've no Obj-C experience either, but give me an app and what I can do to get logs or debugging or whatever and I'll gladly help out. |
This sounds amazing! and exactly what Moonlight needs. Can any devs implement this? |
Sorry for the silence, as of now, the external screen works, but I can't get mouse scrolling and keyboard inputs to work on the external screen. Controller input works however. I would like to implement a little "dashboard" which is displayed on the iPads screen whenever an external display is connected to be able to change the external resolution. |
No apologies necessary! You're solving the final piece of the puzzle for my ideal set up! As a controller user this sounds like all I'd need, but obviously making sure everything works properly is important, and the added dashboard sounds like a lovely QoL improvement. I'd love to see @cgutman help you with this and get it implemented in the next version! |
To be clear, I need to take this branch, build it, and try it out? |
I would also love this feature to be added. Any updates? |
Ping! Just because I think this is a must 🙂 |
I will try to work on it again in a few weeks… |
Hello - just wondering if anyone has made progress on this request? The feature would be an amazing addition to the iOS app. |
It would be great if this feature did not depend on stage manager, because there are ipads that don’t all run stage manager (like ipad mini). |
here's an updated fork that works https://github.com/patroza/moonlight-ios/pull/1/files stage manager solutions are not useful for me because I dont have a pointing device like mouse connected to change the window to full screen on the external display. the external display is even just xreal air glasses. |
Can I use your fork or do we have to wait for merge |
Feel free |
Can confirm it’s working as expected. However when I locked my screen I noticed the image becomes static/stopped moving? Is this an iOS limitation? |
Think so but unsure; how is the behavior with Netflix or TV app? Or Steam link |
I’ve tested Netflix, Plex and nPlayer, they all seem to work fine with screen off. Not sure about steam link though, will test tomorrow Edit: I’ve tested Steam link, it behaves exactly as your forked version. Not sure how apps like Netflix got around this, although I doubt you can send any controller signal to iOS with the screen locked. |
nice find! yea in my case I use controllers connected to my pc, so for me that would be fine. but I imagine for most other people it will be important. |
I'm very interested having this, can we go on with the PR? |
Please prioritize this PR |
any progress on this?I try merge the pr with the master branch and it works. If there are any necessary change need to do before merge, I am willing to help. |
Any updates on the External Screen Support? |
Would love to see this too |
Added External Screen Support for all iOS devices whether through HDMI Out via Lightning/USB-C or Airplay Screen Mirroring.
Feature Request #11 discusses this.