diff --git a/app/src/main/java/org/mozilla/fenix/home/sessioncontrol/viewholders/TabViewHolder.kt b/app/src/main/java/org/mozilla/fenix/home/sessioncontrol/viewholders/TabViewHolder.kt index 0a0a30c2d..73a7d26da 100644 --- a/app/src/main/java/org/mozilla/fenix/home/sessioncontrol/viewholders/TabViewHolder.kt +++ b/app/src/main/java/org/mozilla/fenix/home/sessioncontrol/viewholders/TabViewHolder.kt @@ -56,7 +56,7 @@ class TabViewHolder( } close_tab_button?.run { - increaseTapArea(closeButtonIncreaseDps) + increaseTapArea(buttonIncreaseDps) setOnClickListener { actionEmitter.onNext(TabAction.Close(tab?.sessionId!!)) } @@ -75,10 +75,13 @@ class TabViewHolder( } } - tab_overflow_button.setOnClickListener { - tabMenu.menuBuilder - .build(view.context) - .show(anchor = it, orientation = BrowserMenu.Orientation.DOWN) + tab_overflow_button.run { + increaseTapArea(buttonIncreaseDps) + setOnClickListener { + tabMenu.menuBuilder + .build(view.context) + .show(anchor = it, orientation = BrowserMenu.Orientation.DOWN) + } } } @@ -106,7 +109,7 @@ class TabViewHolder( companion object { const val LAYOUT_ID = R.layout.tab_list_row - const val closeButtonIncreaseDps = 12 + const val buttonIncreaseDps = 12 const val favIconBorderRadiusInPx = 8 } } diff --git a/app/src/main/res/layout/tab_list_row.xml b/app/src/main/res/layout/tab_list_row.xml index 3d6d40465..e69862138 100644 --- a/app/src/main/res/layout/tab_list_row.xml +++ b/app/src/main/res/layout/tab_list_row.xml @@ -77,15 +77,15 @@ + app:layout_constraintTop_toBottomOf="@id/close_tab_button" + app:layout_constraintBottom_toBottomOf="parent"/>