[Search Bar] Mistake in documentation

[dliubarskyi]

Description: It looks like there is a mistake in Search Bar documentation.
There should be app:layout_behavior="@string/appbar_scrolling_view_behavior" instead of app:layout_behavior="@string/searchbar_scrolling_view_behavior", otherwise first item from RecyclerView is displayed behind the search bar.

[drchen]

I think the documentation is supposed to be correct. Can you provide your layout XML and code snippet if you are doing anything other than inflating the layout?

[dliubarskyi]

Hi @drchen I took the layout 1:1 from documentation. In above documentation searchbar_scrolling_view_behavior is used (which does not work for me). In io/material/catalog/search/res/layout/cat_search_fragment.xml appbar_scrolling_view_behavior is used. When I use layout from above documentation, but use appbar_scrolling_view_behavior instead of searchbar_scrolling_view_behavior, everything works as expected. Also, when I use searchbar_scrolling_view_behavior, Android Studio shows warning "The resource @string/searchbar_scrolling_view_behavior is marked as private in com.google.android.material:material:1.9.0-rc01". So, my assumption is, there is a mistake in documentation.

[drchen]

Gotcha. Thanks for the info. Raaj, can you take a look?

[raajkumars]

@ljubarskij Thanks for reporting this issue. @drchen Thanks for following up. searchbar_scrolling_view_behavior is intended behavior for search components. I will look into the issue reported in this thread.

[ryansuhartanto]

Any update here?
Seems like searchbar_scrolling_view_behavior exists in the library but somehow it's set to be private.

[manabu-nakamura]

SearchMainDemoActivity (https://github.com/material-components/material-components-android/blob/master/catalog/java/io/material/catalog/search/res/layout/cat_search_fragment.xml):

• searchbar_scrolling_view_behavior

  <androidx.core.widget.NestedScrollView
      android:id="@+id/nested_scroll_view"
      android:layout_width="match_parent"
      android:layout_height="match_parent"
-     app:layout_behavior="@string/appbar_scrolling_view_behavior">
+     app:layout_behavior="@string/searchbar_scrolling_view_behavior">

• appbar_scrolling_view_behavior

  <androidx.core.widget.NestedScrollView
      android:id="@+id/nested_scroll_view"
      android:layout_width="match_parent"
      android:layout_height="match_parent"
      app:layout_behavior="@string/appbar_scrolling_view_behavior">

[manabu-nakamura]

https://github.com/material-components/material-components-android/blob/master/catalog/java/io/material/catalog/search/res/layout/cat_search_recycler_fragment.xml#L24-L26:

  <!-- Use AppBarLayout scrolling behavior instead of SearchBar
       scrolling behavior since this demo uses an opaque AppBarLayout
       and a non-floating SearchBar. -->