From cd22a64ebd3bd8ee4cfa4a07a2234befb610ec6a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Pandula=20P=C3=A9ter?= <pandulapeter@gmail.com>
Date: Sun, 8 Sep 2024 11:53:08 +0300
Subject: [PATCH] Apply the default theme to activities when needed.

---
 internal-core/src/main/AndroidManifest.xml           |  4 +++-
 .../pandulapeter/beagle/core/BeagleImplementation.kt |  8 ++++++++
 .../beagle/core/manager/BugReportManager.kt          |  7 ++++++-
 .../beagle/core/util/extension/Context.kt            |  4 +++-
 .../beagle/core/view/bugReport/BugReportActivity.kt  | 12 ++++++++++--
 .../src/main/res/layout/beagle_view_media.xml        |  4 +---
 internal-core/src/main/res/values-night/themes.xml   |  6 ++++++
 internal-core/src/main/res/values/themes.xml         |  7 ++++++-
 .../logCrash/implementation/ExceptionHandler.kt      |  3 ++-
 9 files changed, 45 insertions(+), 10 deletions(-)
 create mode 100644 internal-core/src/main/res/values-night/themes.xml

diff --git a/internal-core/src/main/AndroidManifest.xml b/internal-core/src/main/AndroidManifest.xml
index 4ede77e3..7f5d7ccd 100644
--- a/internal-core/src/main/AndroidManifest.xml
+++ b/internal-core/src/main/AndroidManifest.xml
@@ -1,6 +1,6 @@
 <manifest xmlns:android="http://schemas.android.com/apk/res/android">
 
-    <uses-permission android:name="android.permission.POST_NOTIFICATIONS"/>
+    <uses-permission android:name="android.permission.POST_NOTIFICATIONS" />
     <uses-permission android:name="android.permission.FOREGROUND_SERVICE" />
     <uses-permission android:name="android.permission.FOREGROUND_SERVICE_MEDIA_PROJECTION" />
     <uses-permission android:name="android.permission.VIBRATE" />
@@ -11,12 +11,14 @@
             android:name=".view.gallery.GalleryActivity"
             android:exported="false"
             android:launchMode="singleTop"
+            android:theme="@style/BeagleDefaultTheme"
             android:windowSoftInputMode="adjustResize" />
 
         <activity
             android:name=".view.bugReport.BugReportActivity"
             android:exported="false"
             android:launchMode="singleTop"
+            android:theme="@style/BeagleDefaultTheme"
             android:windowSoftInputMode="adjustResize" />
 
         <service
diff --git a/internal-core/src/main/java/com/pandulapeter/beagle/core/BeagleImplementation.kt b/internal-core/src/main/java/com/pandulapeter/beagle/core/BeagleImplementation.kt
index 63aa93cf..3906bdda 100644
--- a/internal-core/src/main/java/com/pandulapeter/beagle/core/BeagleImplementation.kt
+++ b/internal-core/src/main/java/com/pandulapeter/beagle/core/BeagleImplementation.kt
@@ -96,6 +96,14 @@ class BeagleImplementation(val uiManager: UiManagerContract) : BeagleContract {
         BeagleCore.implementation = this
     }
 
+    internal fun invalidateThemeOverride() {
+        if (appearance.themeResourceId != null) {
+            appearance = appearance.copy(
+                themeResourceId = null
+            )
+        }
+    }
+
     override fun initialize(
         application: Application,
         appearance: Appearance,
diff --git a/internal-core/src/main/java/com/pandulapeter/beagle/core/manager/BugReportManager.kt b/internal-core/src/main/java/com/pandulapeter/beagle/core/manager/BugReportManager.kt
index 78f4761c..9a6de147 100644
--- a/internal-core/src/main/java/com/pandulapeter/beagle/core/manager/BugReportManager.kt
+++ b/internal-core/src/main/java/com/pandulapeter/beagle/core/manager/BugReportManager.kt
@@ -30,7 +30,12 @@ internal class BugReportManager {
 
     fun openBugReportingScreen() {
         BeagleCore.implementation.currentActivity?.run {
-            startActivity(BugReportActivity.newIntent(this))
+            startActivity(
+                BugReportActivity.newIntent(
+                    context = this,
+                    themeResourceId = BeagleCore.implementation.appearance.themeResourceId
+                )
+            )
         }
     }
 
diff --git a/internal-core/src/main/java/com/pandulapeter/beagle/core/util/extension/Context.kt b/internal-core/src/main/java/com/pandulapeter/beagle/core/util/extension/Context.kt
index 2cacad52..61d1fbf9 100644
--- a/internal-core/src/main/java/com/pandulapeter/beagle/core/util/extension/Context.kt
+++ b/internal-core/src/main/java/com/pandulapeter/beagle/core/util/extension/Context.kt
@@ -50,7 +50,9 @@ private fun Context.toMaterialContext(): Context {
     return if (isMaterialTheme) {
         this
     } else {
-        ContextThemeWrapper(this, R.style.BeagleDefaultTheme)
+        ContextThemeWrapper(this, R.style.BeagleDefaultTheme).also {
+            BeagleCore.implementation.invalidateThemeOverride()
+        }
     }
 }
 
diff --git a/internal-core/src/main/java/com/pandulapeter/beagle/core/view/bugReport/BugReportActivity.kt b/internal-core/src/main/java/com/pandulapeter/beagle/core/view/bugReport/BugReportActivity.kt
index 127fc7a1..3eae5c38 100644
--- a/internal-core/src/main/java/com/pandulapeter/beagle/core/view/bugReport/BugReportActivity.kt
+++ b/internal-core/src/main/java/com/pandulapeter/beagle/core/view/bugReport/BugReportActivity.kt
@@ -56,6 +56,9 @@ class BugReportActivity : AppCompatActivity() {
             }
         }
     }
