Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Merged by Bors] - Removed Mobile Touch event y-axis flip #6597

Closed
wants to merge 2 commits into from

Conversation

slyedoc
Copy link
Contributor

@slyedoc slyedoc commented Nov 13, 2022

Objective

Fix android touch events being flipped. Only removed test for android, don't have ios device to test with. Tested with emulator and physical device.

Solution

Remove check, no longer needed with coordinate change in 0.9

@alice-i-cecile alice-i-cecile added C-Bug An unexpected or incorrect behavior A-UI Graphical user interfaces, styles, layouts, and widgets labels Nov 13, 2022
@alice-i-cecile alice-i-cecile added this to the 0.9.1 milestone Nov 13, 2022
@alice-i-cecile
Copy link
Member

Can you remove it for iOS too? Even without testing this is virtually guaranteed to not be needed anymore.

@alice-i-cecile alice-i-cecile added the A-Input Player input via keyboard, mouse, gamepad, and more label Nov 13, 2022
@slyedoc
Copy link
Contributor Author

slyedoc commented Nov 13, 2022

Removed it for ios

@james7132 james7132 added O-Android Specific to the Android mobile operating system O-iOS Specific to the iOS mobile operating system labels Nov 15, 2022
@mockersf
Copy link
Member

tested on iOS 👍

@mockersf mockersf added the S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it label Nov 17, 2022
@mockersf
Copy link
Member

Could you change the PR title to reflect it's also for iOS? The title will be used as the commit name in git history

@alice-i-cecile
Copy link
Member

I'll merge this once the title is updated.

@slyedoc slyedoc changed the title Removed android touch event y-axis flip Removed Mobile Touch event y-axis flip Nov 18, 2022
@cart
Copy link
Member

cart commented Nov 18, 2022

bors r+

bors bot pushed a commit that referenced this pull request Nov 18, 2022
# Objective

Fix android touch events being flipped.  Only removed test for android, don't have ios device to test with.  Tested with emulator and physical device.

## Solution

Remove check, no longer needed with coordinate change in 0.9
@bors bors bot changed the title Removed Mobile Touch event y-axis flip [Merged by Bors] - Removed Mobile Touch event y-axis flip Nov 18, 2022
@bors bors bot closed this Nov 18, 2022
cart pushed a commit that referenced this pull request Nov 30, 2022
# Objective

Fix android touch events being flipped.  Only removed test for android, don't have ios device to test with.  Tested with emulator and physical device.

## Solution

Remove check, no longer needed with coordinate change in 0.9
ItsDoot pushed a commit to ItsDoot/bevy that referenced this pull request Feb 1, 2023
# Objective

Fix android touch events being flipped.  Only removed test for android, don't have ios device to test with.  Tested with emulator and physical device.

## Solution

Remove check, no longer needed with coordinate change in 0.9
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 A-UI Graphical user interfaces, styles, layouts, and widgets C-Bug An unexpected or incorrect behavior O-Android Specific to the Android mobile operating system O-iOS Specific to the iOS mobile operating system S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants