Use template literals for 'formatTime'

servo · Jul 22, 2019 · cf9f46b · cf9f46b
1 parent a8419c4
commit cf9f46b
Showing 1 changed file with 12 additions and 18 deletions.
diff --git a/resources/media-controls.js b/resources/media-controls.js
@@ -58,24 +58,18 @@
   }
 
   function formatTime(time, showHours = false) {
-      // Format the duration as "h:mm:ss" or "m:ss"
-      time = Math.round(time / 1000);
-      let hours = Math.floor(time / 3600);
-      let mins  = Math.floor((time % 3600) / 60);
-      let secs  = Math.floor(time % 60);
-      let timeString;
-      if (secs < 10) {
-        secs = "0" + secs;
-      }
-      if (hours || showHours) {
-        if (mins < 10) {
-          mins = "0" + mins;
-        }
-        timeString = hours + ":" + mins + ":" + secs;
-      } else {
-        timeString = mins + ":" + secs;
-      }
-      return timeString;
+    // Format the duration as "h:mm:ss" or "m:ss"
+    time = Math.round(time / 1000);
+
+    const hours = Math.floor(time / 3600);
+    const mins  = Math.floor((time % 3600) / 60);
+    const secs  = Math.floor(time % 60);
+
+    const formattedHours = hours || showHours ?
+      `${hours.toString().padStart(2, "0")}:` :
+      "";
+
+    return `${formattedHours}${mins.toString().padStart(2, "0")}:${secs.toString().padStart(2, "0")}`;
   }
 
   class MediaControls {