Bug 1467232 - Ensure ping explanation is formatted correctly. r?dexter
The "pingDetails" explanation takes only 2 arguments: a link and the name of the ping to be displayed.
MozReview-Commit-ID: CxLnHb73YGu
--- a/toolkit/content/aboutTelemetry.js
+++ b/toolkit/content/aboutTelemetry.js
@@ -322,17 +322,17 @@ var PingPicker = {
let pingTypeText = this._getSelectedPingType();
controls.classList.remove("hidden");
pingType.textContent = pingTypeText;
// Change home page text.
pingName = bundle.formatStringFromName("namedPing", [pingName, pingTypeText], 2);
pingNameSpan.textContent = pingName;
let explanation = bundle.GetStringFromName("pingDetails");
- fragment = BrowserUtils.getLocalizedFragment(document, explanation, pingLink, pingNameSpan, pingTypeText);
+ fragment = BrowserUtils.getLocalizedFragment(document, explanation, pingLink, pingNameSpan);
} else {
// Change sidebar heading text.
controls.classList.add("hidden");
pingType.textContent = bundle.GetStringFromName("currentPingSidebar");
// Change home page text.
pingNameSpan.textContent = pingName;
let explanation = bundle.GetStringFromName("pingDetailsCurrent");