Archives/JetBrains_intellij-platform-plugin-template
1
0
Fork 0
mirror of https://github.com/JetBrains/intellij-platform-plugin-template.git synced 2025-12-05 06:11:52 +00:00
Code Issues Projects Releases Wiki Activity

Cleanup modifier usages

Browse Source
This commit is contained in:
Nebojsa Vuksic 2025-07-29 15:21:44 +02:00
parent 291a35b6bd
commit b800d6bb17
2 changed files with 25 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

35
src/main/kotlin/org/jetbrains/plugins/template/weatherApp/ui/WeatherAppSample.kt
View File

@ -17,9 +17,13 @@ import org.jetbrains.jewel.foundation.lazy.items
import org.jetbrains.jewel.foundation.lazy.rememberSelectableLazyListState
import org.jetbrains.jewel.foundation.theme.JewelTheme
import org.jetbrains.jewel.ui.component.*
import org.jetbrains.plugins.template.ComposeTemplateBundle
import org.jetbrains.plugins.template.weatherApp.model.Location
import org.jetbrains.plugins.template.weatherApp.model.WeatherForecastData
import org.jetbrains.plugins.template.weatherApp.services.*
import org.jetbrains.plugins.template.weatherApp.services.LocationsProvider
import org.jetbrains.plugins.template.weatherApp.services.MyLocationsViewModelApi
import org.jetbrains.plugins.template.weatherApp.services.SearchAutoCompletionItemProvider
import org.jetbrains.plugins.template.weatherApp.services.WeatherViewModelApi
import org.jetbrains.plugins.template.weatherApp.ui.components.SearchToolbarMenu
import org.jetbrains.plugins.template.weatherApp.ui.components.WeatherDetailsCard
@ -30,16 +34,27 @@ internal fun WeatherAppSample(
searchAutoCompletionItemProvided: LocationsProvider
) {
HorizontalSplitLayout(
first = { LeftColumn(myLocationViewModel, modifier = Modifier.fillMaxSize()) },
first = {
LeftColumn(
myLocationViewModel,
modifier = Modifier
.fillMaxSize()
.padding(start = 8.dp, end = 8.dp)
)
},
second = {
RightColumn(
myLocationViewModel,
weatherViewModelApi,
searchAutoCompletionItemProvided,
modifier = Modifier.fillMaxSize()
modifier = Modifier
.fillMaxSize()
.padding(start = 8.dp, end = 8.dp)
)
},
modifier = Modifier.fillMaxSize(),
modifier = Modifier
.fillMaxSize()
.padding(all = 8.dp),
firstPaneMinWidth = 100.dp,
secondPaneMinWidth = 300.dp,
state = rememberSplitLayoutState(.2f)
@ -55,10 +70,10 @@ private fun LeftColumn(
// TODO Set selected item on initial showing
Column(modifier.fillMaxSize().padding(8.dp)) {
GroupHeader("My Locations", modifier = Modifier.padding(bottom = 8.dp))
Column(modifier) {
GroupHeader("My Locations", modifier = Modifier.wrapContentHeight().fillMaxWidth())
Spacer(modifier = Modifier.height(4.dp))
Spacer(modifier = Modifier.height(10.dp))
val listState = rememberSelectableLazyListState()
LaunchedEffect(myLocations) {
@ -69,7 +84,7 @@ private fun LeftColumn(
}
SelectableLazyColumn(
modifier = modifier.fillMaxSize(),
modifier = Modifier.fillMaxSize(),
selectionMode = SelectionMode.Single,
state = listState,
onSelectedIndexesChange = { indices ->
@ -120,7 +135,7 @@ private fun RightColumn(
) {
val weatherForecastData = weatherViewModelApi.weatherForecast.collectAsState(WeatherForecastData.EMPTY).value
Column(modifier.fillMaxWidth().padding(8.dp)) {
Column(modifier) {
SearchToolbarMenu(
searchAutoCompletionItemProvider = searchAutoCompletionItemProvider,
confirmButtonText = "Add",
@ -135,7 +150,7 @@ private fun RightColumn(
WeatherDetailsCard(
modifier = Modifier
.fillMaxWidth()
.align(Alignment.CenterHorizontally), // optional for positioning
.align(Alignment.CenterHorizontally),
weatherForecastData
) {
weatherViewModelApi.onReloadWeatherForecast()

1
src/main/kotlin/org/jetbrains/plugins/template/weatherApp/ui/components/WeatherDetailsCard.kt
View File

@ -46,7 +46,6 @@ internal fun WeatherDetailsCard(
Box(
modifier = modifier
.fillMaxWidth()
.clip(RoundedCornerShape(16.dp))
.background(cardColor)
.padding(16.dp)