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

Unified Diff: adblockplussbrowser/res/xml/preferences_main.xml

Issue 29441558: Issue 5243 - Clipped text across languages (Closed)
Patch Set: Added copyright header, extracted static method to set multiline property in onBindView(), added multi line property for subscriptions screen Created May 30, 2017, 12:25 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
Index: adblockplussbrowser/res/xml/preferences_main.xml
===================================================================
--- a/adblockplussbrowser/res/xml/preferences_main.xml
+++ b/adblockplussbrowser/res/xml/preferences_main.xml
@@ -4,8 +4,10 @@
android:layout_height="fill_parent" >
<PreferenceScreen
+ android:layout="@layout/multiline_preference_screen"
android:summary="@string/manage_subscriptions_summary"
- android:title="@string/manage_subscriptions" >
+ android:title="@string/manage_subscriptions"
+ >
diegocarloslima 2017/05/31 13:33:58 the android:title line shouldn't have changed. the
jens 2017/05/31 13:58:57 Acknowledged.
<org.adblockplus.sbrowser.contentblocker.ListedSubscriptionsPreferenceCategory
android:layout_width="fill_parent"
@@ -35,7 +37,7 @@
<org.adblockplus.sbrowser.contentblocker.WhitelistedWebsitesPreferenceCategory/>
</PreferenceScreen>
- <ListPreference
+ <org.adblockplus.sbrowser.contentblocker.preferences.MultilineListPreference
android:layout_width="fill_parent"
android:layout_height="wrap_content"
android:defaultValue="1"
@@ -49,7 +51,7 @@
android:layout_height="wrap_content"
android:title="@string/acceptable_ads_category" >
- <CheckBoxPreference
+ <org.adblockplus.sbrowser.contentblocker.preferences.MultilineCheckBoxPreference
android:layout_width="fill_parent"
android:layout_height="wrap_content"
android:defaultValue="true"

Powered by Google App Engine
This is Rietveld