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 ad6e0a0c6..8ce82bc82 100644 --- a/app/src/main/java/org/mozilla/fenix/search/SearchFragment.kt +++ b/app/src/main/java/org/mozilla/fenix/search/SearchFragment.kt @@ -222,6 +222,9 @@ class SearchFragment : Fragment(), UserInteractionHandler { inflated.text.text = getString(R.string.search_suggestions_onboarding_text, getString(R.string.app_name)) + + inflated.title.text = + getString(R.string.search_suggestions_onboarding_title) } view.search_suggestions_onboarding.setOnInflateListener((stubListener)) @@ -334,8 +337,12 @@ class SearchFragment : Fragment(), UserInteractionHandler { } private fun updateSearchSuggestionsHintVisibility(state: SearchFragmentState) { - view?.findViewById(R.id.search_suggestions_onboarding) - ?.isVisible = state.showSearchSuggestionsHint + view?.apply { + findViewById(R.id.search_suggestions_onboarding)?.isVisible = state.showSearchSuggestionsHint + + search_suggestions_onboarding_divider?.isVisible = + search_with_shortcuts.isVisible && state.showSearchSuggestionsHint + } } companion object { diff --git a/app/src/main/java/org/mozilla/fenix/search/SearchLayouts.kt b/app/src/main/java/org/mozilla/fenix/search/SearchLayouts.kt index 32b881ecf..c018f39d7 100644 --- a/app/src/main/java/org/mozilla/fenix/search/SearchLayouts.kt +++ b/app/src/main/java/org/mozilla/fenix/search/SearchLayouts.kt @@ -71,7 +71,7 @@ internal fun SearchFragment.setOutOfExperimentConstraints(layout: ConstraintLayo awesomeBar { connect( TOP to TOP of UNSET, - TOP to BOTTOM of awesomeBar_barrier, + TOP to BOTTOM of search_with_shortcuts, BOTTOM to TOP of pillWrapper ) } diff --git a/app/src/main/res/layout/fragment_search.xml b/app/src/main/res/layout/fragment_search.xml index b7e15f073..d49c284d3 100644 --- a/app/src/main/res/layout/fragment_search.xml +++ b/app/src/main/res/layout/fragment_search.xml @@ -23,6 +23,16 @@ app:layout_constraintBottom_toTopOf="@id/awesomeBar_barrier" app:layout_constraintTop_toBottomOf="@id/toolbar_wrapper"/> + + + app:layout_constraintStart_toStartOf="@id/toolbar_wrapper" + app:layout_constraintBottom_toBottomOf="@id/awesomeBar_barrier"> + app:constraint_referenced_ids="fill_link_from_clipboard,search_suggestions_onboarding"/> + app:layout_constraintTop_toTopOf="parent" + tools:tint="@color/contrast_text_private_theme"/> + tools:text="@string/search_suggestions_onboarding_title" + tools:textAppearance="?android:attr/textAppearanceListItem"/> + app:layout_constraintTop_toBottomOf="@id/title" + tools:text="@string/search_suggestions_onboarding_text" + tools:textAppearance="?attr/textAppearanceListItemSmall"/> + app:layout_constraintTop_toBottomOf="@id/text" + tools:textColor="@color/accent_high_contrast_private_theme"/> + app:layout_constraintTop_toBottomOf="@id/learn_more" />