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 0a0a30c2d8..73a7d26da2 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 3d6d404653..e698621385 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"/>