feat: add maximum content width constraint for wide displays (Closes #2)
This commit was merged in pull request #41.
This commit is contained in:
@@ -20,6 +20,7 @@ import androidx.compose.ui.Modifier
|
||||
import androidx.compose.ui.unit.dp
|
||||
import kotlinx.coroutines.launch
|
||||
import org.jetbrains.compose.ui.tooling.preview.Preview
|
||||
import org.shahondin1624.lib.components.UiConstants
|
||||
import org.shahondin1624.lib.components.charactermodel.attributespage.AttributesPage
|
||||
import org.shahondin1624.model.EXAMPLE_CHARACTER
|
||||
import org.shahondin1624.theme.AppTheme
|
||||
@@ -121,15 +122,22 @@ private fun AppContent(
|
||||
)
|
||||
}
|
||||
) { paddingValues ->
|
||||
Column(
|
||||
Box(
|
||||
modifier = Modifier
|
||||
.fillMaxSize()
|
||||
.padding(paddingValues)
|
||||
.windowInsetsPadding(WindowInsets.safeDrawing)
|
||||
.padding(16.dp),
|
||||
horizontalAlignment = Alignment.CenterHorizontally
|
||||
.windowInsetsPadding(WindowInsets.safeDrawing),
|
||||
contentAlignment = Alignment.TopCenter
|
||||
) {
|
||||
AttributesPage(character)
|
||||
Column(
|
||||
modifier = Modifier
|
||||
.widthIn(max = UiConstants.MAX_CONTENT_WIDTH)
|
||||
.fillMaxWidth()
|
||||
.padding(16.dp),
|
||||
horizontalAlignment = Alignment.CenterHorizontally
|
||||
) {
|
||||
AttributesPage(character)
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
@@ -4,4 +4,7 @@ import androidx.compose.ui.unit.dp
|
||||
|
||||
object UiConstants {
|
||||
val SMALL_PADDING = 6.dp
|
||||
|
||||
/** Maximum width for the main content area to prevent uncomfortable stretching on ultra-wide displays. */
|
||||
val MAX_CONTENT_WIDTH = 1200.dp
|
||||
}
|
||||
Reference in New Issue
Block a user