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

Feature: stopwatch and timer #3

Open
poetaman opened this issue Jul 26, 2021 · 5 comments
Open

Feature: stopwatch and timer #3

poetaman opened this issue Jul 26, 2021 · 5 comments
Assignees
Labels
enhancement New feature or request

Comments

@poetaman
Copy link

Stopwatch and timer would make this a killer CLI app! Thanks

@valebes
Copy link
Owner

valebes commented Jul 26, 2021

Thanks for the suggestion.
Now I'm busy with work, later in August I'll work on something.

@valebes valebes self-assigned this Jul 26, 2021
@valebes valebes added the enhancement New feature or request label Jul 26, 2021
@Kyllingene
Copy link
Contributor

Kyllingene commented Apr 24, 2023

That could be a laggy stopwatch, considering that it has to format and draw large block characters (and rsclock is optimized for size). A timer would be handy, but I know from experience that notifying the user of a finished TUI timer is tricky.

You should try your hand at it yourself, and PR when you have it working; even if your PR doesn't get accepted, you have your own version with the functionality you want!

@poetaman
Copy link
Author

poetaman commented Apr 24, 2023

@Kyllingene I wrote this: https://github.com/poetaman/arttime for my needs. It has cross-platform desktop notifications. A stopwatch need not update screen every 10 ms. Are you maintaining this project?

@Kyllingene
Copy link
Contributor

No, simply giving friendly suggestions. I quickly tested what it might look like if rsClock updated rapidly, and I stand corrected; it worked fine (on Alacritty, Arch Linux).

My one remaining concern is desktop notifications. arttime is written in zsh (and only supports environments with notify-send), so unless maintaining cross-platform Commands sounds fun to valebes, that solution isn't portable. notify-rust seems promising (it supports Linux, Mac, and even Windows), if it's kept to a bare minimum (summary and body). I think the reason I dislike this option is just because it's adding several layers of complexity (separate modes of operation, dbus concerns); however, that's entirely up to valebes.

@valebes
Copy link
Owner

valebes commented Apr 29, 2023

Probably adding notification management would be a "little too much"(as said by @Kyllingene), especially in terms of adding complexity to the entire code just for one (secondary) feature.
Maybe add as a feature the option to set an alarm clock? But even there it would seem unnecessary to me to manage with the notification system.
I leave the issue open for possibly future updates.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants