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

Issue 29370610: Issue 4760 - Change Ad blocking settings text (Closed)

Created:
Jan. 2, 2017, 4:07 p.m. by diegocarloslima
Modified:
Jan. 2, 2017, 9:15 p.m.
Reviewers:
Felix Dahlke
CC:
anton, René Jeschke
Visibility:
Public.

Description

Issue 4760 - Change Ad blocking settings text

Patch Set 1 #

Total comments: 1

Patch Set 2 : Renaming category_filters to category_languages #

Unified diffs Side-by-side diffs Delta from patch set Stats (+17 lines, -13 lines) Patch
M mobile/android/base/locales/adblockbrowser/en-US/android_strings.dtd View 1 1 chunk +8 lines, -6 lines 0 comments Download
M mobile/android/base/resources/xml-v11/preferences.xml View 1 chunk +1 line, -1 line 0 comments Download
M mobile/android/base/resources/xml-v11/preferences_abb_abp.xml View 1 1 chunk +2 lines, -2 lines 0 comments Download
M mobile/android/base/resources/xml/preferences.xml View 1 chunk +1 line, -1 line 0 comments Download
M mobile/android/base/resources/xml/preferences_abb_abp.xml View 1 1 chunk +2 lines, -2 lines 0 comments Download
M mobile/android/base/strings.xml.in View 1 1 chunk +3 lines, -1 line 0 comments Download

Messages

Total messages: 3
diegocarloslima
Jan. 2, 2017, 4:09 p.m. (2017-01-02 16:09:37 UTC) #1
Felix Dahlke
Looks good, just one naming thing. https://codereview.adblockplus.org/29370610/diff/29370611/mobile/android/base/resources/xml-v11/preferences_abb_abp.xml File mobile/android/base/resources/xml-v11/preferences_abb_abp.xml (right): https://codereview.adblockplus.org/29370610/diff/29370611/mobile/android/base/resources/xml-v11/preferences_abb_abp.xml#newcode17 mobile/android/base/resources/xml-v11/preferences_abb_abp.xml:17: <PreferenceScreen android:title="@string/abb_pref_category_filters" I'd ...
Jan. 2, 2017, 4:48 p.m. (2017-01-02 16:48:20 UTC) #2
Felix Dahlke
Jan. 2, 2017, 6:32 p.m. (2017-01-02 18:32:29 UTC) #3
Nice, LGTM!

Powered by Google App Engine
This is Rietveld