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

Unified Diff: ext/common.js

Issue 29574648: Issue 5860 - options page is broken in test enviroment (Closed)
Patch Set: Addressed Sebastian's comments Created Oct. 12, 2017, 8:23 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ext/common.js
===================================================================
--- a/ext/common.js
+++ b/ext/common.js
@@ -83,16 +83,16 @@
let getLocaleCandidates = function(selectedLocale)
{
let candidates = [];
- let defaultLocale = "en-US";
+ let defaultLocale = "en_US";
- // e.g. "ja-jp-mac" -> "ja-JP", note that the part after the second
+ // e.g. "ja-jp-mac" -> "ja_JP", note that the part after the second
// dash is dropped, since we only support language and region
let parts = selectedLocale.split("-");
let language = parts[0];
let region = (parts[1] || "").toUpperCase();
if (region)
- candidates.push(language + "-" + region);
+ candidates.push(language + "_" + region);
candidates.push(language);
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld