-
Notifications
You must be signed in to change notification settings - Fork 17
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
Add controller/gamepad support #10
Comments
Very unlikely that this will ever happen. |
I know owner said no but I would also love for this to be implemented.
Following up on the button combo, most controllers nowadays have a home, select, and start button, plus the touchpad for Playstation controllers, but the game leaves at least one unused (share button and home on DS4 for me). A Select + R-stick combo isn't too uncomfortable for me not to get used to. Can't hurt to dream amirite... |
This functionality would be incredibly useful. Even just one ring button on controller would make this a viable solution. This mostly looks like it's written in C#, maybe I'll take a look at it over the weekend and see if I can figure it out. |
I got a first pass implementation of controllers working with some caveats. Since I can't submit to this repo I've forked and put them in the gh10-controller-support branch here : https://github.com/TomNCatz/TPie If you pull this down and load it up I recommend exporting your ring settings and backing them up somewhere. The conversion from the main plugin to this branch has been tested and should be fine, but I have not tested conversion back. As for the state of the work I did it:
Known Issues:
|
That's great! I don't see a pull request from you yet on this repo. Does it not let you even pull request?
It's an open-source project under the AGPL-3.0 license, so you can do what you want, as long as your derivative work is also under the AGPL-3.0 license even if a pull request is not accepted. Anyway, this motivates me to get back into the game and build your project for myself to test out. |
This is the main reason why I never even bothered with controller support. To me this defeats the entire purpose. That being said I'm not against adding the functionality behind some kind of warning message to the user describing the known issues of using a controller with the plugin. Feel free to open a PR and I'll take a look when I have the time. You could also make your own version of the plugin with a more hacky approach for controller support and distribute it as 3rd party. I would personally prefer to ultimately have the feature added to TPie but for that to happen I need to be happy with it. |
Oh I must have read about that issue too quickly to notice. Perhaps there's a way to emulate the L1+L3 gamepad controls so it triggers to gamepad blocking feature of Dalamud while the ring is showing and gamepad support is active in the TPie settings? Then, when the ring is dismissed, another L1+L3 could be triggered again? It's a shot in the dark, and I don't know if that is feasible or not or if I'm even using the right terminology with the word 'emulate'. |
Pull request submitted. This is just a first pass, so I would not expect you to pull it in 'as is' anyway. one big code callout I will make is that right now the gamepad bindings just copy all of the keybind info about jobs. This is unfavorable in several ways, but all of the solutions I could think of would have stood a large chance of disrupting the way keybinds currently work. |
Due to the radial menus, this seems like it would be a perfect fit for controller support. I'm currently about to set it up for controller on my end using Steam's radial menu configuration. I'm just leaving this idea here in the issues so maybe it can be easier for controller users in the future.
I'm thinking that there can be a combination of controller buttons/triggers not yet used by the game, that could serve as the hotkey to trigger the radial menu, then either the left or right thumbstick can control the mouse originating from the center of the screen and, when the user tilts the thumbstick, it can move the mouse pointer away from the center point, and either A or a controller trigger may select an option.
That just my rough idea. I got no expectations that this will get implemented any time soon, if at all, but I wanted to show my support by contributing ideas to improve the plugin. Thanks for your consideration!
The text was updated successfully, but these errors were encountered: