Merge pull request #6048 from element-hq/renovate/app.cash.paparazzi-2.x

chore(deps): update plugin paparazzi to v2.0.0-alpha04

features/messages/impl/src/main/kotlin/io/element/android/features/messages/impl/timeline/components/reactionsummary/ReactionSummaryView.kt

@@ -114,8 +114,10 @@ private fun ReactionSummaryViewContent(
     LaunchedEffect(pagerState.currentPage) {
         selectedReactionKey = summary.reactions[pagerState.currentPage].key
         val visibleInfo = reactionListState.layoutInfo.visibleItemsInfo
-        if (selectedReactionIndex <= visibleInfo.first().index || selectedReactionIndex >= visibleInfo.last().index) {
-            reactionListState.animateScrollToItem(selectedReactionIndex)
+        if (visibleInfo.isNotEmpty()) {
+            if (selectedReactionIndex <= visibleInfo.first().index || selectedReactionIndex >= visibleInfo.last().index) {
+                reactionListState.animateScrollToItem(selectedReactionIndex)
+            }
         }
     }
 

features/preferences/impl/src/main/kotlin/io/element/android/features/preferences/impl/developer/DeveloperSettingsStateProvider.kt

@@ -32,7 +32,12 @@ open class DeveloperSettingsStateProvider : PreviewParameterProvider<DeveloperSe
             ),
             aDeveloperSettingsState(
                 isEnterpriseBuild = true,
-                showColorPicker = true,
+                // Disable the color picker for now, Paparazzi is failing with:
+                // java.lang.IllegalArgumentException: Cannot round NaN value.
+                //  at kotlin.math.MathKt__MathJVMKt.roundToInt(MathJVM.kt:1210)
+                //  at io.mhssn.colorpicker.ext.ColorExtKt.lighten(ColorExt.kt:86)
+                //  at io.mhssn.colorpicker.pickers.ClassicColorPickerKt$ClassicColorPicker$1$1.invokeSuspend(ClassicColorPicker.kt:53)
+                showColorPicker = false,
             ),
         )
 }

gradle/libs.versions.toml

@@ -261,7 +261,7 @@ ktlint = "org.jlleitschuh.gradle.ktlint:14.0.1"
 dependencygraph = "com.savvasdalkitsis.module-dependency-graph:0.12"
 dependencycheck = "org.owasp.dependencycheck:12.2.0"
 dependencyanalysis = { id = "com.autonomousapps.dependency-analysis", version.ref = "dependencyAnalysis" }
-paparazzi = "app.cash.paparazzi:2.0.0-alpha02"
+paparazzi = "app.cash.paparazzi:2.0.0-alpha04"
 roborazzi = { id = "io.github.takahirom.roborazzi", version.ref = "roborazzi" }
 sqldelight = { id = "app.cash.sqldelight", version.ref = "sqldelight" }
 firebaseAppDistribution = { id = "com.google.firebase.appdistribution", version.ref = "firebaseAppDistribution" }

libraries/mediaviewer/impl/src/main/kotlin/io/element/android/libraries/mediaviewer/impl/local/player/ExoPlayerForPreview.kt

@@ -142,7 +142,9 @@ class ExoPlayerForPreview(
     override fun getMediaItemCount(): Int = throw NotImplementedError()
     override fun getMediaItemAt(index: Int): MediaItem = throw NotImplementedError()
     override fun getDuration(): Long = throw NotImplementedError()
-    override fun getCurrentPosition(): Long = throw NotImplementedError()
+
+    // Cannot throw, this method is invoked when recording Paparazzi screenshots.
+    override fun getCurrentPosition(): Long = 0L
     override fun getBufferedPosition(): Long = throw NotImplementedError()
     override fun getBufferedPercentage(): Int = throw NotImplementedError()
     override fun getTotalBufferedDuration(): Long = throw NotImplementedError()