Skip to content

fix: Add mouse wheel delta to virtual button#2946

Merged
Eideren merged 14 commits intostride3d:masterfrom
Acissathar:fix-add-mouse-wheel-delta-to-virtual-button
Oct 27, 2025
Merged

fix: Add mouse wheel delta to virtual button#2946
Eideren merged 14 commits intostride3d:masterfrom
Acissathar:fix-add-mouse-wheel-delta-to-virtual-button

Conversation

@Acissathar
Copy link
Contributor

PR Details

This is sort of a mix of a feat/fix given that no one seems to have really mentioned it (github or discord), so not sure what to really mark it as. Currently, Mouse WheelDelta can only be grabbed directly from the InputManager rather than passed through VirtualButton like all other Mouse fields.

This change adds Mouse WheelDelta as a new field on the VirtualButton.Mouse, so that you can use the full range of the mouse through VirtualButtons.

Using it like so returns values just as if it were a GamePad thumbstick axis for example:
new VirtualButtonBinding(InputActions.FIELD_CAM_ZOOM_BACKWARD, VirtualButton.Mouse.WheelDelta),

I haven't run into any issues with my local fork, albeit I'm still new to Stride so not entirely sure of all scenarios worth testing beyond using it directly as a normal input.

Related Issue

2944

Types of changes

  • Docs change / refactoring / dependency upgrade
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • My change requires a change to the documentation.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • I have built and run the editor to try this change out.

commit 5b14803
Author: Will B <willbentz92@outlook.com>
Date:   Tue Sep 30 10:51:12 2025 -0600

    Move null check to callsite

commit 810b9da
Author: Will B <willbentz92@outlook.com>
Date:   Tue Sep 30 09:51:21 2025 -0600

    fix: Add null check for IsPrimitiveType to prevent crash when selecting multiple entities in GameStudio
commit 2122162
Merge: 1b5a165 71417d3
Author: Will <willbentz92@outlook.com>
Date:   Tue Sep 30 13:34:20 2025 -0600

    Merge branch 'stride3d:master' into feat-Add-Yes-To-All-for-asset-import

commit 1b5a165
Author: Will B <willbentz92@outlook.com>
Date:   Tue Sep 30 13:27:26 2025 -0600

    Move Overwrite buttons to own list as No to all is not a meaningful choice since No cancels the operation entirely.

commit d0b1721
Author: Will B <willbentz92@outlook.com>
Date:   Tue Sep 30 12:46:13 2025 -0600

    Add no to all option as well

commit 62c9e45
Author: Will B <willbentz92@outlook.com>
Date:   Tue Sep 30 12:38:15 2025 -0600

    Add Yes to all option when importing multiple assets for copying to resources and overwriting if relevant
commit 6a3aaa8
Author: Eideren <contact@eideren.com>
Date:   Thu Oct 16 10:15:46 2025 +0200

    fix: Int2-3 Equality using MathF and tolerance (stride3d#2934)
commit 2122162
Merge: 1b5a165 71417d3
Author: Will <willbentz92@outlook.com>
Date:   Tue Sep 30 13:34:20 2025 -0600

    Merge branch 'stride3d:master' into feat-Add-Yes-To-All-for-asset-import

commit 1b5a165
Author: Will B <willbentz92@outlook.com>
Date:   Tue Sep 30 13:27:26 2025 -0600

    Move Overwrite buttons to own list as No to all is not a meaningful choice since No cancels the operation entirely.

commit d0b1721
Author: Will B <willbentz92@outlook.com>
Date:   Tue Sep 30 12:46:13 2025 -0600

    Add no to all option as well

commit 62c9e45
Author: Will B <willbentz92@outlook.com>
Date:   Tue Sep 30 12:38:15 2025 -0600

    Add Yes to all option when importing multiple assets for copying to resources and overwriting if relevant
@Kryptos-FR
Copy link
Member

LGTM but if another maintainer or contributor can test it you before merging that would be nice. I unfortunately don't have that time.

@Kryptos-FR Kryptos-FR linked an issue Oct 24, 2025 that may be closed by this pull request
@Kryptos-FR Kryptos-FR added area-Input enhancement New feature or request labels Oct 24, 2025
@Eideren Eideren merged commit 19cc61d into stride3d:master Oct 27, 2025
7 checks passed
@Eideren
Copy link
Collaborator

Eideren commented Oct 27, 2025

Thanks !

@Acissathar Acissathar deleted the fix-add-mouse-wheel-delta-to-virtual-button branch October 27, 2025 13:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-Input enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

VirtualButton.Mouse does not have MouseWheelDelta

3 participants