fixes #4221 - The tap area for Open tabs' new tab and 3 dot menu buttons is too small
changes size of buttons to 48X48 following Material Design Accessibility guidelines. removed margins from end of layout and spaces between buttons to provide consistency with the buttons from the topmaster
parent
ae74241636
commit
30ecbc6000
|
@ -3,49 +3,45 @@
|
||||||
- License, v. 2.0. If a copy of the MPL was not distributed with this
|
- License, v. 2.0. If a copy of the MPL was not distributed with this
|
||||||
- file, You can obtain one at http://mozilla.org/MPL/2.0/. -->
|
- file, You can obtain one at http://mozilla.org/MPL/2.0/. -->
|
||||||
<androidx.constraintlayout.widget.ConstraintLayout
|
<androidx.constraintlayout.widget.ConstraintLayout
|
||||||
xmlns:android="http://schemas.android.com/apk/res/android"
|
xmlns:android="http://schemas.android.com/apk/res/android"
|
||||||
xmlns:app="http://schemas.android.com/apk/res-auto"
|
xmlns:app="http://schemas.android.com/apk/res-auto"
|
||||||
android:id="@+id/tabs_header"
|
android:id="@+id/tabs_header"
|
||||||
android:layout_marginTop="4dp"
|
android:layout_marginBottom="8dp"
|
||||||
android:layout_marginBottom="20dp"
|
android:layout_width="match_parent"
|
||||||
android:layout_width="match_parent"
|
android:layout_height="wrap_content">
|
||||||
android:layout_height="wrap_content">
|
|
||||||
|
|
||||||
<TextView
|
<TextView
|
||||||
android:id="@+id/header_text"
|
android:id="@+id/header_text"
|
||||||
android:layout_width="wrap_content"
|
android:layout_width="wrap_content"
|
||||||
android:layout_height="wrap_content"
|
android:layout_height="wrap_content"
|
||||||
android:layout_marginStart="4.5dp"
|
android:layout_marginStart="4.5dp"
|
||||||
android:text="@string/tab_header_label"
|
android:text="@string/tab_header_label"
|
||||||
android:textAppearance="@style/HeaderTextStyle"
|
android:textAppearance="@style/HeaderTextStyle"
|
||||||
app:layout_constraintBottom_toBottomOf="parent"
|
app:layout_constraintBottom_toBottomOf="parent"
|
||||||
app:layout_constraintStart_toStartOf="parent"
|
app:layout_constraintStart_toStartOf="parent"
|
||||||
app:layout_constraintTop_toTopOf="parent" />
|
app:layout_constraintTop_toTopOf="parent" />
|
||||||
|
|
||||||
<LinearLayout
|
<LinearLayout
|
||||||
android:layout_width="wrap_content"
|
android:layout_width="wrap_content"
|
||||||
android:layout_height="wrap_content"
|
android:layout_height="wrap_content"
|
||||||
android:orientation="horizontal"
|
android:orientation="horizontal"
|
||||||
android:layout_marginEnd="4.5dp"
|
app:layout_constraintBottom_toBottomOf="parent"
|
||||||
app:layout_constraintBottom_toBottomOf="parent"
|
app:layout_constraintTop_toTopOf="parent"
|
||||||
app:layout_constraintTop_toTopOf="parent"
|
app:layout_constraintEnd_toEndOf="parent">
|
||||||
app:layout_constraintEnd_toEndOf="parent">
|
|
||||||
<ImageButton
|
<ImageButton
|
||||||
android:id="@+id/add_tab_button"
|
android:id="@+id/add_tab_button"
|
||||||
android:layout_width="24dp"
|
android:layout_width="48dp"
|
||||||
android:layout_height="24dp"
|
android:layout_height="48dp"
|
||||||
android:layout_marginEnd="8dp"
|
android:background="?android:attr/selectableItemBackgroundBorderless"
|
||||||
android:background="?android:attr/selectableItemBackgroundBorderless"
|
android:contentDescription="@string/add_tab"
|
||||||
android:contentDescription="@string/add_tab"
|
android:src="@drawable/ic_new"/>
|
||||||
android:src="@drawable/ic_new"/>
|
|
||||||
|
|
||||||
<ImageButton
|
<ImageButton
|
||||||
android:id="@+id/tabs_overflow_button"
|
android:id="@+id/tabs_overflow_button"
|
||||||
android:layout_width="24dp"
|
android:layout_width="48dp"
|
||||||
android:layout_height="24dp"
|
android:layout_height="48dp"
|
||||||
android:layout_marginStart="8dp"
|
android:background="?android:attr/selectableItemBackgroundBorderless"
|
||||||
android:background="?android:attr/selectableItemBackgroundBorderless"
|
android:contentDescription="@string/open_tabs_menu"
|
||||||
android:contentDescription="@string/open_tabs_menu"
|
android:src="@drawable/ic_menu" />
|
||||||
android:src="@drawable/ic_menu" />
|
|
||||||
</LinearLayout>
|
</LinearLayout>
|
||||||
</androidx.constraintlayout.widget.ConstraintLayout>
|
</androidx.constraintlayout.widget.ConstraintLayout>
|
Loading…
Reference in New Issue