Address PR comments

Author: JoseAlcerreca

Jetchat/app/src/main/java/com/example/compose/jetchat/NavActivity.kt

@@ -24,12 +24,12 @@ import androidx.compose.material.rememberScaffoldState
 import androidx.compose.runtime.CompositionLocalProvider
 import androidx.compose.runtime.LaunchedEffect
 import androidx.compose.runtime.collectAsState
+import androidx.compose.runtime.getValue
 import androidx.compose.runtime.rememberCoroutineScope
 import androidx.compose.ui.viewinterop.AndroidViewBinding
 import androidx.core.os.bundleOf
 import androidx.core.view.WindowCompat
 import androidx.navigation.NavController
-import androidx.navigation.findNavController
 import androidx.navigation.fragment.NavHostFragment
 import com.example.compose.jetchat.components.JetchatScaffold
 import com.example.compose.jetchat.conversation.BackPressHandler
@@ -60,8 +60,8 @@ class NavActivity : AppCompatActivity() {
                 ) {
                     val scaffoldState = rememberScaffoldState()
 
-                    val openDrawerEvent = viewModel.drawerShouldBeOpened.collectAsState()
-                    if (openDrawerEvent.value) {
+                    val drawerOpen by viewModel.drawerShouldBeOpened.collectAsState()
+                    if (drawerOpen) {
                         // Open drawer and reset state in VM.
                         LaunchedEffect(Unit) {
                             scaffoldState.drawerState.open()

Jetchat/app/src/main/java/com/example/compose/jetchat/conversation/Conversation.kt

@@ -102,6 +102,7 @@ fun ConversationContent(
 
     val scrollState = rememberLazyListState()
     val scope = rememberCoroutineScope()
+
     Surface(modifier = modifier) {
         Box(modifier = Modifier.fillMaxSize()) {
             Column(Modifier.fillMaxSize()) {
@@ -288,8 +289,6 @@ fun Message(
     isFirstMessageByAuthor: Boolean,
     isLastMessageByAuthor: Boolean
 ) {
-    val painter = painterResource(id = msg.authorImage)
-
     val borderColor = if (isUserMe) {
         MaterialTheme.colors.primary
     } else {
@@ -309,7 +308,7 @@ fun Message(
                     .border(3.dp, MaterialTheme.colors.surface, CircleShape)
                     .clip(CircleShape)
                     .align(Alignment.Top),
-                painter = painter,
+                painter = painterResource(id = msg.authorImage),
                 contentScale = ContentScale.Crop,
                 contentDescription = null,
             )