How to make Javascript Count Up timer work

Question

I am trying to implement a simple count up timer using Javascript and show the timer in the HTML page. This is my code:

var minutesLabel = document.getElementById("minutes");
var secondsLabel = document.getElementById("seconds");
var totalSeconds = 0;
setInterval(setTime, 1000);

function setTime()
{
    ++totalSeconds;
    secondsLabel.innerHTML = pad(totalSeconds%60);
    minutesLabel.innerHTML = pad(parseInt(totalSeconds/60));
}

function pad(val)
{
    var valString = val + "";
    if(valString.length < 2)
    {
        return "0" + valString;
    }
    else
    {
        return valString;
    }
}

HTML:

<label id="minutes">00</label>
    <label id="colon">:</label>
    <label id="seconds">00</label>

But the timer is not working in the HTML. It stays at 00:00

can you show the complete html page? I have copied the same code and it is working fine for me. <script type="text/javascript" src="yourfile.js"></script> — Sagar Rana
– Sagar Rana, Commented Dec 1, 2020 at 5:32
Yes the code works in the snippet, but for some reason it does not work on my end — efuzz
– efuzz, Commented Dec 1, 2020 at 5:37

efuzz · Accepted Answer · 2020-12-01 05:44:11Z

2

Make sure the script is at the bottom on the body tag

answered Dec 1, 2020 at 5:44

efuzz

1171 silver badge9 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

anpel · Accepted Answer · 2020-12-01 05:33:44Z

1

The code you posted works exactly as you describe.

Check this fiddle, I just copy pasted it.

answered Dec 1, 2020 at 5:33

anpel

9416 silver badges16 bronze badges

4 Comments

efuzz Over a year ago

Yes the code works in the snippet, but for some reason it does not work on my end

anpel Over a year ago

Then some other part of your code must have an error. Can you please check your console?

efuzz Over a year ago

yeah I had the script at the beginning of the body. Had to move it to the closing tag of the body

anpel Over a year ago

@efuzz Indeed, getElementById (or any other function that gets DOM nodes) will not work unless the nodes already exist. You can either move your script to the bottom of the body or wrap it in an onload function.

kishore Rajendran · Accepted Answer · 2020-12-01 05:32:36Z

0

The code you mentioned is working perfectly fine in the snippets.

answered Dec 1, 2020 at 5:32

kishore Rajendran

2783 silver badges14 bronze badges

1 Comment

efuzz Over a year ago

Yes the code works in the snippet, but for some reason it does not work on my end

Collectives™ on Stack Overflow

How to make Javascript Count Up timer work

3 Answers 3

Comments

4 Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

4 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related