Add slide in-out banner animation when scrolling search results

Author: ovitrif

WordPress/src/main/java/org/wordpress/android/ui/reader/ReaderPostListFragment.java

@@ -72,6 +72,7 @@
 import org.wordpress.android.ui.EmptyViewMessageType;
 import org.wordpress.android.ui.FilteredRecyclerView;
 import org.wordpress.android.ui.RequestCodes;
+import org.wordpress.android.ui.ScrollableViewInitializedListener;
 import org.wordpress.android.ui.ViewPagerFragment;
 import org.wordpress.android.ui.main.BottomNavController;
 import org.wordpress.android.ui.main.SitePickerActivity;
@@ -773,6 +774,11 @@ public void onResume() {
             showEmptyView();
         }
 
+        if (getActivity() instanceof ScrollableViewInitializedListener) {
+            ((ScrollableViewInitializedListener) getActivity())
+                    .onScrollableViewInitialized(mRecyclerView.getInternalRecyclerView().getId());
+        }
+
         mViewModel.onFragmentResume(mIsTopLevel, isSearching(), isFilterableScreen(),
                 isFilterableScreen() ? mSubFilterViewModel.getCurrentSubfilterValue() : null);
     }
@@ -2651,7 +2657,7 @@ public void onFollowTapped(View view, String blogName, final long blogId, final
 
         if (blogId > 0) {
             WPSnackbar.make(getSnackbarParent(), Html.fromHtml(getString(R.string.reader_followed_blog_notifications,
-                    "<b>", blog, "</b>")), Snackbar.LENGTH_LONG)
+                              "<b>", blog, "</b>")), Snackbar.LENGTH_LONG)
                       .setAction(getString(R.string.reader_followed_blog_notifications_action),
                               new View.OnClickListener() {
                                   @Override public void onClick(View view) {

WordPress/src/main/java/org/wordpress/android/ui/reader/ReaderSearchActivity.kt

@@ -1,12 +1,16 @@
 package org.wordpress.android.ui.reader
 
 import android.os.Bundle
+import android.view.View
+import androidx.recyclerview.widget.RecyclerView
 import dagger.hilt.android.AndroidEntryPoint
 import org.wordpress.android.R
 import org.wordpress.android.WordPress
 import org.wordpress.android.ui.LocaleAwareActivity
+import org.wordpress.android.ui.ScrollableViewInitializedListener
 import org.wordpress.android.ui.reader.tracker.ReaderTracker
 import org.wordpress.android.ui.reader.tracker.ReaderTrackerType.MAIN_READER
+import org.wordpress.android.util.JetpackBrandingUtils
 import javax.inject.Inject
 
 /**
@@ -15,8 +19,10 @@ import javax.inject.Inject
  * into new tested classes without requiring us to change the search behavior.
  */
 @AndroidEntryPoint
-class ReaderSearchActivity : LocaleAwareActivity() {
+class ReaderSearchActivity : LocaleAwareActivity(),
+        ScrollableViewInitializedListener {
     @Inject lateinit var readerTracker: ReaderTracker
+    @Inject lateinit var jetpackBrandingUtils: JetpackBrandingUtils
 
     override fun onCreate(savedInstanceState: Bundle?) {
         super.onCreate(savedInstanceState)
@@ -41,4 +47,24 @@ class ReaderSearchActivity : LocaleAwareActivity() {
         super.onPause()
         readerTracker.stop(MAIN_READER)
     }
+
+    override fun onScrollableViewInitialized(containerId: Int) {
+        val fragmentContainer = supportFragmentManager.findFragmentById(R.id.fragment_container)
+
+        if (fragmentContainer is ReaderPostListFragment) {
+            val fragmentView = fragmentContainer.view ?: return
+
+            fragmentView.post {
+                // post is used to create a minimal delay here. containerId changes just before
+                // onScrollableViewInitialized is called, and findViewById can't find the new id before the delay.
+                val jetpackBannerView = fragmentView.findViewById<View>(R.id.jetpack_banner)
+                val scrollableView = fragmentView.findViewById<View>(containerId) as RecyclerView
+
+                if (jetpackBrandingUtils.shouldShowJetpackBranding()) {
+                    jetpackBrandingUtils.showJetpackBannerIfScrolledToTop(jetpackBannerView, scrollableView)
+                    jetpackBrandingUtils.initJetpackBannerAnimation(jetpackBannerView, scrollableView)
+                }
+            }
+        }
+    }
 }