diff --git a/app/src/main/java/org/mozilla/fenix/home/HomeFragment.kt b/app/src/main/java/org/mozilla/fenix/home/HomeFragment.kt index 2c7ee9203..174adcc84 100644 --- a/app/src/main/java/org/mozilla/fenix/home/HomeFragment.kt +++ b/app/src/main/java/org/mozilla/fenix/home/HomeFragment.kt @@ -192,10 +192,10 @@ class HomeFragment : Fragment() { view.homeLayout.applyConstraintSet { sessionControlComponent.view { connect( - TOP to BOTTOM of view.wordmark, + TOP to BOTTOM of view.wordmark_spacer, START to START of PARENT_ID, END to END of PARENT_ID, - BOTTOM to BOTTOM of PARENT_ID + BOTTOM to TOP of view.bottom_bar ) } } @@ -235,33 +235,32 @@ class HomeFragment : Fragment() { searchIcon.setBounds(0, 0, iconSize, iconSize) withContext(Dispatchers.Main) { - // TODO: FIXME -// search_engine_icon?.setImageDrawable(searchIcon) + search_engine_icon?.setImageDrawable(searchIcon) } } -// view.menuButton.setOnClickListener { -// homeMenu?.menuBuilder?.build(requireContext())?.show( -// anchor = it, -// orientation = BrowserMenu.Orientation.DOWN -// ) -// } -// view.toolbar.compoundDrawablePadding = + view.menuButton.setOnClickListener { + homeMenu?.menuBuilder?.build(requireContext())?.show( + anchor = it, + orientation = BrowserMenu.Orientation.DOWN + ) + } + view.toolbar.compoundDrawablePadding = view.resources.getDimensionPixelSize(R.dimen.search_bar_search_engine_icon_padding) -// view.toolbar_wrapper.setOnClickListener { -// invokePendingDeleteJobs() -// onboarding.finish() -// val directions = HomeFragmentDirections.actionHomeFragmentToSearchFragment( -// sessionId = null, -// showShortcutEnginePicker = true -// ) -// val extras = -// FragmentNavigator.Extras.Builder() -// .addSharedElement(toolbar_wrapper, "toolbar_wrapper_transition") -// .build() -// nav(R.id.homeFragment, directions, extras) -// requireComponents.analytics.metrics.track(Event.SearchBarTapped(Event.SearchBarTapped.Source.HOME)) -// } + view.toolbar_wrapper.setOnClickListener { + invokePendingDeleteJobs() + onboarding.finish() + val directions = HomeFragmentDirections.actionHomeFragmentToSearchFragment( + sessionId = null, + showShortcutEnginePicker = true + ) + val extras = + FragmentNavigator.Extras.Builder() + .addSharedElement(toolbar_wrapper, "toolbar_wrapper_transition") + .build() + nav(R.id.homeFragment, directions, extras) + requireComponents.analytics.metrics.track(Event.SearchBarTapped(Event.SearchBarTapped.Source.HOME)) + } PrivateBrowsingButtonView( privateBrowsingButton, diff --git a/app/src/main/java/org/mozilla/fenix/home/sessioncontrol/viewholders/TabHeaderViewHolder.kt b/app/src/main/java/org/mozilla/fenix/home/sessioncontrol/viewholders/TabHeaderViewHolder.kt index 131603125..30afbe2df 100644 --- a/app/src/main/java/org/mozilla/fenix/home/sessioncontrol/viewholders/TabHeaderViewHolder.kt +++ b/app/src/main/java/org/mozilla/fenix/home/sessioncontrol/viewholders/TabHeaderViewHolder.kt @@ -41,12 +41,6 @@ class TabHeaderViewHolder( } view.apply { - add_tab_button.run { - setOnClickListener { - actionEmitter.onNext(TabAction.Add) - } - } - share_tabs_button.run { setOnClickListener { actionEmitter.onNext(TabAction.ShareTabs) diff --git a/app/src/main/res/layout/fragment_home.xml b/app/src/main/res/layout/fragment_home.xml index d8f02d9e1..efc1b4241 100644 --- a/app/src/main/res/layout/fragment_home.xml +++ b/app/src/main/res/layout/fragment_home.xml @@ -2,7 +2,8 @@ - + + + + + + + + + + + + + + + + + + \ No newline at end of file diff --git a/app/src/main/res/layout/tab_header.xml b/app/src/main/res/layout/tab_header.xml index 45615fda8..b2080a4c6 100644 --- a/app/src/main/res/layout/tab_header.xml +++ b/app/src/main/res/layout/tab_header.xml @@ -18,22 +18,10 @@ android:text="@string/tab_header_label" android:textAppearance="@style/HeaderTextStyle" app:layout_constraintBottom_toBottomOf="parent" - app:layout_constraintEnd_toStartOf="@id/add_tab_button" + app:layout_constraintEnd_toStartOf="@id/share_tabs_button" app:layout_constraintStart_toStartOf="parent" app:layout_constraintTop_toTopOf="parent" /> - -