[UPDATE] Added different icon for technical article.

hossain-khan · Jun 20, 2019 · d20abff · d20abff
1 parent e25ea84
commit d20abff
Show file tree

Hide file tree

Showing 5 changed files with 32 additions and 196 deletions.
diff --git a/.idea/assetWizardSettings.xml b/.idea/assetWizardSettings.xml
diff --git a/app/src/main/java/com/hossainkhan/android/demo/data/ResourceInfo.kt b/app/src/main/java/com/hossainkhan/android/demo/data/ResourceInfo.kt
@@ -41,8 +41,8 @@ data class ResourceInfo(
         var url: String = "",
         var publish_date: Timestamp? = null,
         /**
-         * Possible values
-         * "blog", "techtalk"
+         * Possible values:
+         * "techarticle", "techtalk"
          */
         var type: String = "techtalk"
 ) {

diff --git a/app/src/main/java/com/hossainkhan/android/demo/ui/resource/ResourceListAdapter.kt b/app/src/main/java/com/hossainkhan/android/demo/ui/resource/ResourceListAdapter.kt
@@ -45,7 +45,7 @@ class ResourceListAdapter(
                 parent, false
         )
 
-        binding.actionPlay.setOnClickListener {
+        binding.actionPreviewResource.setOnClickListener {
             binding.data?.let {
                 itemClickCallback?.invoke(it)
             }

diff --git a/app/src/main/res/drawable/ic_library_books_black_42dp.xml b/app/src/main/res/drawable/ic_library_books_black_42dp.xml
@@ -0,0 +1,20 @@
+<!--
+  ~ Copyright (c) 2019 Hossain Khan
+  ~
+  ~ Licensed under the Apache License, Version 2.0 (the "License");
+  ~ you may not use this file except in compliance with the License.
+  ~ You may obtain a copy of the License at
+  ~
+  ~     http://www.apache.org/licenses/LICENSE-2.0
+  ~
+  ~ Unless required by applicable law or agreed to in writing, software
+  ~ distributed under the License is distributed on an "AS IS" BASIS,
+  ~ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+  ~ See the License for the specific language governing permissions and
+  ~ limitations under the License.
+  -->
+
+<vector android:height="42dp" android:viewportHeight="24.0"
+    android:viewportWidth="24.0" android:width="42dp" xmlns:android="http://schemas.android.com/apk/res/android">
+    <path android:fillColor="#FF000000" android:pathData="M4,6L2,6v14c0,1.1 0.9,2 2,2h14v-2L4,20L4,6zM20,2L8,2c-1.1,0 -2,0.9 -2,2v12c0,1.1 0.9,2 2,2h12c1.1,0 2,-0.9 2,-2L22,4c0,-1.1 -0.9,-2 -2,-2zM19,11L9,11L9,9h10v2zM15,15L9,15v-2h6v2zM19,7L9,7L9,5h10v2z"/>
+</vector>
diff --git a/app/src/main/res/layout/list_item_resource_tech_talk.xml b/app/src/main/res/layout/list_item_resource_tech_talk.xml
@@ -34,14 +34,17 @@
             tools:background="#ccc">
 
             <ImageButton
-                android:id="@+id/action_play"
+                android:id="@+id/action_preview_resource"
                 style="@style/Widget.AppCompat.Button.Borderless"
                 android:layout_width="64dp"
                 android:layout_height="wrap_content"
-                android:tint="@color/md_red_500"
+                android:src="@{safeUnbox(data.youTubeUrl)? @drawable/ic_play_circle_filled_black_42dp : @drawable/ic_library_books_black_42dp}"
+                android:tint="@{safeUnbox(data.youTubeUrl)? @color/md_red_500 : @color/md_cyan_800}"
                 app:layout_constraintStart_toStartOf="@+id/guideline_vertical_start"
                 app:layout_constraintTop_toTopOf="@+id/resource_title"
-                app:srcCompat="@drawable/ic_play_circle_filled_black_42dp" />
+                tools:ignore="ContentDescription"
+                tools:src="@drawable/ic_play_circle_filled_black_42dp"
+                tools:tint="@color/md_red_500" />
 
             <TextView
                 android:id="@+id/resource_summary"
@@ -71,12 +74,12 @@
                 android:id="@+id/resource_title"
                 style="@style/TextAppearance.AppCompat.Title"
                 android:layout_width="0dp"
-                android:gravity="end"
                 android:layout_height="wrap_content"
                 android:layout_marginTop="@dimen/material_vertical_margin"
+                android:gravity="end"
                 android:text="@{data.title}"
                 app:layout_constraintEnd_toEndOf="@id/guideline_vertical_end"
-                app:layout_constraintStart_toEndOf="@id/action_play"
+                app:layout_constraintStart_toEndOf="@id/action_preview_resource"
                 app:layout_constraintTop_toTopOf="parent"
                 tools:text="This is the title of the talk. This can become multi line" />
 
@@ -90,7 +93,7 @@
                 android:gravity="end"
                 android:text="@{@string/tech_talk_author_and_venue(data.author, data.event)}"
                 app:layout_constraintEnd_toEndOf="@id/guideline_vertical_end"
-                app:layout_constraintStart_toEndOf="@+id/action_play"
+                app:layout_constraintStart_toEndOf="@+id/action_preview_resource"
                 app:layout_constraintTop_toBottomOf="@+id/resource_title"
                 tools:text="by Author of the talk." />