diff --git a/app/src/main/java/org/mozilla/fenix/browser/BrowserFragment.kt b/app/src/main/java/org/mozilla/fenix/browser/BrowserFragment.kt index 7c607ecd1..efba6023e 100644 --- a/app/src/main/java/org/mozilla/fenix/browser/BrowserFragment.kt +++ b/app/src/main/java/org/mozilla/fenix/browser/BrowserFragment.kt @@ -135,6 +135,7 @@ class BrowserFragment : Fragment(), BackHandler, CoroutineScope { view.browserLayout, ActionBusFactory.get(this), customTabSessionId, (activity as HomeActivity).browsingModeManager.isPrivate, + false, search_engine_icon, FenixViewModelProvider.create( this, diff --git a/app/src/main/java/org/mozilla/fenix/components/toolbar/ToolbarComponent.kt b/app/src/main/java/org/mozilla/fenix/components/toolbar/ToolbarComponent.kt index f7ac658aa..f887c7120 100644 --- a/app/src/main/java/org/mozilla/fenix/components/toolbar/ToolbarComponent.kt +++ b/app/src/main/java/org/mozilla/fenix/components/toolbar/ToolbarComponent.kt @@ -26,6 +26,7 @@ class ToolbarComponent( bus: ActionBusFactory, private val sessionId: String?, private val isPrivate: Boolean, + private val inSearchFragment: Boolean, private val engineIconView: ImageView? = null, viewModelProvider: UIComponentViewModelProvider ) : @@ -40,6 +41,7 @@ class ToolbarComponent( override fun initView() = ToolbarUIView( sessionId, isPrivate, + inSearchFragment, container, actionEmitter, changesObservable, diff --git a/app/src/main/java/org/mozilla/fenix/components/toolbar/ToolbarUIView.kt b/app/src/main/java/org/mozilla/fenix/components/toolbar/ToolbarUIView.kt index 59ad3d386..97524d29f 100644 --- a/app/src/main/java/org/mozilla/fenix/components/toolbar/ToolbarUIView.kt +++ b/app/src/main/java/org/mozilla/fenix/components/toolbar/ToolbarUIView.kt @@ -24,6 +24,7 @@ import org.mozilla.fenix.mvi.UIView class ToolbarUIView( sessionId: String?, isPrivate: Boolean, + inSearchFragment: Boolean, container: ViewGroup, actionEmitter: Observer, changesObservable: Observable, @@ -48,6 +49,10 @@ class ToolbarUIView( ?: sessionManager.selectedSession view.apply { + if (inSearchFragment) { + editMode() + } + elevation = resources.pxToDp(TOOLBAR_ELEVATION).toFloat() setOnUrlCommitListener { diff --git a/app/src/main/java/org/mozilla/fenix/search/SearchFragment.kt b/app/src/main/java/org/mozilla/fenix/search/SearchFragment.kt index ced178e21..84dac8958 100644 --- a/app/src/main/java/org/mozilla/fenix/search/SearchFragment.kt +++ b/app/src/main/java/org/mozilla/fenix/search/SearchFragment.kt @@ -83,6 +83,7 @@ class SearchFragment : Fragment(), BackHandler { ActionBusFactory.get(this), sessionId, isPrivate, + true, view.search_engine_icon, FenixViewModelProvider.create( this, diff --git a/app/src/main/res/layout/fragment_home.xml b/app/src/main/res/layout/fragment_home.xml index c0f3eb957..2c79419aa 100644 --- a/app/src/main/res/layout/fragment_home.xml +++ b/app/src/main/res/layout/fragment_home.xml @@ -50,38 +50,38 @@ + android:layout_height="24dp" + android:layout_margin="12dp" + android:transitionName="toolbar_icon_transition" /> + android:textSize="15sp" + android:transitionName="toolbar_transition" /> + android:background="?above" + tools:context=".search.SearchFragment"> + + app:layout_constraintTop_toTopOf="parent" /> + + app:layout_constraintTop_toTopOf="parent" /> + android:id="@+id/search_with_shortcuts" + android:layout_width="wrap_content" + android:layout_height="wrap_content" + android:layout_marginTop="15dp" + android:fontFamily="Inter UI" + android:letterSpacing="0.15" + android:text="@string/search_shortcuts_search_with" + android:textAllCaps="true" + android:textColor="?secondaryText" + android:textSize="12sp" + android:textStyle="bold" + android:visibility="gone" + app:layout_constraintStart_toStartOf="@id/toolbar_wrapper" + app:layout_constraintTop_toBottomOf="@id/toolbar_wrapper" /> + app:layout_constraintStart_toStartOf="parent">