From 655d0e719e18e86b8bb1345a6511ca5f3a35d629 Mon Sep 17 00:00:00 2001
From: Uwe Trottmann <uwe+git@uwetrottmann.com>
Date: Thu, 16 Nov 2023 13:08:15 +0100
Subject: [PATCH] DRAFT Use navigation rail on 590dp+ wide screens.

This won't work without significant adjustments to the layout, e.g. the
app bar is supposed to sit on top. But that will break hiding on scroll,
tabs, ...
---
 .../seriesguide/ui/BaseTopActivity.kt         |  4 +-
 .../main/res/layout-w600dp/activity_shows.xml | 48 +++++++++++++++++++
 app/src/main/res/values/styles.xml            |  8 ++++
 app/src/main/res/values/themes.xml            |  5 ++
 4 files changed, 63 insertions(+), 2 deletions(-)
 create mode 100644 app/src/main/res/layout-w600dp/activity_shows.xml

diff --git a/app/src/main/java/com/battlelancer/seriesguide/ui/BaseTopActivity.kt b/app/src/main/java/com/battlelancer/seriesguide/ui/BaseTopActivity.kt
index 6d5e737a5b..fe9f828cc1 100644
--- a/app/src/main/java/com/battlelancer/seriesguide/ui/BaseTopActivity.kt
+++ b/app/src/main/java/com/battlelancer/seriesguide/ui/BaseTopActivity.kt
@@ -30,7 +30,7 @@ import com.battlelancer.seriesguide.ui.ShowsActivity
 import com.battlelancer.seriesguide.util.SupportTheDev
 import com.battlelancer.seriesguide.util.SupportTheDev.buildSnackbar
 import com.battlelancer.seriesguide.util.Utils
-import com.google.android.material.bottomnavigation.BottomNavigationView
+import com.google.android.material.navigation.NavigationBarView
 import com.google.android.material.snackbar.Snackbar
 import timber.log.Timber
 
