InputSystem
InputSystem copied to clipboard
FIX: ISXB-704 scroll wheel support windows raw input value
Description
Fixes ISXB-704. Based on recently landed trunk PR.
Details of what was changed and the thought process can be found in this Google Document: https://docs.google.com/document/d/1XCC6GqaWzKf1M1jm-UmKRn6sIlmvF7rvF1C8Ws5Yxa4/edit?usp=sharing
Changes made
Please write down a short description of what changes were made.
Notes
Please write down any additional notes, remove the section if not applicable.
Checklist
Before review:
- [ ] Changelog entry added.
- Explains the change in
Changed,Fixed,Addedsections. - For API change contains an example snippet and/or migration example.
- FogBugz ticket attached, example
([case %number%](https://issuetracker.unity3d.com/issues/...)). - FogBugz is marked as "Resolved" with next release version correctly set.
- Explains the change in
- [ ] Tests added/changed, if applicable.
- Functional tests
Area_CanDoX,Area_CanDoX_EvenIfYIsTheCase,Area_WhenIDoX_AndYHappens_ThisIsTheResult. - Performance tests.
- Integration tests.
- Functional 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.
Looks like a branch update and CI rerun might be in order.
Couple of concerns from me:
- I did not need these changes to fix the issue, simply opening the Editor version that has your latest changes is enough to fix it
- The ifdefs for the code are not granular enough and I still get errors about missing Editor code on 6.0.6f1 Editor, should that be changed?
Also, please take a look at the slack discussion about this change in our input channel
You say you only need the Editor changes, are you sure there's no copy of my package fixes that survived in your Library folder? Otherwise I can't see how the InputSystemUIInputModule fixes could come from the Editor only... makes no sense :-(
About the ifdefs not being granular enough, I'll comment in the slack discussion but I'm not sure there's a clean solution.
Couple of concerns from me:
- I did not need these changes to fix the issue, simply opening the Editor version that has your latest changes is enough to fix it
- The ifdefs for the code are not granular enough and I still get errors about missing Editor code on 6.0.6f1 Editor, should that be changed?
Also, please take a look at the slack discussion about this change in our input channel
You say you only need the Editor changes, are you sure there's no copy of my package fixes that survived in your Library folder? Otherwise I can't see how the InputSystemUIInputModule fixes could come from the Editor only... makes no sense :-(
It's the user project from ISXB-704 with no library and 1.8.2 input system
Couple of concerns from me:
- I did not need these changes to fix the issue, simply opening the Editor version that has your latest changes is enough to fix it
- The ifdefs for the code are not granular enough and I still get errors about missing Editor code on 6.0.6f1 Editor, should that be changed?
Also, please take a look at the slack discussion about this change in our input channel
You say you only need the Editor changes, are you sure there's no copy of my package fixes that survived in your Library folder? Otherwise I can't see how the InputSystemUIInputModule fixes could come from the Editor only... makes no sense :-(
It's the user project from ISXB-704 with no library and 1.8.2 input system
Hi! I've updated the PR description to emphasize the changes that this PR contains. It's not observable scroll range fixes in any of the UI systems, but just the option to allow InputSystem actions returning values in the -120 to 120 range.