Unified Diff: lib/utils.js
Issue 29338190:
Issue 3697 - Fall back to i18n.getUILanguage if @ui_locale isn't supported (Closed)
Patch Set: Address the nits
Created Aug. 30, 2017, 10:59 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« lib/notificationHelper.js ('k')
|
no next file »
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: lib/utils.js |
| =================================================================== |
| --- a/lib/utils.js |
| +++ b/lib/utils.js |
| @@ -43,10 +43,23 @@ |
| }, |
| get appLocale() |
| { |
| - let locale = ext.i18n.getMessage("@@ui_locale").replace(/_/g, "-"); |
| + let locale = ext.i18n.getUILanguage(); |
| Object.defineProperty(this, "appLocale", {value: locale, enumerable: true}); |
| return this.appLocale; |
| }, |
| + get readingDirection() |
| + { |
| + let direction = ext.i18n.getMessage("@@bidi_dir"); |
| + // This fallback is only necessary for Microsoft Edge |
| + if (!direction) |
| + direction = /^(?:ar|fa|he|ug|ur)\b/.test(this.appLocale) ? "rtl" : "ltr"; |
| + Object.defineProperty( |
| + this, |
| + "readingDirection", |
| + {value: direction, enumerable: true} |
| + ); |
| + return this.readingDirection; |
| + }, |
| generateChecksum(lines) |
| { |
| // We cannot calculate MD5 checksums yet :-( |
« no previous file with comments
|
« lib/notificationHelper.js ('k')
|
no next file »
|
no next file with comments »
