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

Feature/switch-controllers on main #96

Closed
wants to merge 27 commits into from

Conversation

KiritoDv
Copy link
Contributor

No description provided.

@KiritoDv KiritoDv changed the title Feature/switch on main Feature/switch-controllers on main Jan 17, 2023
@KiritoDv KiritoDv force-pushed the feature/switch-on-main branch 2 times, most recently from d9645a7 to f4fafa6 Compare January 17, 2023 18:12
@Kenix3 Kenix3 force-pushed the main branch 3 times, most recently from b54cbcb to ace0c03 Compare January 18, 2023 01:53
@leggettc18
Copy link
Contributor

Left this comment on the SoH PR, also leaving it here in case it's more visible here.

This works great on my Switch apart from one small issue. If I attempt to use a pro controller while in handheld mode, the inputs go through just fine but rumble and Gyro stay on the joycons. They both work fine on the Pro Controller when the switch is docked, and they work great with the joycons in handheld mode, but I do sometimes find myself setting my switch up on my desk and playing with a pro controller with the Switch in handheld mode, would be nice if that would work. I think always using the pro controller for Rumble and Gyro if one is connected would be the ideal solution.

@@ -1046,7 +1046,8 @@ enum ImGuiInputTextFlags_
ImGuiInputTextFlags_NoUndoRedo = 1 << 16, // Disable undo/redo. Note that input text owns the text data while active, if you want to provide your own undo/redo stack you need e.g. to call ClearActiveID().
ImGuiInputTextFlags_CharsScientific = 1 << 17, // Allow 0123456789.+-*/eE (Scientific notation input)
ImGuiInputTextFlags_CallbackResize = 1 << 18, // Callback on buffer capacity changes request (beyond 'buf_size' parameter value), allowing the string to grow. Notify when the string wants to be resized (for string types which hold a cache of their Size). You will be provided a new BufSize in the callback and NEED to honor it. (see misc/cpp/imgui_stdlib.h for an example of using this)
ImGuiInputTextFlags_CallbackEdit = 1 << 19 // Callback on any edit (note that InputText() already returns true on edit, the callback is useful mainly to manipulate the underlying buffer while focus is active)
ImGuiInputTextFlags_CallbackEdit = 1 << 19, // Callback on any edit (note that InputText() already returns true on edit, the callback is useful mainly to manipulate the underlying buffer while focus is active)
ImGuiInputTextFlags_NoPlusMinusButtons = 1 << 20 // Disable increment/decrement buttons
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is this new flag doing?

@briaguya-ai
Copy link
Collaborator

I tested this using HarbourMasters/Shipwright#2444

First I merged in latest LUS, this exists on https://github.com/briaguya-ai/libultraship/tree/update-switch-on-main
Then I ran into some issues with #97, so I made https://github.com/briaguya-ai/libultraship/tree/update-switch-on-main-two to address this

In my testing, I found:

  • Rumble on joycons never worked (either connected or dual wireless)
  • When a pro controller was connected, both the pro controller and the connected joycons could be used at the same time
    • When playing with the connected joycons, the pro controller would rumble

@KiritoDv these issues will need to be addressed before this can be considered ready to merge.

@Kenix3
Copy link
Owner

Kenix3 commented Dec 13, 2023

While we want the changes, this will have to be redone under the new controller system. Feel free to open a new PR to do that.

@Kenix3 Kenix3 closed this Dec 13, 2023
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.

None yet

6 participants