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

Fix running iOS timers when the proximity sensor is engaged #41262

Closed
wants to merge 1 commit into from

Conversation

saghul
Copy link
Contributor

@saghul saghul commented Oct 31, 2023

Summary:

When the proximity sensor is engaged and it detects "close", the screen is disabled so timers stop working. Treat the close proximity status as if the app went into the background so CADisplayLink based timers are not used.

Changelog:

[iOS] [Fixed] - Fix running timers when the proximity sensor detects close

Test Plan:

When the proximity sensor is engaged and it detects "close", the screen
is disabled so timers stop working. Treat the close proximity status as
if the app went into the background so CADisplayLink based timers are
not used.
@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team. labels Oct 31, 2023
@facebook-github-bot
Copy link
Contributor

@cipolleschi has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@cipolleschi
Copy link
Contributor

/rebase - this comment automatically rebase this PR on top of main.

Copy link

github-actions bot commented Nov 2, 2023

This pull request was successfully merged by @saghul in 79eac96.

When will my fix make it into a release? | Upcoming Releases

@github-actions github-actions bot added the Merged This PR has been merged. label Nov 2, 2023
@saghul
Copy link
Contributor Author

saghul commented Nov 2, 2023

Thank you for merging this so quickly!

Othinn pushed a commit to Othinn/react-native that referenced this pull request Jan 9, 2024
…#41262)

Summary:
When the proximity sensor is engaged and it detects "close", the screen is disabled so timers stop working. Treat the close proximity status as if the app went into the background so CADisplayLink based timers are not used.

bypass-github-export-checks

## Changelog:

[iOS] [Fixed] - Fix running timers when the proximity sensor detects close

Pull Request resolved: facebook#41262

Reviewed By: dmytrorykun

Differential Revision: D50839017

Pulled By: cipolleschi

fbshipit-source-id: 3f7dc47d346eb88b687c8219fc905cf2a42262fe
joosissimo added a commit to gracetech-services/jitsi-meet that referenced this pull request Jun 13, 2024
patch not needed on react-native 0.74 (fixed in facebook/react-native#41262)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged This PR has been merged. Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants