Dbrant has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/394454 )

Change subject: Update text colors in About activity.
......................................................................

Update text colors in About activity.

Bug: T181640
Change-Id: Ib9ce17b368aa6bfca0632940cd8b81a259ce6114
---
M app/src/main/res/layout/activity_about.xml
1 file changed, 110 insertions(+), 115 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/apps/android/wikipedia 
refs/changes/54/394454/1

diff --git a/app/src/main/res/layout/activity_about.xml 
b/app/src/main/res/layout/activity_about.xml
index 0b1532e..d1b595b 100644
--- a/app/src/main/res/layout/activity_about.xml
+++ b/app/src/main/res/layout/activity_about.xml
@@ -1,141 +1,136 @@
 <?xml version="1.0" encoding="utf-8"?>
 
 <ScrollView
-        xmlns:android="http://schemas.android.com/apk/res/android";
-        xmlns:app="http://schemas.android.com/apk/res-auto";
-        xmlns:tools="http://schemas.android.com/tools";
-        android:id="@+id/about_scrollview"
-        android:layout_width="match_parent"
-        android:layout_height="match_parent">
+    xmlns:android="http://schemas.android.com/apk/res/android";
+    xmlns:app="http://schemas.android.com/apk/res-auto";
+    xmlns:tools="http://schemas.android.com/tools";
+    android:id="@+id/about_scrollview"
+    android:layout_width="match_parent"
+    android:layout_height="match_parent">
+
     <LinearLayout
-            android:id="@+id/about_container"
-            android:orientation="vertical"
-            android:layout_width="match_parent"
-            android:layout_height="wrap_content"
-            android:padding="24dp"
-            android:gravity="center"
-            >
+        android:id="@+id/about_container"
+        android:orientation="vertical"
+        android:layout_width="match_parent"
+        android:layout_height="wrap_content"
+        android:padding="24dp"
+        android:gravity="center">
+
         <ImageView
             android:id="@+id/about_logo_image"
             android:layout_width="48dp"
             android:layout_height="48dp"
             android:src="@drawable/w_nav_mark"
             android:tint="?attr/primary_text_color"
-            android:contentDescription="@null"
+            android:contentDescription="@null" />
 
-            />
-        <LinearLayout
-                android:layout_width="wrap_content"
-                android:layout_height="wrap_content"
-                android:gravity="center_vertical"
-                android:layout_marginBottom="8dp"
-                android:orientation="vertical">
-                <ImageView
-                    android:layout_width="match_parent"
-                    android:layout_height="wrap_content"
-                    android:src="@drawable/wp_wordmark"
-                    android:tint="@color/base30"
-                    android:contentDescription="@null"
-                    />
-                <TextView android:id="@+id/about_version_text"
-                          android:layout_width="wrap_content"
-                          android:layout_height="wrap_content"
-                          tools:text="2.x-sample-version"
-                          style="?android:textAppearanceSmall"
-                          android:textIsSelectable="true"
-                          android:textColor="#888"
-                    />
-        </LinearLayout>
+        <ImageView
+            android:layout_width="128dp"
+            android:layout_height="wrap_content"
+            android:src="@drawable/wp_wordmark"
+            android:tint="?attr/primary_text_color"
+            android:contentDescription="@null" />
+
+        <TextView android:id="@+id/about_version_text"
+            android:layout_width="wrap_content"
+            android:layout_height="wrap_content"
+            android:layout_marginBottom="8dp"
+            tools:text="2.x-sample-version"
+            style="?android:textAppearanceSmall"
+            android:textIsSelectable="true"
+            android:textColor="?attr/primary_text_color" />
 
         <TextView
-                android:layout_width="match_parent"
-                android:layout_height="wrap_content"
-                android:layout_marginTop="8dp"
-                android:layout_marginBottom="8dp"
-                style="?android:textAppearanceSmall"
-                android:text="@string/about_contributors_heading" />
+            android:layout_width="match_parent"
+            android:layout_height="wrap_content"
+            android:layout_marginTop="8dp"
+            android:layout_marginBottom="8dp"
+            style="?android:textAppearanceSmall"
+            android:textColor="?attr/primary_text_color"
+            android:text="@string/about_contributors_heading" />
+
         <org.wikipedia.views.AppTextView
-                android:layout_width="match_parent"
-                android:layout_height="wrap_content"
-                android:layoutDirection="ltr"
-                android:text="@string/contributors_list"
-                android:textAppearance="?android:attr/textAppearanceSmall"
-                android:textIsSelectable="true"
-                android:lineSpacingMultiplier="1.2"
-                android:textColor="#555"
-                android:layout_marginBottom="8dp"
-                />
+            android:layout_width="match_parent"
+            android:layout_height="wrap_content"
+            android:layoutDirection="ltr"
+            android:text="@string/contributors_list"
+            android:textAppearance="?android:attr/textAppearanceSmall"
+            android:textIsSelectable="true"
+            android:lineSpacingMultiplier="1.2"
+            android:textColor="?attr/secondary_text_color"
+            android:layout_marginBottom="8dp" />
 
         <TextView
-                android:layout_width="match_parent"
-                android:layout_height="wrap_content"
-                android:layout_marginTop="8dp"
-                android:layout_marginBottom="8dp"
-                style="?android:textAppearanceSmall"
-                android:text="@string/about_translators_heading" />
+            android:layout_width="match_parent"
+            android:layout_height="wrap_content"
+            android:layout_marginTop="8dp"
+            android:layout_marginBottom="8dp"
+            style="?android:textAppearanceSmall"
+            android:textColor="?attr/primary_text_color"
+            android:text="@string/about_translators_heading" />
+
         <org.wikipedia.views.AppTextView
-                android:id="@+id/about_translators"
-                android:layout_width="match_parent"
-                android:layout_height="wrap_content"
-                style="@style/RtlAwareTextView"
-                android:text="@string/about_translators_translatewiki"
-                android:textAppearance="?android:attr/textAppearanceSmall"
-                android:textIsSelectable="true"
-                android:lineSpacingMultiplier="1.2"
-                android:textColor="#555"
-                android:layout_marginBottom="8dp"
-                />
+            android:id="@+id/about_translators"
+            android:layout_width="match_parent"
+            android:layout_height="wrap_content"
+            style="@style/RtlAwareTextView"
+            android:text="@string/about_translators_translatewiki"
+            android:textAppearance="?android:attr/textAppearanceSmall"
+            android:textIsSelectable="true"
+            android:lineSpacingMultiplier="1.2"
+            android:textColor="?attr/secondary_text_color"
+            android:layout_marginBottom="8dp" />
 
         <TextView
-                android:layout_width="match_parent"
-                android:layout_height="wrap_content"
-                style="?android:textAppearanceSmall"
-                android:text="@string/about_libraries_heading"
-                android:layout_marginTop="8dp"
-                android:layout_marginBottom="8dp" />
+            android:layout_width="match_parent"
+            android:layout_height="wrap_content"
+            style="?android:textAppearanceSmall"
+            android:textColor="?attr/primary_text_color"
+            android:text="@string/about_libraries_heading"
+            android:layout_marginTop="8dp"
+            android:layout_marginBottom="8dp" />
+
         <org.wikipedia.views.AppTextView
-                android:id="@+id/activity_about_libraries"
-                android:layout_width="match_parent"
-                android:layout_height="wrap_content"
-                android:layoutDirection="ltr"
-                android:text="@string/libraries_list"
-                android:textAppearance="?android:attr/textAppearanceSmall"
-                android:textIsSelectable="true"
-                android:lineSpacingMultiplier="1.2"
-                android:textColor="#555"
-                android:layout_marginBottom="8dp"
-                />
+            android:id="@+id/activity_about_libraries"
+            android:layout_width="match_parent"
+            android:layout_height="wrap_content"
+            android:layoutDirection="ltr"
+            android:text="@string/libraries_list"
+            android:textAppearance="?android:attr/textAppearanceSmall"
+            android:textIsSelectable="true"
+            android:lineSpacingMultiplier="1.2"
+            android:textColor="?attr/secondary_text_color"
+            android:layout_marginBottom="8dp" />
 
         <TextView
-                android:layout_width="match_parent"
-                android:layout_height="wrap_content"
-                style="?android:textAppearanceSmall"
-                android:text="@string/about_app_license_heading"
-                android:layout_marginTop="8dp"
-                android:layout_marginBottom="8dp" />
+            android:layout_width="match_parent"
+            android:layout_height="wrap_content"
+            style="?android:textAppearanceSmall"
+            android:textColor="?attr/primary_text_color"
+            android:text="@string/about_app_license_heading"
+            android:layout_marginTop="8dp"
+            android:layout_marginBottom="8dp" />
+
         <org.wikipedia.views.AppTextView
-                android:id="@+id/about_app_license"
-                android:layout_width="match_parent"
-                android:layout_height="wrap_content"
-                style="@style/RtlAwareTextView"
-                android:text="@string/about_app_license"
-                android:textAppearance="?android:attr/textAppearanceSmall"
-                android:textIsSelectable="true"
-                android:lineSpacingMultiplier="1.2"
-                android:textColor="#555"
-                android:layout_marginBottom="8dp"
-                />
+            android:id="@+id/about_app_license"
+            android:layout_width="match_parent"
+            android:layout_height="wrap_content"
+            style="@style/RtlAwareTextView"
+            android:text="@string/about_app_license"
+            android:textAppearance="?android:attr/textAppearanceSmall"
+            android:textIsSelectable="true"
+            android:lineSpacingMultiplier="1.2"
+            android:textColor="?attr/secondary_text_color"
+            android:layout_marginBottom="8dp" />
 
         <TextView
-                android:id="@+id/send_feedback_text"
-                android:layout_width="match_parent"
-                android:layout_height="wrap_content"
-                tools:text="@string/send_feedback"
-                style="?android:textAppearanceMedium"
-                android:textColor="#555"
-                android:layout_marginTop="32dp"
-                android:layout_marginBottom="8dp"
-                />
+            android:id="@+id/send_feedback_text"
+            android:layout_width="match_parent"
+            android:layout_height="wrap_content"
+            tools:text="@string/send_feedback"
+            style="?android:textAppearanceMedium"
+            android:layout_marginTop="32dp"
+            android:layout_marginBottom="8dp" />
 
         <View
             android:layout_width="match_parent"
@@ -150,7 +145,7 @@
             android:layout_margin="12dp"
             android:padding="4dp"
             app:srcCompat="@drawable/ic_wmf_logo"
-            android:tint="@color/base30"
+            android:tint="?attr/secondary_text_color"
             android:contentDescription="@null" />
 
         <org.wikipedia.views.AppTextView
@@ -161,7 +156,7 @@
             style="?android:textAppearanceSmall"
             android:textIsSelectable="true"
             android:layout_weight="1"
-            android:textColor="#555" />
+            android:textColor="?attr/secondary_text_color" />
 
     </LinearLayout>
 </ScrollView>

-- 
To view, visit https://gerrit.wikimedia.org/r/394454
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib9ce17b368aa6bfca0632940cd8b81a259ce6114
Gerrit-PatchSet: 1
Gerrit-Project: apps/android/wikipedia
Gerrit-Branch: master
Gerrit-Owner: Dbrant <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to