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

Input redirection support #35

Open
Eiim opened this issue Apr 28, 2024 · 3 comments
Open

Input redirection support #35

Eiim opened this issue Apr 28, 2024 · 3 comments
Labels
enhancement New feature or request

Comments

@Eiim
Copy link
Owner

Eiim commented Apr 28, 2024

I don't know much about input redirection, so it would be useful to first understand the landscape. Which other clients support it? How is their support implemented? Are there differences in what features they support? Is there any reason to support something like the old mod by Stary2001 in addition to Rosalina?

Once we have that we should look to an existing implementation(s) to copy for networking details. InputRedirectionClient-Qt looks promising, as it's just one file under MIT, but it hasn't been updated in a while. We can use Rosalina source code but need to be more careful about licensing in that case. If Rosalina has it documented somewhere that's an option. Otherwise other NTR clients may be options.

Finally, on the frontend side, handling keyboard inputs should be no issue since we have existing controls infrastructure for that, but mouse clicks for touchscreen might be trickier. We should try to be pixel-perfect to non-integer upscaling if possible. I don't think this will be a huge issue though.

@Eiim Eiim added the enhancement New feature or request label Apr 28, 2024
@Eiim Eiim added this to Chokistream Apr 28, 2024
@github-project-automation github-project-automation bot moved this to Todo - Medium Term in Chokistream Apr 28, 2024
@trevorwatkins1129
Copy link

I can probably handle this when I get time. The old mod has been out of support for a long time now, and the new Rosalina one has been supported since 9.0, therefore the Stary2001 mod should not need support unless there is enough requests. If I can find the network code in InputRedirection-Qt, I can rewrite in it Psuedo-Java (idk full java yet).

@trevorwatkins1129
Copy link

In order to not clog up this issue + to allow a progress log, I made a discord server. Feel free to join for updates (or Eiim for posting progress updates + talking about it).

https://discord.gg/J3NFdkAqNj

@ChainSwordCS
Copy link
Collaborator

ChainSwordCS commented Oct 11, 2024

Hm, I meant to post a comment a couple weeks ago, I guess I never got around to it.

It's worth noting, Chokistream can be used in conjunction with Rosalina's InputRedirection (and a client like InputRedirection-Qt) and they don't seem to interfere with each other. So, y'know, implementing this as a feature of Chokistream would be neat maybe, but it's low priority.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Todo - Medium Term
Development

No branches or pull requests

3 participants