From ae6ca49e4e7d007a79fed39ddce655f0bed32bc8 Mon Sep 17 00:00:00 2001
From: Alex Hart <alex@signal.org>
Date: Fri, 17 Jun 2022 11:10:29 -0300
Subject: [PATCH] Fix toolbar overlap in all media screen.

---
 .../mediaoverview/MediaOverviewActivity.java    |  2 ++
 .../main/res/layout/media_overview_activity.xml | 17 ++++++++++-------
 2 files changed, 12 insertions(+), 7 deletions(-)

diff --git a/app/src/main/java/org/thoughtcrime/securesms/mediaoverview/MediaOverviewActivity.java b/app/src/main/java/org/thoughtcrime/securesms/mediaoverview/MediaOverviewActivity.java
index 16faad727c..fb69b607a2 100644
--- a/app/src/main/java/org/thoughtcrime/securesms/mediaoverview/MediaOverviewActivity.java
+++ b/app/src/main/java/org/thoughtcrime/securesms/mediaoverview/MediaOverviewActivity.java
@@ -213,11 +213,13 @@ public final class MediaOverviewActivity extends PassphraseRequiredActivity {
   public void onEnterMultiSelect() {
     tabLayout.setEnabled(false);
     viewPager.setEnabled(false);
+    toolbar.setVisibility(View.INVISIBLE);
   }
 
   public void onExitMultiSelect() {
     tabLayout.setEnabled(true);
     viewPager.setEnabled(true);
+    toolbar.setVisibility(View.VISIBLE);
   }
 
   private void showSortOrderDialog(View v) {
diff --git a/app/src/main/res/layout/media_overview_activity.xml b/app/src/main/res/layout/media_overview_activity.xml
index fa1a93a083..212d922a88 100644
--- a/app/src/main/res/layout/media_overview_activity.xml
+++ b/app/src/main/res/layout/media_overview_activity.xml
@@ -9,6 +9,7 @@
         android:id="@+id/appBarLayout"
         android:layout_width="0dp"
         android:layout_height="wrap_content"
+        android:background="@color/signal_colorBackground"
         app:layout_constraintEnd_toEndOf="parent"
         app:layout_constraintStart_toStartOf="parent"
         app:layout_constraintTop_toTopOf="parent">
@@ -16,16 +17,18 @@
         <androidx.appcompat.widget.Toolbar
             android:id="@+id/toolbar"
             android:layout_width="match_parent"
-            android:layout_height="?android:attr/actionBarSize"
-            android:background="@color/signal_background_primary"
+            android:layout_height="@dimen/signal_m3_toolbar_height"
+            android:minHeight="@dimen/signal_m3_toolbar_height"
             app:layout_scrollFlags="scroll|enterAlways"
-            app:titleTextColor="@color/media_overview_toolbar_foreground" />
+            app:titleTextAppearance="@style/Signal.Text.TitleLarge"
+            app:titleTextColor="@color/media_overview_toolbar_foreground"
+            tools:title="Media Overview" />
 
         <org.thoughtcrime.securesms.components.ControllableTabLayout
             android:id="@+id/tab_layout"
             android:layout_width="match_parent"
             android:layout_height="wrap_content"
-            android:background="@color/signal_background_primary"
+            android:background="@color/signal_colorBackground"
             app:tabIndicatorColor="@color/signal_inverse_primary"
             app:tabIndicatorFullWidth="false"
             app:tabMode="scrollable"
@@ -38,8 +41,8 @@
     <androidx.constraintlayout.widget.ConstraintLayout
         android:id="@id/sorting"
         android:layout_width="0dp"
-        android:layout_marginTop="8dp"
         android:layout_height="wrap_content"
+        android:layout_marginTop="8dp"
         app:layout_constraintEnd_toEndOf="parent"
         app:layout_constraintStart_toStartOf="parent"
         app:layout_constraintTop_toBottomOf="@+id/appBarLayout">
@@ -53,8 +56,8 @@
             android:layout_marginBottom="12dp"
             android:gravity="center_vertical"
             android:textAppearance="@style/TextAppearance.Signal.Body1.Bold"
-            android:textStyle="bold"
             android:textColor="@color/signal_inverse_primary"
+            android:textStyle="bold"
             app:layout_constraintStart_toStartOf="@+id/sorting"
             app:layout_constraintTop_toTopOf="parent"
             tools:text="@string/MediaOverviewActivity_Storage_used" />
@@ -77,8 +80,8 @@
             android:layout_width="48dp"
             android:layout_height="48dp"
             android:layout_gravity="center"
-            android:gravity="center"
             android:layout_marginEnd="@dimen/media_overview_toggle_gutter"
+            android:gravity="center"
             app:layout_constraintBottom_toBottomOf="@+id/sort_order_arrow"
             app:layout_constraintEnd_toEndOf="@+id/sorting"
             app:layout_constraintTop_toTopOf="@+id/sort_order_arrow">