Browse Source

Fix detekt issues.

feature/bma/flipper
Benoit Marty 2 years ago
parent
commit
42075176a1
  1. 2
      libraries/matrix/src/main/java/io/element/android/x/matrix/core/EventId.kt
  2. 2
      libraries/matrix/src/main/java/io/element/android/x/matrix/media/MediaKeyer.kt
  3. 2
      libraries/matrix/src/main/java/io/element/android/x/matrix/util/Error.kt
  4. 2
      libraries/textcomposer/src/main/java/io/element/android/x/textcomposer/RichTextComposerLayout.kt
  5. 111
      libraries/textcomposer/src/main/java/io/element/android/x/textcomposer/TextComposer.kt

2
libraries/matrix/src/main/java/io/element/android/x/matrix/core/EventId.kt

@ -3,4 +3,4 @@ package io.element.android.x.matrix.core @@ -3,4 +3,4 @@ package io.element.android.x.matrix.core
import java.io.Serializable
@JvmInline
value class EventId(val value: String) : Serializable
value class EventId(val value: String) : Serializable

2
libraries/matrix/src/main/java/io/element/android/x/matrix/media/MediaKeyer.kt

@ -7,4 +7,4 @@ internal class MediaKeyer : Keyer<MediaResolver.Meta> { @@ -7,4 +7,4 @@ internal class MediaKeyer : Keyer<MediaResolver.Meta> {
override fun key(data: MediaResolver.Meta, options: Options): String? {
return "${data.source.url()}_${data.kind}"
}
}
}

2
libraries/matrix/src/main/java/io/element/android/x/matrix/util/Error.kt

@ -12,4 +12,4 @@ fun logError(throwable: Throwable) { @@ -12,4 +12,4 @@ fun logError(throwable: Throwable) {
Timber.e("Error", throwable)
}
}
}
}

2
libraries/textcomposer/src/main/java/io/element/android/x/textcomposer/RichTextComposerLayout.kt

@ -485,7 +485,7 @@ class RichTextComposerLayout @JvmOverloads constructor( @@ -485,7 +485,7 @@ class RichTextComposerLayout @JvmOverloads constructor(
} else {
views.composerModeGroup.isGone = true
(mode as? MessageComposerMode.Normal)?.content?.let { text ->
// TODO: un-comment once we update to a version of the lib > 0.8.0
// TODO un-comment once we update to a version of the lib > 0.8.0
/*
if (isTextFormattingEnabled) {
replaceFormattedContent(text)

111
libraries/textcomposer/src/main/java/io/element/android/x/textcomposer/TextComposer.kt

@ -22,80 +22,81 @@ import io.element.android.x.element.resources.R as ElementR @@ -22,80 +22,81 @@ import io.element.android.x.element.resources.R as ElementR
@Composable
fun TextComposer(
onSendMessage: (String) -> Unit,
modifier: Modifier = Modifier,
fullscreen: Boolean,
onFullscreenToggle: () -> Unit,
composerText: String?,
composerMode: MessageComposerMode,
onCloseSpecialMode: () -> Unit,
onComposerTextChange: (CharSequence) -> Unit,
composerCanSendMessage: Boolean,
composerText: String?,
modifier: Modifier = Modifier,
onSendMessage: (String) -> Unit = {},
onFullscreenToggle: () -> Unit = {},
onCloseSpecialMode: () -> Unit = {},
onComposerTextChange: (CharSequence) -> Unit = {},
) {
if (LocalInspectionMode.current) {
FakeComposer(modifier)
return
}
} else {
val isInDarkMode = isSystemInDarkTheme()
AndroidView(
modifier = modifier,
factory = { context ->
RichTextComposerLayout(context).apply {
// Sets up listeners for View -> Compose communication
this.callback = object : Callback {
override fun onRichContentSelected(contentUri: Uri): Boolean {
return false
}
val isInDarkMode = isSystemInDarkTheme()
AndroidView(
modifier = modifier,
factory = { context ->
RichTextComposerLayout(context).apply {
// Sets up listeners for View -> Compose communication
this.callback = object : Callback {
override fun onRichContentSelected(contentUri: Uri): Boolean {
return false
}
override fun onTextChanged(text: CharSequence) {
onComposerTextChange(text)
}
override fun onTextChanged(text: CharSequence) {
onComposerTextChange(text)
}
override fun onCloseRelatedMessage() {
onCloseSpecialMode()
}
override fun onCloseRelatedMessage() {
onCloseSpecialMode()
}
override fun onSendMessage(text: CharSequence) {
// text contains markdown.
onSendMessage(text.toString())
}
override fun onSendMessage(text: CharSequence) {
// text contains markdown.
onSendMessage(text.toString())
}
override fun onAddAttachment() {
}
override fun onAddAttachment() {
}
override fun onExpandOrCompactChange() {
}
override fun onExpandOrCompactChange() {
override fun onFullScreenModeChanged() {
onFullscreenToggle()
}
}
override fun onFullScreenModeChanged() {
onFullscreenToggle()
setFullScreen(fullscreen, animated = false, manageKeyboard = true)
(this as MessageComposerView).apply {
setup(isInDarkMode, composerMode)
}
}
setFullScreen(fullscreen, animated = false, manageKeyboard = true)
(this as MessageComposerView).apply {
setup(isInDarkMode, composerMode)
}
}
},
update = { view ->
// View's been inflated or state read in this block has been updated
// Add logic here if necessary
},
update = { view ->
// View's been inflated or state read in this block has been updated
// Add logic here if necessary
// As selectedItem is read here, AndroidView will recompose
// whenever the state changes
// Example of Compose -> View communication
val messageComposerView = (view as MessageComposerView)
view.setFullScreen(fullscreen, animated = false, manageKeyboard = false)
messageComposerView.renderComposerMode(composerMode)
messageComposerView.sendButton.isInvisible = !composerCanSendMessage
messageComposerView.setTextIfDifferent(composerText ?: "")
messageComposerView.editText.requestFocus()
}
)
// As selectedItem is read here, AndroidView will recompose
// whenever the state changes
// Example of Compose -> View communication
val messageComposerView = (view as MessageComposerView)
view.setFullScreen(fullscreen, animated = false, manageKeyboard = false)
messageComposerView.renderComposerMode(composerMode)
messageComposerView.sendButton.isInvisible = !composerCanSendMessage
messageComposerView.setTextIfDifferent(composerText ?: "")
messageComposerView.editText.requestFocus()
}
)
}
}
@Composable
private fun FakeComposer(modifier: Modifier) {
private fun FakeComposer(
modifier: Modifier = Modifier,
) {
// AndroidView is not Available in this mode, just render a Text
Box(
modifier = modifier

Loading…
Cancel
Save