Bug 1468343 - Part 3: Change tooltip content to address infinity duration. r?jdescottes draft
authorDaisuke Akatsuka <dakatsuka@mozilla.com>
Fri, 22 Jun 2018 14:39:40 +0900
changeset 809468 1c66dcc9a4ebcf3253cf249d6543c637beca1167
parent 809467 6200cbb855f3579a02dcb26336b30d807de43d3c
child 809469 fdb71ab7453261098ee943a3517fd7484ae8706b
push id113686
push userbmo:dakatsuka@mozilla.com
push dateFri, 22 Jun 2018 05:40:18 +0000
reviewersjdescottes
bugs1468343
milestone62.0a1
Bug 1468343 - Part 3: Change tooltip content to address infinity duration. r?jdescottes MozReview-Commit-ID: 4DXXbhGgWF1
devtools/client/inspector/animation/components/graph/SummaryGraph.js
devtools/client/locales/en-US/animationinspector.properties
--- a/devtools/client/inspector/animation/components/graph/SummaryGraph.js
+++ b/devtools/client/inspector/animation/components/graph/SummaryGraph.js
@@ -52,17 +52,19 @@ class SummaryGraph extends PureComponent
     if (state.delay) {
       text += getStr("player.animationDelayLabel") + " ";
       text += getTime(state.delay);
       text += "\n";
     }
 
     // Adding the duration.
     text += getStr("player.animationDurationLabel") + " ";
-    text += getTime(state.duration);
+    text += state.duration === Infinity
+              ? getStr("player.infiniteDurationText")
+              : getTime(state.duration);
     text += "\n";
 
     // Adding the endDelay.
     if (state.endDelay) {
       text += getStr("player.animationEndDelayLabel") + " ";
       text += getTime(state.endDelay);
       text += "\n";
     }
@@ -71,20 +73,21 @@ class SummaryGraph extends PureComponent
     if (state.iterationCount !== 1) {
       text += getStr("player.animationIterationCountLabel") + " ";
       text += state.iterationCount || getStr("player.infiniteIterationCountText");
       text += "\n";
     }
 
     // Adding the iteration start.
     if (state.iterationStart !== 0) {
-      const iterationStartTime = state.iterationStart * state.duration / 1000;
       text += getFormatStr("player.animationIterationStartLabel",
                            state.iterationStart,
-                           numberWithDecimals(iterationStartTime, 2));
+                           state.duration === Infinity
+                             ? getStr("player.infiniteDurationText")
+                             : getTime(state.iterationStart * state.duration));
       text += "\n";
     }
 
     // Adding the easing if it is not "linear".
     if (state.easing && state.easing !== "linear") {
       text += getStr("player.animationOverallEasingLabel") + " ";
       text += state.easing;
       text += "\n";
--- a/devtools/client/locales/en-US/animationinspector.properties
+++ b/devtools/client/locales/en-US/animationinspector.properties
@@ -73,17 +73,17 @@ player.infiniteIterationCount=&#8734;
 player.infiniteIterationCountText=∞
 
 # LOCALIZATION NOTE (player.animationIterationStartLabel):
 # This string is displayed in a tooltip that appears when hovering over
 # animations in the timeline. It is the label displayed before the animation
 # iterationStart value.
 # %1$S will be replaced by the original iteration start value
 # %2$S will be replaced by the actual time of iteration start
-player.animationIterationStartLabel=Iteration start: %1$S (%2$Ss)
+player.animationIterationStartLabel=Iteration start: %1$S (%2$S)
 
 # LOCALIZATION NOTE (player.animationOverallEasingLabel):
 # This string is displayed in a tooltip that appears when hovering over
 # animations in the timeline. It is the label displayed before the easing
 # that applies to a whole iteration of an animation as opposed to the
 # easing that applies between animation keyframes.
 player.animationOverallEasingLabel=Overall easing: