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

Pomodoro program updated #82

Merged
merged 5 commits into from
Feb 13, 2024
Merged

Pomodoro program updated #82

merged 5 commits into from
Feb 13, 2024

Conversation

sepandhaghighi
Copy link
Owner

@sepandhaghighi sepandhaghighi commented Feb 12, 2024

Reference Issues/PRs

What does this implement/fix? Explain your changes.

  • short-break program duration changed from 10 minutes to 5 minutes
  • long-break program duration changed from 30 minutes to 15 minutes
  • pomodoro program updated

Any other comments?

Most websites offer a Pomodoro timer that follows the 25-5-15 sequence. We plan to add an "Animedoro" #83 timer in the near future, so it would be better to update the Pomodoro program to the standard version.

@sepandhaghighi sepandhaghighi added bug Something isn't working enhancement New feature or request labels Feb 12, 2024
@sepandhaghighi sepandhaghighi added this to the mytimer v1.3 milestone Feb 12, 2024
@sepandhaghighi sepandhaghighi self-assigned this Feb 12, 2024
@sepandhaghighi sepandhaghighi marked this pull request as ready for review February 12, 2024 20:00
@@ -114,7 +114,7 @@
<tr align="center">
<td>Pomodoro</td>
<td><code>pomodoro</code></td>
<td>02:40:00</td>
<td>02:10:00</td>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't it be 2:15:00 (4 * (25 + 5) + 15 = 135 minutes)?

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Work + Short-Break + Work + Short-Break + Work + Short-Break + Work + Long-Break

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, you are right.

@sadrasabouri sadrasabouri merged commit cbd37f7 into dev Feb 13, 2024
51 checks passed
@sadrasabouri sadrasabouri deleted the pomodoro_edit branch February 13, 2024 06:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request minor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants