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

Lots of new features #1

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

clayton-grey
Copy link

@clayton-grey clayton-grey commented Oct 28, 2019

Hello. I'm offering this up in case you have interest. I've spent a couple days making some updates to this. Originally I just wanted to add sound, but had some other ideas for ways to improve the application. You can now start in a specific step within a Pomodoro set (mostly useful if you accidentally cancel). There is a preference for sounds for the notifications (default on). There is a preference to show the timer in the bar (default off). I also added a single letter hint to the status bar to indicate the phase type to help verify that something is happening. The in menu info also shows you which step you're in within the set. I also fixed a few bugs. Anywho, for your consideration!

  • Fix the clock only showing a single digit for seconds
  • Add a notification sound
  • Lots of UI tweaks
  • Add a preference for whether the notification makes sound
  • “Start…”/"Break..." with procedural sub-menu
  • Timer display was off by one second.
  • Refactored terminology to use “Pomodoro” in lieu of session and “set” as a the collection pomodoros before the long break
  • Optional show timer in bar.
  • Show current Pomodoro # for working sessions in UI
  • Reset to 1 after set length
  • Try to get the timer to update faster in the status bar once a timer starts, it usually shows up a second or two later.
  • Show State hint in the Bar

- [x] Fix the clock only showing a single digit for seconds
- [x] Add a notification sound
- [x] Lots of UI tweaks
- [x] Add a preference for whether the notification makes sound
- [x] “Start…”/"Break..." with procedural sub-menu
- [x] Timer display was off by one second.
- [x] Refactored terminology to use “Pomodoro” in lieu of session and “set” as a the collection pomodoros before  the long break
- [x] Optional show timer in bar.
- [x] Show current Pomodoro # for working sessions in UI
- [x] Reset to 1 after set length
- [x] Try to get the timer to update faster in the status bar once a timer starts, it usually shows up a second or two later.
- [x] Show State hint in the Bar
@jeop10
Copy link

jeop10 commented Jun 29, 2020

@clayton-grey I just found out this project and I just wanted to say thank you for this changes the app is now perfect for me.

I am going to look around and see if I can add some tweaks of my own on my fork.

Thank you so much and have a great day!!!

Conversion to Swift 4.2 project to allow more recent XCodes to open the probject.
- Converted to Swift 5 and building for Apple Silicon
- Fixed a warning on the preferences window for a redundant tooltip that I missed.
-Set the build to export for release.
-Updated readme and added a screenshot to reflect my alternations.
Updated repository name to reflect the application name, and changing the readme to reflect this as well.
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.

2 participants