Skip to content

[Merged by Bors] - Added pressed state to input clear so it clears everything#4410

Closed
Sheepyhead wants to merge 4 commits intobevyengine:mainfrom
Sheepyhead:feature/3383-clear-pressed-input
Closed

[Merged by Bors] - Added pressed state to input clear so it clears everything#4410
Sheepyhead wants to merge 4 commits intobevyengine:mainfrom
Sheepyhead:feature/3383-clear-pressed-input

Conversation

@Sheepyhead
Copy link
Copy Markdown
Contributor

Objective

Adds pressed state to Input::clear(). Fixes #3383

@github-actions github-actions bot added the S-Needs-Triage This issue needs to be labelled label Apr 4, 2022
@IceSentry IceSentry added A-Input Player input via keyboard, mouse, gamepad, and more C-Bug An unexpected or incorrect behavior and removed S-Needs-Triage This issue needs to be labelled labels Apr 4, 2022
@cart
Copy link
Copy Markdown
Member

cart commented Apr 4, 2022

bors r+

bors bot pushed a commit that referenced this pull request Apr 4, 2022
# Objective

Adds `pressed` state to Input::clear(). Fixes #3383
@bors bors bot changed the title Added pressed state to input clear so it clears everything [Merged by Bors] - Added pressed state to input clear so it clears everything Apr 4, 2022
@bors bors bot closed this Apr 4, 2022
bors bot pushed a commit that referenced this pull request Apr 4, 2022
# Objective

- Revert #4410 
- `Input<T>.clear()` is the method call at the end of each frame for inputs. Clearing `pressed` in it mean that checking if a key is pressed will always return false
bors bot pushed a commit that referenced this pull request Apr 25, 2022
# Objective

- Part of the splitting process of #3692.

## Solution

- Add more tests to `input.rs` inside of `bevy_input`.

## Note

- The tests would now catch a change like #4410 and fail accordingly.
exjam pushed a commit to exjam/bevy that referenced this pull request May 22, 2022
# Objective

- Part of the splitting process of bevyengine#3692.

## Solution

- Add more tests to `input.rs` inside of `bevy_input`.

## Note

- The tests would now catch a change like bevyengine#4410 and fail accordingly.
aevyrie pushed a commit to aevyrie/bevy that referenced this pull request Jun 7, 2022
aevyrie pushed a commit to aevyrie/bevy that referenced this pull request Jun 7, 2022
# Objective

- Revert bevyengine#4410 
- `Input<T>.clear()` is the method call at the end of each frame for inputs. Clearing `pressed` in it mean that checking if a key is pressed will always return false
ItsDoot pushed a commit to ItsDoot/bevy that referenced this pull request Feb 1, 2023
ItsDoot pushed a commit to ItsDoot/bevy that referenced this pull request Feb 1, 2023
# Objective

- Revert bevyengine#4410 
- `Input<T>.clear()` is the method call at the end of each frame for inputs. Clearing `pressed` in it mean that checking if a key is pressed will always return false
ItsDoot pushed a commit to ItsDoot/bevy that referenced this pull request Feb 1, 2023
# Objective

- Part of the splitting process of bevyengine#3692.

## Solution

- Add more tests to `input.rs` inside of `bevy_input`.

## Note

- The tests would now catch a change like bevyengine#4410 and fail accordingly.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-Input Player input via keyboard, mouse, gamepad, and more C-Bug An unexpected or incorrect behavior

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Input::reset_all

3 participants