FIX: Use correct data format for composite stick axes in HID layout builder

[myss]

The generic HID layout builder made a faulty assumption that the X and Y axes of a composite "Stick" control always used a simple bitfield format (SBIT/BIT).

This assumption is incorrect for many modern devices, most notably 6DoF controllers like the 3Dconnexion SpaceMouse, which report their axes as 16-bit shorts (SHRT). This bug caused the builder to apply the wrong data format, resulting in incorrect parsing and unusable stick input for these devices.

This commit resolves the issue by replacing the brittle, hardcoded format logic with a call to the element.DetermineFormat() helper method. This is the same robust method used by the general-purpose element processor later in the same function.

This change:

• Fixes axis detection for 6DoF controllers and other HID devices that use SHRT or BYTE formats.
• Makes the special-case "Stick" logic consistent with the rest of the HID parser.

Description

Fix HID handling for 3dconnexion SpaceMouse. Without the fix, the "/stick", "/stick/x" and "/stick/y" values contained garbage.

Testing status & QA

Only tested on a 3dconnexion SpaceMouse Wireless.

Overall Product Risks

Comments to reviewers

Checklist

Before review:

• Changelog entry added.
  • Explains the change in Changed, Fixed, Added sections.
  • For API change contains an example snippet and/or migration example.
  • JIRA ticket linked, example (case %%). If it is a private issue, just add the case ID without a link.
  • Jira port for the next release set as "Resolved".
• Tests added/changed, if applicable.
  • Functional tests Area_CanDoX, Area_CanDoX_EvenIfYIsTheCase, Area_WhenIDoX_AndYHappens_ThisIsTheResult.
  • Performance tests.
  • Integration tests.
• Docs for new/changed API's.
  • Xmldoc cross references are set correctly.
  • Added explanation how the API works.
  • Usage code examples added.
  • The manual is updated, if needed.

During merge:

• Commit message for squash-merge is prefixed with one of the list:
  • NEW: ___.
  • FIX: ___.
  • DOCS: ___.
  • CHANGE: ___.
  • RELEASE: 1.1.0-preview.3.

After merge:

• Create forward/backward port if needed. If you are blocked from creating a forward port now please add a task to ISX-1444.

[ekcoh]

Thanks for finding this @myss , yes assuming the format specifier is just a format bit is definitely incorrect, good catch. I suggest we pick this PR up, complement it with a unit test with a HID descriptor using this format, e.g. 3D connexion space mouse, update the changeling to reflect the change and land this.c

[unity-cla-assistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ ekcoh
❌ myss
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

[ekcoh]

Merged develop into this branch to get critical CI fix from #2260. Hope this doesn't caused any inconvenience for you.

[myss]

No, not a problem. This is a simple change, which is now being handled in #2246. Feel free to close this PR.

[myss]

Or feel free to reopen if needed.

[jfreire-unity]

Thanks a lot for the contribution @myss