Fix style issues

Author: OwenScott-GDS

app/src/main/java/com/example/reply/MainActivity.kt

@@ -51,28 +51,34 @@ class MainActivity : ComponentActivity() {
 @Composable
 fun ReplyAppCompactPreview() {
     ReplyTheme {
-        ReplyApp(
-            windowSize = WindowWidthSizeClass.Compact,
-        )
+        Surface {
+            ReplyApp(
+                windowSize = WindowWidthSizeClass.Compact,
+            )
+        }
     }
 }
 
 @Preview(showBackground = true, widthDp = 700)
 @Composable
 fun ReplyAppMediumPreview() {
     ReplyTheme {
-        ReplyApp(
-            windowSize = WindowWidthSizeClass.Medium,
-        )
+        Surface {
+            ReplyApp(
+                windowSize = WindowWidthSizeClass.Medium,
+            )
+        }
     }
 }
 
 @Preview(showBackground = true, widthDp = 1000)
 @Composable
 fun ReplyAppExpandedPreview() {
     ReplyTheme {
-        ReplyApp(
-            windowSize = WindowWidthSizeClass.Expanded,
-        )
+        Surface {
+            ReplyApp(
+                windowSize = WindowWidthSizeClass.Expanded,
+            )
+        }
     }
-}
+}

app/src/main/java/com/example/reply/data/Account.kt

@@ -16,6 +16,7 @@
 package com.example.reply.data
 
 import androidx.annotation.DrawableRes
+import androidx.annotation.StringRes
 
 /**
  * A class which represents an account
@@ -24,14 +25,14 @@ data class Account(
     /** Unique ID of a user **/
     val id: Long,
     /** User's first name **/
-    val firstName: Int,
+    @StringRes val firstName: Int,
     /** User's last name **/
-    val lastName: Int,
+    @StringRes val lastName: Int,
     /** User's email address **/
-    val email: Int,
+    @StringRes val email: Int,
     /** User's avatar image resource id **/
     @DrawableRes val avatar: Int
 ) {
     /** User's full name **/
     val fullName: String = "$firstName $lastName"
-}
+}

app/src/main/java/com/example/reply/data/Email.kt

@@ -15,6 +15,8 @@
  */
 package com.example.reply.data
 
+import androidx.annotation.StringRes
+
 /**
  * A simple data class to represent an Email
  */
@@ -26,9 +28,9 @@ data class Email(
     /** Recipient(s) of the email **/
     val recipients: List<Account> = emptyList(),
     /** Title of the email **/
-    val subject: Int = -1,
+    @StringRes val subject: Int = -1,
     /** Content of the email **/
-    val body: Int = -1,
+    @StringRes val body: Int = -1,
     /** Which mailbox it is in **/
     var mailbox: MailboxType = MailboxType.Inbox,
     /**

app/src/main/java/com/example/reply/data/MailboxType.kt

@@ -20,4 +20,4 @@ package com.example.reply.data
  */
 enum class MailboxType {
     Inbox, Drafts, Sent, Spam
-}
+}

app/src/main/java/com/example/reply/data/local/LocalAccountsDataProvider.kt

@@ -114,4 +114,4 @@ object LocalAccountsDataProvider {
         return allUserContactAccounts.firstOrNull { it.id == accountId }
             ?: allUserContactAccounts.first()
     }
-}
+}

app/src/main/java/com/example/reply/ui/ReplyApp.kt

@@ -71,4 +71,4 @@ fun ReplyApp(
         },
         modifier = modifier
     )
-}
+}

app/src/main/java/com/example/reply/ui/ReplyDetailsScreen.kt

@@ -54,8 +54,8 @@ import com.example.reply.data.MailboxType
 @Composable
 fun ReplyDetailsScreen(
     replyUiState: ReplyUiState,
+    onBackPressed: () -> Unit,
     modifier: Modifier = Modifier,
-    onBackPressed: () -> Unit = {},
     isFullScreen: Boolean = false
 ) {
     BackHandler {
@@ -83,10 +83,11 @@ fun ReplyDetailsScreen(
                     email = replyUiState.currentSelectedEmail,
                     mailboxType = replyUiState.currentMailbox,
                     isFullScreen = isFullScreen,
-                    modifier = if (isFullScreen)
+                    modifier = if (isFullScreen) {
                         Modifier.padding(horizontal = dimensionResource(R.dimen.detail_card_outer_padding_horizontal))
-                    else
+                    } else {
                         Modifier.padding(end = dimensionResource(R.dimen.detail_card_outer_padding_horizontal))
+                    }
                 )
             }
         }
@@ -287,17 +288,20 @@ private fun ActionButton(
                 .padding(vertical = dimensionResource(R.dimen.detail_action_button_padding_vertical)),
             colors = ButtonDefaults.buttonColors(
                 containerColor =
-                if (!containIrreversibleAction)
+                if (!containIrreversibleAction) {
                     MaterialTheme.colorScheme.primaryContainer
-                else MaterialTheme.colorScheme.onErrorContainer
+                } else {
+                    MaterialTheme.colorScheme.onErrorContainer
+                }
             )
         ) {
             Text(
                 text = text,
-                color =
-                if (!containIrreversibleAction)
+                color = if (!containIrreversibleAction) {
                     MaterialTheme.colorScheme.onSurfaceVariant
-                else MaterialTheme.colorScheme.onError
+                } else {
+                    MaterialTheme.colorScheme.onError
+                }
             )
         }
     }

app/src/main/java/com/example/reply/ui/ReplyHomeContent.kt

@@ -130,10 +130,11 @@ fun ReplyEmailListItem(
     Card(
         modifier = modifier,
         colors = CardDefaults.cardColors(
-            containerColor = if (selected)
+            containerColor = if (selected) {
                 MaterialTheme.colorScheme.primaryContainer
-            else
+            } else {
                 MaterialTheme.colorScheme.secondaryContainer
+            }
         ),
         onClick = onCardClick
     ) {

app/src/main/java/com/example/reply/ui/ReplyHomeScreen.kt

@@ -33,7 +33,6 @@ import androidx.compose.material.icons.filled.Drafts
 import androidx.compose.material.icons.filled.Inbox
 import androidx.compose.material.icons.filled.Report
 import androidx.compose.material.icons.filled.Send
-import androidx.compose.material3.ExperimentalMaterial3Api
 import androidx.compose.material3.Icon
 import androidx.compose.material3.MaterialTheme
 import androidx.compose.material3.NavigationBar
@@ -60,7 +59,6 @@ import com.example.reply.data.local.LocalAccountsDataProvider
 import com.example.reply.ui.utils.ReplyContentType
 import com.example.reply.ui.utils.ReplyNavigationType
 
-@OptIn(ExperimentalMaterial3Api::class)
 @Composable
 fun ReplyHomeScreen(
     navigationType: ReplyNavigationType,
@@ -252,7 +250,6 @@ private fun ReplyBottomNavigationBar(
     }
 }
 
-@OptIn(ExperimentalMaterial3Api::class)
 @Composable
 private fun NavigationDrawerContent(
     selectedDestination: MailboxType,
@@ -313,4 +310,4 @@ private data class NavigationItemContent(
     val mailboxType: MailboxType,
     val icon: ImageVector,
     val text: String
-)
+)

app/src/main/java/com/example/reply/ui/ReplyViewModel.kt

@@ -69,4 +69,4 @@ class ReplyViewModel : ViewModel() {
             )
         }
     }
-}
+}