@@ -72,7 +72,7 @@ abstract class BaseTopActivity : BaseMessageActivity() {
     }
 
     fun setupBottomNavigation(@IdRes selectedItemId: Int) {
-        val bottomNav = findViewById<BottomNavigationView>(R.id.bottomNavigation)
+        val bottomNav = findViewById<NavigationBarView>(R.id.bottomNavigation)
         bottomNav.selectedItemId = selectedItemId
         // Disable hideous bold font for active item.
         bottomNav.setItemTextAppearanceActiveBoldEnabled(false)
diff --git a/app/src/main/res/layout-w600dp/activity_shows.xml b/app/src/main/res/layout-w600dp/activity_shows.xml
new file mode 100644
index 0000000000..68b8a9983e
--- /dev/null
+++ b/app/src/main/res/layout-w600dp/activity_shows.xml
@@ -0,0 +1,48 @@
+<?xml version="1.0" encoding="utf-8"?>
+
+<!-- Copyright 2023 Uwe Trottmann -->
+<!-- SPDX-License-Identifier: Apache-2.0 -->
+
+<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
+    xmlns:app="http://schemas.android.com/apk/res-auto"
+    android:id="@+id/rootLayoutShows"
+    android:layout_width="match_parent"
+    android:layout_height="match_parent"
+    android:orientation="horizontal">
+
+    <com.google.android.material.navigationrail.NavigationRailView
+        android:id="@+id/bottomNavigation"
+        android:layout_width="wrap_content"
+        android:layout_height="match_parent"
+        app:labelVisibilityMode="labeled"
+        app:menu="@menu/bottom_navigation_menu" />
+
+    <androidx.coordinatorlayout.widget.CoordinatorLayout
+        android:id="@+id/coordinatorLayoutShows"
+        android:layout_width="0dp"
+        android:layout_height="match_parent"
+        android:layout_weight="1">
+
+        <include layout="@layout/top_app_bar_scrolling_tabs_progress" />
+
+        <androidx.viewpager2.widget.ViewPager2
+            android:id="@+id/viewPagerTabs"
+            android:layout_width="match_parent"
+            android:layout_height="match_parent"
+            app:layout_behavior="@string/appbar_scrolling_view_behavior" />
+
+        <com.google.android.material.floatingactionbutton.FloatingActionButton
+            android:id="@+id/buttonShowsAdd"
+            android:layout_width="wrap_content"
+            android:layout_height="wrap_content"
+            android:layout_gravity="bottom|end"
+            android:layout_marginStart="0dp"
+            android:layout_marginTop="0dp"
+            android:layout_marginEnd="@dimen/fab_margin"
+            android:layout_marginBottom="@dimen/fab_margin"
+            android:contentDescription="@string/action_shows_add"
+            app:srcCompat="@drawable/ic_add_white_24dp" />
+
+    </androidx.coordinatorlayout.widget.CoordinatorLayout>
+
+</LinearLayout>
diff --git a/app/src/main/res/values/styles.xml b/app/src/main/res/values/styles.xml
index 2774fe1cb1..6fffbc4b1e 100644
--- a/app/src/main/res/values/styles.xml
+++ b/app/src/main/res/values/styles.xml
@@ -1,4 +1,7 @@
 <?xml version="1.0" encoding="utf-8"?>
+<!-- Copyright 2023 Uwe Trottmann -->
+<!-- SPDX-License-Identifier: Apache-2.0 -->
+
 <resources xmlns:android="http://schemas.android.com/apk/res/android" xmlns:tools="http://schemas.android.com/tools">
 
     <!-- Text styles -->
@@ -246,6 +249,11 @@
         <item name="materialThemeOverlay">@style/ThemeOverlay.SeriesGuide.BottomNavigationView</item>
     </style>
 
+    <!-- NavigationRailView -->
+    <style name="Widget.SeriesGuide.NavigationRailView" parent="Widget.Material3.NavigationRailView">
+        <item name="materialThemeOverlay">@style/ThemeOverlay.SeriesGuide.BottomNavigationView</item>
+    </style>
+
     <style name="ThemeOverlay.SeriesGuide.BottomNavigationView" parent="">
         <!-- Used for Widget.Material3.BottomNavigationView.ActiveIndicator color -->
         <item name="colorSecondaryContainer" tools:ignore="PrivateResource">?attr/colorPrimaryContainer</item>
diff --git a/app/src/main/res/values/themes.xml b/app/src/main/res/values/themes.xml
index 4e02b6bea3..ef11367d03 100644
--- a/app/src/main/res/values/themes.xml
+++ b/app/src/main/res/values/themes.xml
@@ -1,4 +1,7 @@
 <?xml version="1.0" encoding="utf-8"?>
+<!-- Copyright 2023 Uwe Trottmann -->
+<!-- SPDX-License-Identifier: Apache-2.0 -->
+
 <resources xmlns:android="http://schemas.android.com/apk/res/android">
 
     <!-- ####################################### -->
@@ -71,6 +74,7 @@
         <item name="preferenceTheme">@style/ThemeOverlay.SeriesGuide.Preference</item>
 
         <item name="bottomNavigationStyle">@style/Widget.SeriesGuide.BottomNavigationView</item>
+        <item name="navigationRailStyle">@style/Widget.SeriesGuide.NavigationRailView</item>
 
         <item name="textAppearanceButton">@style/TextAppearance.SeriesGuide.Button</item>
 
@@ -143,6 +147,7 @@
         <item name="preferenceTheme">@style/ThemeOverlay.SeriesGuide.Preference</item>
 
         <item name="bottomNavigationStyle">@style/Widget.SeriesGuide.BottomNavigationView</item>
+        <item name="navigationRailStyle">@style/Widget.SeriesGuide.NavigationRailView</item>
 
         <item name="textAppearanceButton">@style/TextAppearance.SeriesGuide.Button</item>