[fenix] For mozilla-mobile/fenix#25810 - Unified search should give h…

…int to user that there are more menu items available by scrolling (mozilla-mobile/fenix#27895) * For mozilla-mobile/fenix#25810 - Unified search should give hint to user that there are more menu items available by scrolling * For mozilla-mobile/fenix#25810: Updated AC version Co-authored-by: t-p-white <t-p-white>
mozilla-releng · Jan 9, 2023 · 89b37d4 · 89b37d4
1 parent ac88c88
commit 89b37d4
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 5 deletions.
diff --git a/fenix/app/src/main/java/org/mozilla/fenix/home/HomeFragment.kt b/fenix/app/src/main/java/org/mozilla/fenix/home/HomeFragment.kt
@@ -361,7 +361,7 @@ class HomeFragment : Fragment() {
                 }
 
                 UnifiedSearch.searchMenuTapped.record(NoExtras())
-                searchSelectorMenu.menuController.show(anchor = it, orientation = orientation, forceOrientation = true)
+                searchSelectorMenu.menuController.show(anchor = it, orientation = orientation)
             }
         }
 

diff --git a/fenix/app/src/main/java/org/mozilla/fenix/search/toolbar/SearchSelectorToolbarAction.kt b/fenix/app/src/main/java/org/mozilla/fenix/search/toolbar/SearchSelectorToolbarAction.kt
@@ -63,7 +63,7 @@ class SearchSelectorToolbarAction(
                 }
 
                 UnifiedSearch.searchMenuTapped.record(NoExtras())
-                menu.menuController.show(anchor = it, orientation = orientation, forceOrientation = true)
+                menu.menuController.show(anchor = it, orientation = orientation)
             }
 
             val topPadding = resources.getDimensionPixelSize(R.dimen.search_engine_engine_icon_top_margin)

diff --git a/fenix/app/src/test/java/org/mozilla/fenix/search/toolbar/SearchSelectorToolbarActionTest.kt b/fenix/app/src/test/java/org/mozilla/fenix/search/toolbar/SearchSelectorToolbarActionTest.kt
@@ -87,15 +87,15 @@ class SearchSelectorToolbarActionTest {
 
         assertNotNull(UnifiedSearch.searchMenuTapped.testGetValue())
         verify {
-            menu.menuController.show(view, Orientation.DOWN, true)
+            menu.menuController.show(view, Orientation.DOWN)
         }
 
         every { settings.shouldUseBottomToolbar } returns true
         view.performClick()
 
         assertNotNull(UnifiedSearch.searchMenuTapped.testGetValue())
         verify {
-            menu.menuController.show(view, Orientation.UP, true)
+            menu.menuController.show(view, Orientation.UP)
         }
     }
 

diff --git a/fenix/buildSrc/src/main/java/AndroidComponents.kt b/fenix/buildSrc/src/main/java/AndroidComponents.kt
@@ -3,5 +3,5 @@
  * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
 
 object AndroidComponents {
-    const val VERSION = "110.0.20230108143241"
+    const val VERSION = "110.0.20230109143319"
 }
-Original file line number
+Diff line change
@@ Expand Up / @@ -361,7 +361,7 @@ class HomeFragment : Fragment() { @@
                     }
                     UnifiedSearch.searchMenuTapped.record(NoExtras())
-                    searchSelectorMenu.menuController.show(anchor = it, orientation = orientation, forceOrientation = true)
+                    searchSelectorMenu.menuController.show(anchor = it, orientation = orientation)
                 }
             }
@@ Expand Down @@