Trace UI enhancements

[shubham7109]

Related to issue: #5512

Description:

PR to update the UtilityNetworkTrace tool's UI to handle user Modifier content sizes dynamically. (fillMaxSize & wrapContentSize etc). Currently, a fillMaxSize Modifier is hard-coded within the Trace surface scaffold.

This PR aims to update the UI to adapt the Trace composable to react dynamically towards constraint sizes between Modifier content size changes, and TraceNavigation composable content changes.

Summary of changes:

• Removed fillMaxSize from TraceScaffold
• Removed fillMaxSize from StartingPointRow
• Added a sizeTransform to the NavHost to be able to react to content size changes like between the TraceOptionsScreen -> AddStartingPointScreen -> TraceOptionsScreen
• Added max weight to the LazyColumn in TraceOptionsScreen such that the "Trace" button is always visible in the content.
• Similarly, added max weight to the LazyColumn in TraceResultScreen such that the "Clear all results" button is always visible in the content.
• Added minor layout fixes to help keep a consistent vertical padding & vertical centering with the above layout content changes.

Test code

package com.arcgismaps.toolkit.utilitynetworktraceapp.screens

import androidx.compose.foundation.layout.fillMaxSize
import androidx.compose.foundation.layout.padding
import androidx.compose.material3.BottomSheetScaffold
import androidx.compose.material3.ExperimentalMaterial3Api
import androidx.compose.material3.MaterialTheme
import androidx.compose.material3.SheetValue
import androidx.compose.material3.rememberBottomSheetScaffoldState
import androidx.compose.material3.rememberStandardBottomSheetState
import androidx.compose.runtime.Composable
import androidx.compose.runtime.rememberCoroutineScope
import androidx.compose.ui.Modifier
import com.arcgismaps.toolkit.geoviewcompose.MapView
import com.arcgismaps.toolkit.utilitynetworks.Trace
import kotlinx.coroutines.launch

@OptIn(ExperimentalMaterial3Api::class)
@Composable
fun MainScreen(viewModel: TraceViewModel) {
    val coroutineScope = rememberCoroutineScope()

    val scaffoldState = rememberBottomSheetScaffoldState(
        bottomSheetState = rememberStandardBottomSheetState(
            initialValue = SheetValue.Expanded,
            skipHiddenState = true
        )
    )

    BottomSheetScaffold(
        sheetContent = { Trace(traceState = viewModel.traceState) },
        modifier = Modifier.fillMaxSize(),
        scaffoldState = scaffoldState,
        sheetContainerColor = MaterialTheme.colorScheme.background,
        sheetSwipeEnabled = true,
    ) { padding ->
        MapView(
            arcGISMap = viewModel.arcGISMap,
            mapViewProxy = viewModel.mapViewProxy,
            graphicsOverlays = listOf(viewModel.graphicsOverlay),
            modifier = Modifier
                .padding(padding)
                .fillMaxSize(),
            onSingleTapConfirmed = { singleTapConfirmedEvent ->
                singleTapConfirmedEvent.mapPoint?.let {
                    coroutineScope.launch {
                        viewModel.traceState.addStartingPoint(it)
                    }
                }
            }
        )
    }
}

Pre-merge Checklist

• a vTest Job for this PR has been run
  • link: https://runtime-kotlin.esri.com/view/all/job/vtest/job/toolkit/531/
• Unit and/or integration tests have been added to exercise this PR's logic, and the tests are passing:
  • Yes
  • No

[sorenoid]

In addition to removing this line, consider editing TraceScaffold to provide a default parameter modifier = Modifier.fillMaxSize(). Current behavior would be preserved.