Merge pull request #22 from crazo7924/fix-auto-brightness

NewPlayerUIState: correctly handle auto brightness

Author: Stypox

new-player/src/main/java/net/newpipe/newplayer/ui/NewPlayerUI.kt

@@ -149,7 +149,7 @@ fun NewPlayerUI(
             val defaultBrightness = getDefaultBrightness(activity)
 
             setScreenBrightness(
-                uiState.brightness ?: defaultBrightness, activity
+                if (uiState.brightness < 0) defaultBrightness else uiState.brightness, activity
             )
         }
 

new-player/src/main/java/net/newpipe/newplayer/ui/common/utils.kt

@@ -72,7 +72,7 @@ internal fun LockScreenOrientation(orientation: Int) {
 internal fun getDefaultBrightness(activity: Activity): Float {
     val window = activity.window
     val layout = window.attributes as WindowManager.LayoutParams
-    return if (layout.screenBrightness < 0) 0.5f else layout.screenBrightness
+    return if (layout.screenBrightness < 0) -1f else layout.screenBrightness
 }
 
 @SuppressLint("NewApi")

new-player/src/main/java/net/newpipe/newplayer/uiModel/NewPlayerUIState.kt

@@ -103,9 +103,9 @@ data class NewPlayerUIState(
     val soundVolume: Float,
 
     /**
-     * The brightness volume. Might be null if the system is in control of the brightness.
+     * The brightness volume. Might be negative if the system is in control of the brightness.
      */
-    val brightness: Float?,
+    val brightness: Float,
 
     /**
      * This is used to restore several values when switching back from a fullscreen view to an
@@ -214,7 +214,7 @@ data class NewPlayerUIState(
             playbackPositionInMs = 0,
             fastSeekSeconds = 0,
             soundVolume = 0f,
-            brightness = null,
+            brightness = -1f,
             embeddedUiConfig = null,
             playList = emptyList(),
             chapters = emptyList(),