+    private val themeResourceId by lazy {
+        intent.getIntExtra(THEME_RESOURCE_ID, -1)
+    }
 
     @Suppress("UNCHECKED_CAST")
     private val viewModel by lazy {
@@ -79,7 +82,9 @@ class BugReportActivity : AppCompatActivity() {
     }
 
     override fun onCreate(savedInstanceState: Bundle?) {
-        BeagleCore.implementation.appearance.themeResourceId?.let { setTheme(it) }
+        if (themeResourceId != -1) {
+            setTheme(themeResourceId)
+        }
         super.onCreate(savedInstanceState)
         binding = BeagleActivityBugReportBinding.inflate(layoutInflater)
         setContentView(binding.root)
@@ -219,13 +224,16 @@ class BugReportActivity : AppCompatActivity() {
     companion object {
         private const val CRASH_LOG_ENTRY_TO_SHOW = "crashLogEntry"
         private const val RESTORE_MODEL = "restoreModel"
+        private const val THEME_RESOURCE_ID = "themeResourceId"
 
         fun newIntent(
             context: Context,
             crashLogEntryToShowJson: String = "",
-            restoreModelJson: String = ""
+            restoreModelJson: String = "",
+            themeResourceId: Int? = null,
         ) = Intent(context, BugReportActivity::class.java)
             .putExtra(CRASH_LOG_ENTRY_TO_SHOW, crashLogEntryToShowJson)
             .putExtra(RESTORE_MODEL, restoreModelJson)
+            .putExtra(THEME_RESOURCE_ID, themeResourceId)
     }
 }
\ No newline at end of file
diff --git a/internal-core/src/main/res/layout/beagle_view_media.xml b/internal-core/src/main/res/layout/beagle_view_media.xml
index 8f523580..967a83df 100644
--- a/internal-core/src/main/res/layout/beagle_view_media.xml
+++ b/internal-core/src/main/res/layout/beagle_view_media.xml
@@ -16,9 +16,7 @@
         android:id="@+id/beagle_check_box"
         android:layout_width="wrap_content"
         android:layout_height="wrap_content"
-        android:layout_gravity="end"
-        android:layout_marginTop="-8dp"
-        android:layout_marginEnd="-12dp" />
+        android:layout_gravity="end" />
 
     <com.google.android.material.textview.MaterialTextView
         android:id="@+id/beagle_text_view"
diff --git a/internal-core/src/main/res/values-night/themes.xml b/internal-core/src/main/res/values-night/themes.xml
new file mode 100644
index 00000000..d2e75d38
--- /dev/null
+++ b/internal-core/src/main/res/values-night/themes.xml
@@ -0,0 +1,6 @@
+<?xml version="1.0" encoding="utf-8"?>
+<resources>
+
+    <style name="BeagleDefaultTheme" parent="BeagleDefaultThemeBase" />
+
+</resources>
\ No newline at end of file
diff --git a/internal-core/src/main/res/values/themes.xml b/internal-core/src/main/res/values/themes.xml
index c1df4726..1fcc3acb 100644
--- a/internal-core/src/main/res/values/themes.xml
+++ b/internal-core/src/main/res/values/themes.xml
@@ -1,10 +1,15 @@
 <?xml version="1.0" encoding="utf-8"?>
 <resources>
 
-    <style name="BeagleDefaultTheme" parent="Theme.Material3.DayNight.NoActionBar">
+    <style name="BeagleDefaultThemeBase" parent="Theme.Material3.DayNight.NoActionBar">
         <item name="colorPrimary">@color/beagle_accent</item>
         <item name="colorOnPrimary">@color/beagle_on_accent</item>
         <item name="colorControlActivated">@color/beagle_accent</item>
+        <item name="android:statusBarColor">@android:color/transparent</item>
+    </style>
+
+    <style name="BeagleDefaultTheme" parent="BeagleDefaultThemeBase">
+        <item name="android:windowLightStatusBar">true</item>
     </style>
 
 </resources>
\ No newline at end of file
diff --git a/log-crash/src/main/java/com/pandulapeter/beagle/logCrash/implementation/ExceptionHandler.kt b/log-crash/src/main/java/com/pandulapeter/beagle/logCrash/implementation/ExceptionHandler.kt
index 7028f5c5..8f46c360 100644
--- a/log-crash/src/main/java/com/pandulapeter/beagle/logCrash/implementation/ExceptionHandler.kt
+++ b/log-crash/src/main/java/com/pandulapeter/beagle/logCrash/implementation/ExceptionHandler.kt
@@ -39,7 +39,8 @@ internal class ExceptionHandler private constructor(
                             networkLogs = networkLogsToRestore,
                             lifecycleLogs = lifecycleLogsToRestore
                         )
-                    )
+                    ),
+                    themeResourceId = BeagleCore.implementation.appearance.themeResourceId
                 ).setFlags(Intent.FLAG_ACTIVITY_NEW_TASK)
             )
             tryToCallDefaultExceptionHandler(thread, exception)