www.webdeveloper.com
Results 1 to 2 of 2

Thread: onclick image

  1. #1
    Join Date
    Aug 2013
    Posts
    1

    onclick image

    newbie to this forum and a learner in javascript. i got a timer from online and how to make timer to start only when an image is clicked. the code i got from online

    Code:
    <script type="text/javascript">
    paused = false;
    // set minutes
    var mins = 30;
    // calculate the seconds 
    var secs = mins * 60;
    var t=0;
      var flagTimer='resume';
    function countdown() {
        t = setTimeout('Decrement()',1000);
    }
    function Decrement() {
        if (document.getElementById) {
            minutes = document.getElementById("minutes");
            seconds = document.getElementById("seconds");
            // if less than a minute remaining
            if (seconds < 59) {
                seconds.value = secs;
            } else {
                minutes.value = getminutes();
                seconds.value = getseconds();
            }
            secs--;
            t= setTimeout('Decrement()',1000);
        }
    }
    function getminutes() {
        // minutes is seconds divided by 60, rounded down
        mins = Math.floor(secs / 60);
        return mins;
    }
    function getseconds() {
        // take mins remaining (as seconds) away from total seconds remaining
        return secs-Math.round(mins *60);
    }
    function pause() { 
      if( flagTimer=='resume')
      {
        clearTimeout(t);
        t=0;alert(' 12c  ya');
        document.getElementById('Pause').value="Resume";
        flagTimer='pause';
      }
      else
      {
        flagTimer='resume';
        document.getElementById('Pause').value="Pause";
        resume();
      }
    
    }
    function resume() {
        t= setTimeout('Decrement()',1000);
    }
    
    </script>

  2. #2
    Join Date
    Dec 2012
    Posts
    46
    looks like you would only need to do something like:

    Code:
    <img src="pic.bmp" onclick="countdown()">
    Of course the more accepted method would be to use jQuery but thats a bit of a deep rabbit hole for now.

    If I might suggest a tip. I would look into how to make the entire thing an object and then use it that way. Your current method has a bunch of globals and if you try to insert any other apps from other pages that also have pause functions that are global the entire page will fall apart.

    I know its out of the scope of the question but something like:
    Code:
    <script type="text/javascript">
    function MyTimer(){
      paused = false;
      // set minutes
      var mins = 30;
      // calculate the seconds  
      var secs = mins * 60;
      var t=0;
      var flagTimer='resume';
      this.countdown = countdown;
      function countdown() {
        t = setTimeout('this.Decrement',1000);
      }
      this.Decrement = Decrement;
      function Decrement() {
        if (document.getElementById) {
            minutes = document.getElementById("minutes");
            seconds = document.getElementById("seconds");
            // if less than a minute remaining
            if (seconds < 59) {
                seconds.value = secs;
            } else {
                minutes.value = getminutes();
                seconds.value = getseconds();
            }
            secs--;
            t= setTimeout('this.Decrement()',1000);
        }
      }
      function getminutes() {
        // minutes is seconds divided by 60, rounded down
        mins = Math.floor(secs / 60);
        return mins;
      }
      function getseconds() {
        // take mins remaining (as seconds) away from total seconds remaining
        return secs-Math.round(mins *60);
      }
      function pause() { 
        if( flagTimer=='resume')
        {
          clearTimeout(t);
          t=0;alert(' 12c  ya');
          document.getElementById('Pause').value="Resume";
          flagTimer='pause';
        }
        else
        {
          flagTimer='resume';
          document.getElementById('Pause').value="Pause";
          resume();
        }
      }
      function resume() {
          t= setTimeout('Decrement()',1000);
      }
    }
    var ThisTimer = new MyTimer();
    </script>
    ........
    ........
    <img src="pic.bmp" onclick="ThisTimer.countdown()">
    Might be a little safer. I'm still not an expert on JavaScript by any means, but I have learned the dangers of globals over the years in other languages.
    If you included your html file so I saw how you were using this timer I could test it fully but since I could not I just am hoping this code will work for you ;o).

    Good luck.
    Last edited by Lesshardtoofind; 08-24-2013 at 11:59 PM.

Thread Information

Users Browsing this Thread

There are currently 1 users browsing this thread. (0 members and 1 guests)

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •  
HTML5 Development Center



Recent Articles