bootstrap-notify icon indicating copy to clipboard operation
bootstrap-notify copied to clipboard

Suggestion: Add option to disable to ticker if desired

Open Zxurian opened this issue 10 years ago • 4 comments

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.

Zxurian avatar Mar 18 '15 22:03 Zxurian

Is the internal polling for it causing an error when no progressbar is found? I hadn't noticed any when I was testing.

mouse0270 avatar Mar 18 '15 22:03 mouse0270

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()

Zxurian avatar Mar 19 '15 13:03 Zxurian

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 [email protected] 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() for delay / timer, you can just have a single setTimeout()

Reply to this email directly or view it on GitHub: https://github.com/mouse0270/bootstrap-notify/issues/76#issuecomment-83558639

mouse0270 avatar Mar 19 '15 13:03 mouse0270

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.

Zxurian avatar Mar 27 '15 17:03 Zxurian