CPanel Date and Time

[givascu]

Fix #530

Display:

[iulianR]

LGTM 👍

[mihneadb]

I would've preferred no recursive call via timeout and just a setInterval outside the function. Either way is fine, just saying. :)

[givascu]

@mihneadb you are right.
Should I revert this pull request and change to setInterval ?

[mihneadb]

Don't worry, just make another one.

On Sat, Sep 6, 2014 at 11:02 PM, Ivascu Gabriel notifications@github.com
wrote:

In wouso/resources/templates/cpanel/index.html:

• <script type="text/javascript">

•    function display_time() {
  

•        var x = new Date()
  

•        var hour = x.getHours()
  

•        var min = x.getMinutes()
  

•        var sec = x.getSeconds()
  

•        // add leading zeros
  

•        if (hour < 10) {hour = "0" + hour;}
  

•        if (min < 10) {min = "0" + min;}
  

•        if (sec < 10) {sec = "0" + sec;}
  

•        var time =   hour + ":" +  min + ":" +  sec;
  

•        document.getElementById('time').innerHTML = time;
  

•        // refresh rate 999 ms
  

•        setTimeout('display_time()', 999)
  

@mihneadb https://github.com/mihneadb you are right.
Should I revert this pull request and change to setInterval ?

Reply to this email directly or view it on GitHub
https://github.com/rosedu/wouso/pull/531/files#r17211005.

Mihnea Dobrescu-Balaur