refactor(pawn, interaction): removes legacy fire & toggle mode input actions
- Removes the Fire and Toggle Navigation Mode interaction on the pawn class
- Removes the old WidgetInteraction and replaces it with a new one
- Removes the old input actions, mappings and configs that go along with it
- Moves the new interaction component to the Interactors folder, as it is a component that's to be added to the pawn
- Added new input actions and context for it
- Changed attachment process of the ray mesh
- Removed all the custom traces etc
- Added the new component to the pawn
Merge request reports
Activity
added Refactor label
requested review from @daniel.rupp
assigned to @david.gilbert
- Resolved by David Gilbert
We need to also change
UBasicVRInteractionComponent
, since itsBeing/EndInteraction()
methods are not called anymore. And probably should not, but somehow this has to be unified with the stuff by Marcel (or deleted altogether if now everythung exists in other components)
added 1 commit
- 22e13beb - refactor(pawn, interaction): removes some other deprecated files regarding pawn input
added 1 commit
- b7068808 - refactor(interaction): Restructures the widget interaction to work with the current system
requested review from @marcel.krueger and removed review request for @daniel.rupp
@marcel.krueger @jonathan.ehret @daniel.rupp can you look over this?
- Resolved by David Gilbert
- Resolved by David Gilbert
- Resolved by David Gilbert
- Resolved by David Gilbert
- Resolved by David Gilbert
- Resolved by David Gilbert
- Resolved by David Gilbert
- Resolved by David Gilbert
- Resolved by David Gilbert
- Resolved by David Gilbert
bCanRaytraceEveryTick should be removed, it doesn't do anything anymore as we're not tracing ourselves.
added 1 commit
- 85d47f35 - refactor(interaction): Removes bCanRaytraceEveryTick as it is now unnecessary
added 1 commit
- 73d08eea - refactor(interaction): Puts ray creation into own function
added 1 commit
- 9ec1269b - refactor(interaction): Adds logs to potential return statements and comments...
@marcel.krueger Should be addressed, skim it again in case I forgot something. Tested it and works as expected.
requested review from @jonathan.ehret and removed review request for @marcel.krueger
mentioned in commit 2b82ad57