
#Countdown timer javascript codepen full
Var start = document.Menus Icon Bar Menu Icon Accordion Tabs Vertical Tabs Tab Headers Full Page Tabs Hover Tabs Top Navigation Responsive Topnav Split Navigation Navbar with Icons Search Menu Search Bar Fixed Sidebar Side Navigation Responsive Sidebar Fullscreen Navigation Off-Canvas Menu Hover Sidenav Buttons Sidebar with Icons Horizontal Scroll Menu Vertical Menu Bottom Navigation Responsive Bottom Nav Bottom Border Nav Links Right Aligned Menu Links Centered Menu Link Equal Width Menu Links Fixed Menu Slide Down Bar on Scroll Hide Navbar on Scroll Shrink Navbar on Scroll Sticky Navbar Navbar on Image Hover Dropdowns Click Dropdowns Cascading Dropdown Dropdown in Topnav Dropdown in Sidenav Resp Navbar Dropdown Subnavigation Menu Dropup Mega Menu Mobile Menu Curtain Menu Collapsed Sidebar Collapsed Sidepanel Pagination Breadcrumbs Button Group Vertical Button Group Sticky Social Bar Pill Navigation Responsive Header

Var minute_text = cd.minutes + (cd.minutes > 1 ? ' minutes' : ' minute') Įl.innerHTML = minute_text + ' ' + cd.seconds + ' ' + second_text + '' You should get it once per function call at the beginning and cache it.Īt that point, you function would look more or less like this function countdown(element, minutes, seconds) Įlement = 'countdown1' ? cd = countdown1 : cd = countdown2 So they really shouldn't be there.Ī performance issue, you may want to know about is that you are doing a DOM lookup for the counters EVERY second. That variable will be set to the click event on the event handler (and again, is not being used) and will be undefined on setInterval. Then, you have a timer variable as a parameter for the click handler and setInterval but never used. Get rid of one of them (preferably get rid of the inline version). Secondly you are setting the click handler BOTH inline and within the script. The minutes and seconds are probably best passed as parameters and interval should be defined within the scope of function.

That way, every time the function is called, a new set of variables is created for its execution.

So first thing is to get rid of the three variables at the top and recreate them WITHIN the scope of the function. If you want a reusable timer, you can't hard set the variables it will use. There are few things you're doing that prevent you from expanding the code. Var start = document.getElementById('start') Var minute_text = minutes + (minutes > 1 ? ' minutes' : ' minute') Įl.innerHTML = minute_text + ' ' + seconds + ' ' + second_text + '' Var el = document.getElementById(element) Is there anyone who can easily change this script to a functioning one for two timers?
#Countdown timer javascript codepen how to
Here's the script I'm using, but I don't know how to duplicate the timer and let each timer work independently. timer1 lasts 10 secs and timer2 lasts 20). The timers need to have a start button and both have different timings (i.e. Currently working on a project that requires two timers on one page.
