Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: chrome/content/ui/sendReport.js

Issue 8329305: Fixed issue reporter`s progress labels for right-to-left languages (Closed)
Patch Set: Created Sept. 14, 2012, 9:09 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« chrome/content/ui/progressBar.js ('K') | « chrome/content/ui/progressBar.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/content/ui/sendReport.js
===================================================================
--- a/chrome/content/ui/sendReport.js
+++ b/chrome/content/ui/sendReport.js
@@ -1152,16 +1152,26 @@ let dataCollectors = [reportsListDataSou
subscriptionUpdateDataSource, issuesDataSource];
//
// Wizard logic
//
function initWizard()
{
+ const isRTL = (window.getComputedStyle(document.documentElement).direction == "rtl");
+ if(isRTL)
+ {
+ let progressLabels = Array.prototype.slice.call(E("progressBar").getElementsByTagName("label"));
+ for(let i=progressLabels.length; i--;)
+ {
+ progressLabels[i].parentNode.appendChild(progressLabels[i]);
+ }
+ }
+
Wladimir Palant 2012/09/14 21:20:40 This seems to be the wrong place to do that, it's
// Make sure no issue type is selected by default
E("typeGroup").selectedItem = null;
document.documentElement.addEventListener("pageshow", updateNextButton, false);
// Move wizard header
let header = document.getAnonymousElementByAttribute(document.documentElement, "class", "wizard-header");
if (header)
{
« chrome/content/ui/progressBar.js ('K') | « chrome/content/ui/progressBar.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld