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

Prefer calculation from actual time rather than increment set* #486

Merged
merged 7 commits into from
Jul 6, 2022

Conversation

kachick
Copy link
Member

@kachick kachick commented Jun 30, 2022

Fixes #484

I have avoided worker based solutions as written as #484 (comment).

The choice is actual duration based, I guess it reduces the differences even in background tabs. (Should not be perfect.)

https://stackoverflow.com/a/5927432/1212807

I hope background tasks should not take high power. So I hope this resolves the issues...

@pankona Could you review? (I didn't check long time run.)

@kachick kachick requested review from a team and pankona and removed request for a team June 30, 2022 04:31
src/session/Timer.tsx Outdated Show resolved Hide resolved
kachick added a commit that referenced this pull request Jul 1, 2022
@kachick kachick requested a review from pankona July 1, 2022 08:29
@kachick kachick merged commit 4bb5d68 into main Jul 6, 2022
@kachick kachick deleted the calc-from-actual-time branch July 6, 2022 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Timer doesn't look working precisely if mobu goes background
2 participants