Remove unused lambda parameter. The link is opened by the LocalUriHandler now.

This commit is contained in:
Benoit Marty
2023-08-30 10:21:27 +02:00
parent 1c50786299
commit 1b87565870
3 changed files with 0 additions and 16 deletions
@@ -37,7 +37,6 @@ import io.element.android.libraries.ui.strings.CommonStrings
fun AnalyticsPreferencesView( fun AnalyticsPreferencesView(
state: AnalyticsPreferencesState, state: AnalyticsPreferencesState,
modifier: Modifier = Modifier, modifier: Modifier = Modifier,
onOpenAnalyticsPolicy: (url: String) -> Unit,
) { ) {
fun onEnabledChanged(isEnabled: Boolean) { fun onEnabledChanged(isEnabled: Boolean) {
state.eventSink(AnalyticsOptInEvents.EnableAnalytics(isEnabled = isEnabled)) state.eventSink(AnalyticsOptInEvents.EnableAnalytics(isEnabled = isEnabled))
@@ -90,6 +89,5 @@ internal fun AnalyticsPreferencesViewDarkPreview(@PreviewParameter(AnalyticsPref
private fun ContentToPreview(state: AnalyticsPreferencesState) { private fun ContentToPreview(state: AnalyticsPreferencesState) {
AnalyticsPreferencesView( AnalyticsPreferencesView(
state = state, state = state,
onOpenAnalyticsPolicy = {},
) )
} }
@@ -16,19 +16,15 @@
package io.element.android.features.preferences.impl.analytics package io.element.android.features.preferences.impl.analytics
import android.app.Activity
import androidx.compose.runtime.Composable import androidx.compose.runtime.Composable
import androidx.compose.ui.Modifier import androidx.compose.ui.Modifier
import androidx.compose.ui.platform.LocalContext
import com.bumble.appyx.core.modality.BuildContext import com.bumble.appyx.core.modality.BuildContext
import com.bumble.appyx.core.node.Node import com.bumble.appyx.core.node.Node
import com.bumble.appyx.core.plugin.Plugin import com.bumble.appyx.core.plugin.Plugin
import dagger.assisted.Assisted import dagger.assisted.Assisted
import dagger.assisted.AssistedInject import dagger.assisted.AssistedInject
import io.element.android.anvilannotations.ContributesNode import io.element.android.anvilannotations.ContributesNode
import io.element.android.libraries.androidutils.browser.openUrlInChromeCustomTab
import io.element.android.libraries.di.SessionScope import io.element.android.libraries.di.SessionScope
import io.element.android.libraries.theme.ElementTheme
@ContributesNode(SessionScope::class) @ContributesNode(SessionScope::class)
class AnalyticsSettingsNode @AssistedInject constructor( class AnalyticsSettingsNode @AssistedInject constructor(
@@ -37,19 +33,12 @@ class AnalyticsSettingsNode @AssistedInject constructor(
private val presenter: AnalyticsSettingsPresenter, private val presenter: AnalyticsSettingsPresenter,
) : Node(buildContext, plugins = plugins) { ) : Node(buildContext, plugins = plugins) {
private fun onOpenAnalyticsPolicy(activity: Activity, darkTheme: Boolean, url: String) {
activity.openUrlInChromeCustomTab(null, darkTheme, url)
}
@Composable @Composable
override fun View(modifier: Modifier) { override fun View(modifier: Modifier) {
val activity = LocalContext.current as Activity
val isDark = ElementTheme.colors.isLight.not()
val state = presenter.present() val state = presenter.present()
AnalyticsSettingsView( AnalyticsSettingsView(
state = state, state = state,
onBackPressed = ::navigateUp, onBackPressed = ::navigateUp,
onOpenAnalyticsPolicy = { onOpenAnalyticsPolicy(activity, darkTheme = isDark, it) },
modifier = modifier modifier = modifier
) )
} }
@@ -31,7 +31,6 @@ import io.element.android.libraries.ui.strings.CommonStrings
fun AnalyticsSettingsView( fun AnalyticsSettingsView(
state: AnalyticsSettingsState, state: AnalyticsSettingsState,
onBackPressed: () -> Unit, onBackPressed: () -> Unit,
onOpenAnalyticsPolicy: (url: String) -> Unit,
modifier: Modifier = Modifier, modifier: Modifier = Modifier,
) { ) {
PreferenceView( PreferenceView(
@@ -41,7 +40,6 @@ fun AnalyticsSettingsView(
) { ) {
AnalyticsPreferencesView( AnalyticsPreferencesView(
state = state.analyticsState, state = state.analyticsState,
onOpenAnalyticsPolicy = onOpenAnalyticsPolicy,
) )
} }
} }
@@ -61,6 +59,5 @@ private fun ContentToPreview(state: AnalyticsSettingsState) {
AnalyticsSettingsView( AnalyticsSettingsView(
state = state, state = state,
onBackPressed = {}, onBackPressed = {},
onOpenAnalyticsPolicy = {},
) )
} }