Skip to content
This repository has been archived by the owner on Jun 10, 2022. It is now read-only.

Warning of incoming break #5

Closed
Frozenfire92 opened this issue Jul 14, 2014 · 10 comments
Closed

Warning of incoming break #5

Frozenfire92 opened this issue Jul 14, 2014 · 10 comments
Assignees
Milestone

Comments

@Frozenfire92
Copy link
Collaborator

A user set time for when to pop up a warning so the user can save work before a break.

@Frozenfire92
Copy link
Collaborator Author

If it was set to 0 no warning would occur

@MoOx
Copy link

MoOx commented Jul 14, 2014

What about a OS notification ? (like OS X notification center).
I would be nice to trigger a sound too.

@MoOx
Copy link

MoOx commented Jul 14, 2014

Also using OS notification will allow us to be warned even if we are not coding ;)

@MoOx
Copy link

MoOx commented Jul 14, 2014

I think node-notifier can do the job.

@Glavin001
Copy link
Collaborator

+1!

@ddaws
Copy link
Collaborator

ddaws commented Jul 14, 2014

👍

@MoOx
Copy link

MoOx commented Jul 16, 2014

@Frozenfire92 Save & Save all shortcuts still work while the break popup is hre. So maybe you can just put a notice in the README ?
Do you need a hand to handle notifications (if you are open to).

@MoOx
Copy link

MoOx commented Jul 16, 2014

I ask for that because I'm missing a lot of break (I'm not coding all the time)

@Frozenfire92
Copy link
Collaborator Author

Hey @MoOx I am in favor of the OS notifications as I have also missed a few breaks. I likely won't have time until the weekend to work on it, however if you want to fork and make a pull request it would be welcome :)

Update See #15

Currently the break just overlays a div element and doesn't do anything to stop input. This is going to need some work and I will make an issue specifically for it, thanks for pointing that out!

Update See #14

@VernonGrant VernonGrant added this to the Redesign milestone Sep 9, 2016
@VernonGrant VernonGrant self-assigned this Sep 9, 2016
@VernonGrant
Copy link
Owner

@MoOx @ddaws OS notifications are now implemented in version 2.0.0. At the current moment they will fire 1 min before the break starts. This will be customizable in a future release.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants