Fixed issue reporter`s progress labels for right-to-left languages

chrome/content/ui/progressBar.js

 /*
  * This Source Code is subject to the terms of the Mozilla Public License
  * version 2.0 (the "License"). You can obtain a copy of the License at
  * http://mozilla.org/MPL/2.0/.
  */
 
 (function()
 {
   let progressBar, canvas, headers, isRTL;
 
@@ skipping 27 matching lines @@
     const gapWidth = 5;
     const arrowheadWidth = 5;
 
     let width = canvas.width = canvas.offsetWidth;
     let height = canvas.height = canvas.offsetHeight;
 
     let context = canvas.getContext("2d");
     context.fillStyle = window.getComputedStyle(progressBar, "").color;
     context.strokeStyle = window.getComputedStyle(progressBar, "").color;
     context.lineWidth = 1;
-    if(isRTL)

[Wladimir Palant, 2012/09/14 21:20:40]

Style nit: please insert a space between "if" and the parenthesis, makes the
code easier to "parse".

+    if (isRTL)
     {
       context.translate(width, 0);
       context.scale(-1, 1);
     }
 
     let panelCount = headers.length;
     let panelWidth = (width - gapWidth * (panelCount - 1) - 1) / panelCount;
     for (let i = 0; i < panelCount; i++)
     {
       context.save();
@@ skipping 17 matching lines @@
         context.lineTo(0, height - 1);
         context.lineTo(0, 0);
       }
 
       context.stroke();
       context.restore();
 
       let childLeft = Math.round(i * (panelWidth + gapWidth) + 1);
       let childWidth = panelWidth - arrowheadWidth - 2;
       let child = headers[i];
-      child.style.marginLeft = childLeft + "px";
-      child.style.marginRight = (width - childLeft - childWidth) + "px";
+      child.style.MozMarginStart = childLeft + "px";
+      child.style.MozMarginEnd = (width - childLeft - childWidth) + "px";
       child.style.width = childWidth + "px";

[Wladimir Palant, 2012/09/14 21:20:40]

Since we are already positioning headers manually, this seems to be the right
place to change header positions - no need to change the order in the document.
Once could tweak childLeft but using MozMarginStart/MozMarginEnd
(https://developer.mozilla.org/en-US/docs/CSS/-moz-margin-start) instead of
marginLeft/marginRight should already be sufficient.

     }
 
     // Resize after initialization should be ignored
     canvas.parentNode.removeAttribute("flex");
   }
 
   function getActiveItem()
   {
     for (let i = 0; i < headers.length; i++)
     {
       let header = headers[i];
       if (header.classList.contains("active"))
         return header;
     }
     return null;
   }
 
   function setActiveItem(val)
   {
     let complete = true;
     for (let i = 0; i < headers.length; i++)
     {
-      let header = headers[(isRTL) ? headers.length-i-1 : i];

[Wladimir Palant, 2012/09/14 21:20:40]

Style nit: please a space before and after a binary operator meaning
"headers.length - i - 1" here.

+      let header = headers[i];
       if (header == val)
         complete = false;
 
       if (!complete && header.value[0] == "✔")
         header.value = header.value.replace(/^✔\s*/, "");
       else if (complete && header.value[0] != "✔")
         header.value = "✔ " + header.value;
 
       if (header == val)
         header.classList.add("active");
@@ skipping 16 matching lines @@
     let activeItem = this.activeItem;
     if (!activeItem)
       return;
 
     if (!val && activeItem.value[0] == "✔")
       activeItem.value = activeItem.value.replace(/^✔\s*/, "");
     else if (val && activeItem.value[0] != "✔")
       activeItem.value = "✔ " + activeItem.value;
   }
 })();