Open Zxurian opened 9 years ago
Is the internal polling for it causing an error when no progressbar is found? I hadn't noticed any when I was testing.
it's not causing an error, no. The workaround of just not having the dom there to update is enough for now. I was just thinking it might be better to have an option to get rid of the polling entirely, so instead of setInterval()
for delay / timer
, you can just have a single setTimeout()
How does the set timeout workaround you developed handle when someone would like the timer to pause of the cursor is hovered over it?
— Sent from Mailbox
On Thu, Mar 19, 2015 at 9:16 AM, Zxurian notifications@github.com wrote:
it's not causing an error, no. The workaround of just not having the dom there to update is enough for now. I was just thinking it might be better to have an option to get rid of the polling entirely, so instead of
setInterval()
fordelay / timer
, you can just have a singlesetTimeout()
Reply to this email directly or view it on GitHub: https://github.com/mouse0270/bootstrap-notify/issues/76#issuecomment-83558639
I hadn't considered, the hover, but that could be taken care of by storing the start time of the display in a variable, and then on hover, just cleat the timeout an update the time elapsed, and on mouseout, start another setTimout with the remaining time.
currently I've done workaround by using a custom template that doesn't have the progressbar html in it, however the internal polling is still happening for it.
It'd be nice if there was an option to either enable or disable the ticker for notifications, and make it a global settings as well so notifications can default to either using or not.