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

Issue 29777679: Issue 6657 - Add whitelist translations (Closed)

Created:
May 11, 2018, 12:18 a.m. by diegocarloslima
Modified:
May 14, 2018, 9:45 p.m.
Reviewers:
anton, jens
Visibility:
Public.

Description

Issue 6657 - Add whitelist translations

Patch Set 1 #

Patch Set 2 : Adjusting one string in pt-BR as discussed with Tamara #

Unified diffs Side-by-side diffs Delta from patch set Stats (+149 lines, -149 lines) Patch
M mobile/android/base/locales/adblockbrowser/cs/android_strings.dtd View 2 chunks +7 lines, -7 lines 0 comments Download
M mobile/android/base/locales/adblockbrowser/de/android_strings.dtd View 2 chunks +7 lines, -7 lines 0 comments Download
M mobile/android/base/locales/adblockbrowser/el/android_strings.dtd View 2 chunks +7 lines, -7 lines 0 comments Download
M mobile/android/base/locales/adblockbrowser/es-AR/android_strings.dtd View 2 chunks +7 lines, -7 lines 0 comments Download
M mobile/android/base/locales/adblockbrowser/es-ES/android_strings.dtd View 2 chunks +7 lines, -7 lines 0 comments Download
M mobile/android/base/locales/adblockbrowser/es-MX/android_strings.dtd View 2 chunks +7 lines, -7 lines 0 comments Download
M mobile/android/base/locales/adblockbrowser/fr/android_strings.dtd View 2 chunks +7 lines, -7 lines 0 comments Download
M mobile/android/base/locales/adblockbrowser/it/android_strings.dtd View 2 chunks +7 lines, -7 lines 0 comments Download
M mobile/android/base/locales/adblockbrowser/ja/android_strings.dtd View 2 chunks +7 lines, -7 lines 0 comments Download
M mobile/android/base/locales/adblockbrowser/ko/android_strings.dtd View 2 chunks +7 lines, -7 lines 0 comments Download
M mobile/android/base/locales/adblockbrowser/ms/android_strings.dtd View 2 chunks +7 lines, -7 lines 0 comments Download
M mobile/android/base/locales/adblockbrowser/nl/android_strings.dtd View 2 chunks +7 lines, -7 lines 0 comments Download
M mobile/android/base/locales/adblockbrowser/pl/android_strings.dtd View 2 chunks +7 lines, -7 lines 0 comments Download
M mobile/android/base/locales/adblockbrowser/pt-BR/android_strings.dtd View 1 2 chunks +8 lines, -8 lines 0 comments Download
M mobile/android/base/locales/adblockbrowser/pt-PT/android_strings.dtd View 2 chunks +7 lines, -7 lines 0 comments Download
M mobile/android/base/locales/adblockbrowser/ru/android_strings.dtd View 2 chunks +7 lines, -7 lines 0 comments Download
M mobile/android/base/locales/adblockbrowser/th/android_strings.dtd View 2 chunks +7 lines, -7 lines 0 comments Download
M mobile/android/base/locales/adblockbrowser/tr/android_strings.dtd View 2 chunks +8 lines, -8 lines 0 comments Download
M mobile/android/base/locales/adblockbrowser/vi/android_strings.dtd View 2 chunks +7 lines, -7 lines 0 comments Download
M mobile/android/base/locales/adblockbrowser/zh-CN/android_strings.dtd View 2 chunks +7 lines, -7 lines 0 comments Download
M mobile/android/base/locales/adblockbrowser/zh-TW/android_strings.dtd View 2 chunks +7 lines, -7 lines 0 comments Download

Messages

Total messages: 3
diegocarloslima
May 11, 2018, 12:20 a.m. (2018-05-11 00:20:42 UTC) #1
anton
On 2018/05/11 00:20:42, diegocarloslima wrote: LGTM (though `abb_whitelist_website_dialog_hint` can be removed from all the lists ...
May 11, 2018, 9:40 a.m. (2018-05-11 09:40:40 UTC) #2
jens
May 11, 2018, 12:46 p.m. (2018-05-11 12:46:25 UTC) #3
On 2018/05/11 09:40:40, anton wrote:
> On 2018/05/11 00:20:42, diegocarloslima wrote:
> 
> LGTM (though `abb_whitelist_website_dialog_hint` can be removed from all the
> lists as it's not actually translated, assuming it's left for readability).

LGTM

Powered by Google App Engine
This is Rietveld