Skip to content

Commit d171489

Browse files
committed
Code cleanup
1 parent 8682c0b commit d171489

File tree

1 file changed

+15
-21
lines changed
  • AdaptiveNavigationSample/app/src/main/java/com/google/sample/adaptivenavigationsample

1 file changed

+15
-21
lines changed

AdaptiveNavigationSample/app/src/main/java/com/google/sample/adaptivenavigationsample/MainActivity.kt

Lines changed: 15 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -65,8 +65,7 @@ enum class AppDestinations(
6565
val iconSelected: ImageVector,
6666
@StringRes val contentDescription: Int
6767
) {
68-
HOME(R.string.home, Icons.Outlined.Home, Icons.Filled.Home, R.string.home),
69-
FAVORITES(
68+
HOME(R.string.home, Icons.Outlined.Home, Icons.Filled.Home, R.string.home), FAVORITES(
7069
R.string.favorites, Icons.Outlined.Favorite, Icons.Filled.Favorite, R.string.favorites
7170
),
7271
SHOPPING(
@@ -92,16 +91,15 @@ class MainActivity : ComponentActivity() {
9291
@Composable
9392
fun NavigationSample() {
9493
var selectedItem by remember { mutableIntStateOf(0) }
95-
var verticalArrangement by remember { mutableStateOf<Arrangement.Vertical>(Arrangement.Center) }
96-
val radioOptions = listOf(
97-
"Top" to Arrangement.Top,
98-
"Center" to Arrangement.Center,
99-
"Bottom" to Arrangement.Bottom
94+
val options = listOf(
95+
"Top" to Arrangement.Top, "Center" to Arrangement.Center, "Bottom" to Arrangement.Bottom
10096
)
101-
val (selectedOption, onOptionSelected) = remember { mutableStateOf(radioOptions[1].first) }
97+
98+
val (verticalArrangement, onOptionSelected) = remember { mutableStateOf(options[1]) }
10299

103100
NavigationSuiteScaffold(
104-
modifier = Modifier.fillMaxSize(), navigationItems = {
101+
modifier = Modifier.fillMaxSize(),
102+
navigationItems = {
105103
AppDestinations.entries.forEachIndexed { index, item ->
106104
NavigationSuiteItem(
107105
icon = {
@@ -118,7 +116,8 @@ fun NavigationSample() {
118116
selected = index == selectedItem,
119117
onClick = { selectedItem = index })
120118
}
121-
}, navigationItemVerticalArrangement = verticalArrangement
119+
},
120+
navigationItemVerticalArrangement = verticalArrangement.second
122121
) {
123122
Column(
124123
modifier = Modifier
@@ -127,29 +126,24 @@ fun NavigationSample() {
127126
verticalArrangement = Arrangement.Center
128127
) {
129128
Text("NavRail icons arrangement", style = MaterialTheme.typography.headlineSmall)
130-
radioOptions.forEach { item ->
131-
val (text, arrangement) = item
129+
options.forEach { item ->
132130
Row(
133131
Modifier
134132
.fillMaxWidth()
135133
.height(56.dp)
136134
.selectable(
137-
selected = (text == selectedOption),
138-
onClick = {
139-
onOptionSelected(text);
140-
verticalArrangement = arrangement
141-
},
142-
role = Role.RadioButton
135+
selected = (item == verticalArrangement), onClick = {
136+
onOptionSelected(item)
137+
}, role = Role.RadioButton
143138
)
144139
.padding(horizontal = 16.dp),
145140
verticalAlignment = Alignment.CenterVertically
146141
) {
147142
RadioButton(
148-
selected = (text == selectedOption),
149-
onClick = null
143+
selected = (item == verticalArrangement), onClick = null
150144
)
151145
Text(
152-
text = text,
146+
text = item.first,
153147
style = MaterialTheme.typography.bodyLarge,
154148
modifier = Modifier.padding(start = 16.dp)
155149
)

0 commit comments

Comments
 (0)