Merge pull request #203 from necatisozer/patch-1

Put some modifiers in correct order to respect official API guidelines

feature-foryou/src/main/java/com/google/samples/apps/nowinandroid/feature/foryou/ForYouScreen.kt

@@ -389,8 +389,8 @@ private fun SingleTopicButton(
 
 @Composable
 fun TopicIcon(
-    modifier: Modifier = Modifier,
-    imageUrl: String
+    imageUrl: String,
+    modifier: Modifier = Modifier
 ) {
     AsyncImage(
         // TODO b/228077205, show loading image visual instead of static placeholder

feature-interests/src/main/java/com/google/samples/apps/nowinandroid/feature/interests/InterestsScreen.kt

@@ -49,9 +49,9 @@ import com.google.samples.apps.nowinandroid.core.ui.JankMetricDisposableEffect
 @OptIn(ExperimentalLifecycleComposeApi::class)
 @Composable
 fun InterestsRoute(
-    modifier: Modifier = Modifier,
     navigateToAuthor: (String) -> Unit,
     navigateToTopic: (String) -> Unit,
+    modifier: Modifier = Modifier,
     viewModel: InterestsViewModel = hiltViewModel()
 ) {
     val uiState by viewModel.uiState.collectAsStateWithLifecycle()