Fix part of #632: Refactor topic list adapter to use bindable adapter

app/BUILD.bazel

@@ -122,6 +122,10 @@ VIEW_MODELS_WITH_RESOURCE_IMPORTS = [
     "src/main/java/org/oppia/android/app/help/HelpItemViewModel.kt",
     "src/main/java/org/oppia/android/app/help/HelpListViewModel.kt",
     "src/main/java/org/oppia/android/app/help/faq/FAQListViewModel.kt",
+    "src/main/java/org/oppia/android/app/home/HomeViewModel.kt",
+    "src/main/java/org/oppia/android/app/home/promotedlist/PromotedStoryListViewModel.kt",
+    "src/main/java/org/oppia/android/app/home/promotedlist/PromotedStoryViewModel.kt",
+    "src/main/java/org/oppia/android/app/home/topiclist/TopicSummaryViewModel.kt",
     "src/main/java/org/oppia/android/app/onboarding/OnboadingSlideViewModel.kt",
     "src/main/java/org/oppia/android/app/onboarding/OnboardingViewModel.kt",
     "src/main/java/org/oppia/android/app/parser/StringToFractionParser.kt",
@@ -168,9 +172,6 @@ VIEW_MODELS = [
     "src/main/java/org/oppia/android/app/home/recentlyplayed/RecentlyPlayedItemViewModel.kt",
     "src/main/java/org/oppia/android/app/home/recentlyplayed/SectionTitleViewModel.kt",
     "src/main/java/org/oppia/android/app/home/topiclist/AllTopicsViewModel.kt",
-    "src/main/java/org/oppia/android/app/home/topiclist/PromotedStoryListViewModel.kt",
-    "src/main/java/org/oppia/android/app/home/topiclist/PromotedStoryViewModel.kt",
-    "src/main/java/org/oppia/android/app/home/topiclist/TopicSummaryViewModel.kt",
     "src/main/java/org/oppia/android/app/home/UserAppHistoryViewModel.kt",
     "src/main/java/org/oppia/android/app/home/WelcomeViewModel.kt",
     "src/main/java/org/oppia/android/app/onboarding/OnboardingSlideFinalViewModel.kt",
@@ -312,6 +313,7 @@ VIEWS = [
     "src/main/java/org/oppia/android/app/customview/interaction/NumericInputInteractionView.kt",
     "src/main/java/org/oppia/android/app/customview/interaction/TextInputInteractionView.kt",
     "src/main/java/org/oppia/android/app/customview/interaction/RatioInputInteractionView.kt",
+    "src/main/java/org/oppia/android/app/home/promotedlist/PromotedStoryListView.kt",
     "src/main/java/org/oppia/android/app/player/state/DragDropSortInteractionView.kt",
     "src/main/java/org/oppia/android/app/player/state/ImageRegionSelectionInteractionView.kt",
     "src/main/java/org/oppia/android/app/player/state/SelectionInteractionView.kt",
@@ -405,6 +407,8 @@ in order to keep Gradle building.
      """,
 ) for file in BINDING_ADAPTERS_WITH_RESOURCE_IMPORTS]
 
+SNAP_HELPER = ["src/main/java/org/oppia/android/app/recyclerview/StartSnapHelper.kt"]
+
 """
 Files to be excluded from the source list for the app library. This list includes all the files
 built by the following libraries:
@@ -415,7 +419,7 @@ built by the following libraries:
 - binding_adapters
 """
 
-EXCLUDED_APP_LIB_FILES = VIEW_MODELS + VIEW_MODELS_WITH_RESOURCE_IMPORTS + VIEWS + VIEWS_WITH_RESOURCE_IMPORTS + BINDING_ADAPTERS + BINDING_ADAPTERS_WITH_RESOURCE_IMPORTS + LISTENERS + ANNOTATIONS
+EXCLUDED_APP_LIB_FILES = VIEW_MODELS + VIEW_MODELS_WITH_RESOURCE_IMPORTS + VIEWS + VIEWS_WITH_RESOURCE_IMPORTS + BINDING_ADAPTERS + BINDING_ADAPTERS_WITH_RESOURCE_IMPORTS + LISTENERS + ANNOTATIONS + SNAP_HELPER
 
 """
 Files to be built by the app library. All of these files are assumed to have resource imports and
@@ -530,6 +534,7 @@ kt_android_library(
         ":listeners",
         ":resources",
         ":view_models",
+        ":snap_helper",
         "//model",
         "@circularimageview//circularimageview:circular_image_view",
         artifact("androidx.appcompat:appcompat"),
@@ -572,6 +577,14 @@ kt_android_library(
     ],
 )
 
+# Library for snap helper used to build views.
+kt_android_library(
+    name = "snap_helper",
+    srcs = SNAP_HELPER,
+    custom_package = "org.oppia.android.app.recyclerview",
+    deps = [artifact("androidx.recyclerview:recyclerview:1.0.0"),],
+)
+
 # TODO(#1639): Translate binding adapters back to Kotlin post-Bazel modularization.
 # Library for all custom data-binding adapters under org/oppia/android/app/databinding.
 android_library(

app/src/main/AndroidManifest.xml

@@ -126,6 +126,7 @@
     <activity
       android:name=".app.testing.TestFontScaleConfigurationUtilActivity"
       android:theme="@style/OppiaThemeWithoutActionBar" />
+    <activity android:name=".app.testing.HomeFragmentTestActivity" />
     <activity android:name=".app.testing.HomeInjectionActivity" />
     <activity android:name=".app.testing.HtmlParserTestActivity" />
     <activity android:name=".app.testing.HomeTestActivity" />

app/src/main/java/org/oppia/android/app/activity/ActivityComponent.kt

@@ -40,6 +40,7 @@ import org.oppia.android.app.testing.ConceptCardFragmentTestActivity
 import org.oppia.android.app.testing.DragDropTestActivity
 import org.oppia.android.app.testing.ExplorationInjectionActivity
 import org.oppia.android.app.testing.ExplorationTestActivity
+import org.oppia.android.app.testing.HomeFragmentTestActivity
 import org.oppia.android.app.testing.HomeInjectionActivity
 import org.oppia.android.app.testing.HomeTestActivity
 import org.oppia.android.app.testing.HtmlParserTestActivity
@@ -90,6 +91,7 @@ interface ActivityComponent {
   fun inject(testFontScaleConfigurationUtilActivity: TestFontScaleConfigurationUtilActivity)
   fun inject(helpActivity: HelpActivity)
   fun inject(homeActivity: HomeActivity)
+  fun inject(homeFragmentTestActivity: HomeFragmentTestActivity)
   fun inject(homeInjectionActivity: HomeInjectionActivity)
   fun inject(homeTestActivity: HomeTestActivity)
   fun inject(htmlParserTestActivity: HtmlParserTestActivity)

app/src/main/java/org/oppia/android/app/databinding/ViewBindingAdapters.java

@@ -12,7 +12,9 @@
 public final class ViewBindingAdapters {
 
   /**
-   * BindingAdapter to set the height of a View.
+   * BindingAdapter to set the height of a View. If this value is calculated in data fetching, the
+   * layout will require a default value since binding adapters aren't called until after initial
+   * view measurements and layouts are formatted.
    */
   @BindingAdapter("android:layout_height")
   public static void setLayoutHeight(@NonNull View view, float height) {
@@ -21,6 +23,18 @@ public static void setLayoutHeight(@NonNull View view, float height) {
     view.setLayoutParams(layoutParams);
   }
 
+  /**
+   * BindingAdapter to set the width of a View. If this value is calculated in data fetching, the
+   * layout will require a default value since binding adapters aren't called until after initial
+   * view measurements and layouts are formatted.
+   */
+  @BindingAdapter("android:layout_width")
+  public static void setLayoutWidth(@NonNull View view, float width) {
+    ViewGroup.LayoutParams layoutParams = view.getLayoutParams();
+    layoutParams.width = (int) width;
+    view.setLayoutParams(layoutParams);
+  }
+
   @BindingAdapter(
       value = {
           "app:isRotationAnimationClockwise",

app/src/main/java/org/oppia/android/app/home/HomeActivity.kt

@@ -8,11 +8,16 @@ import org.oppia.android.app.activity.InjectableAppCompatActivity
 import org.oppia.android.app.drawer.ExitProfileDialogFragment
 import org.oppia.android.app.drawer.KEY_NAVIGATION_PROFILE_ID
 import org.oppia.android.app.drawer.TAG_SWITCH_PROFILE_DIALOG
+import org.oppia.android.app.home.recentlyplayed.RecentlyPlayedActivity
 import org.oppia.android.app.topic.TopicActivity
 import javax.inject.Inject
 
 /** The central activity for all users entering the app. */
-class HomeActivity : InjectableAppCompatActivity(), RouteToTopicListener {
+class HomeActivity :
+  InjectableAppCompatActivity(),
+  RouteToTopicListener,
+  RouteToTopicPlayStoryListener,
+  RouteToRecentlyPlayedListener {
   @Inject
   lateinit var homeActivityPresenter: HomeActivityPresenter
   private var internalProfileId: Int = -1
@@ -47,4 +52,24 @@ class HomeActivity : InjectableAppCompatActivity(), RouteToTopicListener {
       .newInstance(isFromNavigationDrawer = false)
     dialogFragment.showNow(supportFragmentManager, TAG_SWITCH_PROFILE_DIALOG)
   }
+
+  override fun routeToTopicPlayStory(internalProfileId: Int, topicId: String, storyId: String) {
+    startActivity(
+      TopicActivity.createTopicPlayStoryActivityIntent(
+        this,
+        internalProfileId,
+        topicId,
+        storyId
+      )
+    )
+  }
+
+  override fun routeToRecentlyPlayed() {
+    startActivity(
+      RecentlyPlayedActivity.createRecentlyPlayedActivityIntent(
+        this,
+        internalProfileId
+      )
+    )
+  }
 }