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

Unified Diff: libadblockplus-android-settings/res/layout/fragment_adblock_whitelisted_domains_settings.xml

Issue 29361445: Issue 4399 - Add WebView inheritor with ad blocking (Closed)
Patch Set: Created Nov. 1, 2016, 12:14 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: libadblockplus-android-settings/res/layout/fragment_adblock_whitelisted_domains_settings.xml
diff --git a/libadblockplus-android-settings/res/layout/fragment_adblock_whitelisted_domains_settings.xml b/libadblockplus-android-settings/res/layout/fragment_adblock_whitelisted_domains_settings.xml
new file mode 100644
index 0000000000000000000000000000000000000000..faf117c4f945f48db5ec8ed63bdc6b0d460bf27e
--- /dev/null
+++ b/libadblockplus-android-settings/res/layout/fragment_adblock_whitelisted_domains_settings.xml
@@ -0,0 +1,52 @@
+<LinearLayout
+ xmlns:android="http://schemas.android.com/apk/res/android"
+ xmlns:tools="http://schemas.android.com/tools"
+ android:layout_width="match_parent"
+ android:layout_height="match_parent"
+ android:orientation="vertical"
+ android:padding="@dimen/fragment_adblock_wl_padding"
+ tools:context="org.adblockplus.libadblockplus.android.settings.AdblockWhitelistedDomainsSettingsFragment">
+
+ <LinearLayout
+ android:layout_width="match_parent"
+ android:layout_height="wrap_content"
+ android:orientation="horizontal">
+
+ <!-- Add domain input -->
+ <EditText
+ android:id="@+id/fragment_adblock_wl_add_label"
+ android:layout_width="1px"
+ android:layout_weight="1"
diegocarloslima 2016/11/08 16:30:39 It's usually recommended to use 0dp here when layo
anton 2016/11/09 12:30:29 Acknowledged.
+ android:layout_height="wrap_content"
+ android:lines="1"
+ android:hint="@string/fragment_adblock_settings_add_domain"
+ android:inputType="textUri"
+ android:textAppearance="?android:attr/textAppearanceMedium" />
+
+ <!-- Add button -->
+ <ImageView
+ android:id="@+id/fragment_adblock_wl_add_button"
+ android:layout_width="wrap_content"
+ android:layout_height="wrap_content"
+ android:layout_gravity="center_vertical"
+ android:src="@drawable/fragment_adblock_whitelisted_domains_add"
+ android:paddingLeft="@dimen/fragment_adblock_wl_item_padding" />
+
+ </LinearLayout>
+
+ <!-- Whitelisted domain label -->
+ <TextView
+ android:layout_width="match_parent"
+ android:layout_height="wrap_content"
+ android:text="@string/fragment_adblock_settings_wl_domains"
+ android:paddingTop="@dimen/fragment_adblock_wl_padding"
+ android:paddingBottom="@dimen/fragment_adblock_wl_padding"
+ android:textAppearance="?android:attr/textAppearanceMedium"/>
+
+ <!-- ListView -->
+ <ListView
+ android:id="@+id/fragment_adblock_wl_listview"
+ android:layout_width="match_parent"
+ android:layout_height="match_parent"/>
+
+</LinearLayout>

Powered by Google App Engine
This is Rietveld