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

Issue 29574648: Issue 5860 - options page is broken in test enviroment (Closed)

Created:
Oct. 12, 2017, 3:52 p.m. by saroyanm
Modified:
Oct. 13, 2017, 8:55 a.m.
Visibility:
Public.

Description

Issue 5860 - options page is broken in test enviroment

Patch Set 1 #

Total comments: 6

Patch Set 2 : Addressed Sebastian's comments #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -3 lines) Patch
M ext/common.js View 1 1 chunk +3 lines, -3 lines 0 comments Download

Messages

Total messages: 5
saroyanm
Thomas can you please have a look.
Oct. 12, 2017, 4 p.m. (2017-10-12 16:00:28 UTC) #1
saroyanm
This affects only test environment, so it's not urgent.
Oct. 12, 2017, 4:01 p.m. (2017-10-12 16:01:47 UTC) #2
Sebastian Noack
https://codereview.adblockplus.org/29574648/diff/29574649/ext/common.js File ext/common.js (right): https://codereview.adblockplus.org/29574648/diff/29574649/ext/common.js#newcode90 ext/common.js:90: let parts = selectedLocale.split("_"); This seems wrong. window.navigator.language still ...
Oct. 12, 2017, 6:10 p.m. (2017-10-12 18:10:33 UTC) #3
saroyanm
Thanks Sebastian for having a look. https://codereview.adblockplus.org/29574648/diff/29574649/ext/common.js File ext/common.js (right): https://codereview.adblockplus.org/29574648/diff/29574649/ext/common.js#newcode90 ext/common.js:90: let parts = ...
Oct. 12, 2017, 8:26 p.m. (2017-10-12 20:26:38 UTC) #4
Sebastian Noack
Oct. 12, 2017, 9:54 p.m. (2017-10-12 21:54:11 UTC) #5
LGTM

Powered by Google App Engine
This is Rietveld