From 7de83e5e6f111455aeb02a996c5ad7bf0f3c515d Mon Sep 17 00:00:00 2001
From: Dan Nesfeder <daniel.nesfeder@proof.com>
Date: Thu, 20 Aug 2026 12:38:39 -0500
Subject: [PATCH] Make the create-credential auth WebView scrollable and
 full-height

The auth WebView was wrapped in a LazyColumn item with wrap-content
height. The WebView collapses to its initial content height (leaving
the sheet nearly empty), and the LazyColumn consumes vertical drag
gestures, so pages taller than the viewport cannot be scrolled -
interactive authorization flows get stuck on unreachable controls.

Let the auth section fill the sheet and give the WebView the height
remaining below the title, so it handles its own scrolling like a
normal browser surface on any screen size.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
---
 .../createcred/CreateCredentialActivity.kt    | 60 +++++++++----------
 1 file changed, 29 insertions(+), 31 deletions(-)

diff --git a/app/src/main/java/com/credman/cmwallet/createcred/CreateCredentialActivity.kt b/app/src/main/java/com/credman/cmwallet/createcred/CreateCredentialActivity.kt
index eb29fdc..8f90862 100644
--- a/app/src/main/java/com/credman/cmwallet/createcred/CreateCredentialActivity.kt
+++ b/app/src/main/java/com/credman/cmwallet/createcred/CreateCredentialActivity.kt
@@ -16,11 +16,10 @@ import androidx.compose.foundation.layout.Arrangement
 import androidx.compose.foundation.layout.Box
 import androidx.compose.foundation.layout.Column
 import androidx.compose.foundation.layout.Row
+import androidx.compose.foundation.layout.fillMaxSize
 import androidx.compose.foundation.layout.fillMaxWidth
-import androidx.compose.foundation.layout.height
 import androidx.compose.foundation.layout.padding
-import androidx.compose.foundation.layout.wrapContentHeight
-import androidx.compose.foundation.lazy.LazyColumn
+import androidx.compose.foundation.layout.statusBarsPadding
 import androidx.compose.material3.BottomSheetDefaults
 import androidx.compose.material3.Button
 import androidx.compose.material3.ExperimentalMaterial3Api
@@ -102,13 +101,14 @@ class CreateCredentialActivity : ComponentActivity() {
             onDismissRequest = {
                 this@CreateCredentialActivity.finish()
             },
+            modifier = Modifier.statusBarsPadding(),
             sheetState = sheetState
         ) {
             val credentials = uiState.credentialsToSave
 
             if (uiState.authServer != null) {
                 Column(
-                    modifier = Modifier.fillMaxWidth(),
+                    modifier = Modifier.fillMaxSize(),
                     horizontalAlignment = Alignment.CenterHorizontally
                 ) {
                     Row(
@@ -122,7 +122,7 @@ class CreateCredentialActivity : ComponentActivity() {
                         )
                     }
                     Box(
-                        modifier = Modifier.fillMaxWidth().wrapContentHeight()
+                        modifier = Modifier.fillMaxWidth().weight(1f)
                     ) {
                         AuthWebView(
                             url = uiState.authServer.url,
@@ -289,40 +289,38 @@ class CreateCredentialActivity : ComponentActivity() {
         redirectUrl: String,
         onDone: (String) -> Unit
     ) {
-        LazyColumn(
-            modifier = Modifier.fillMaxWidth().wrapContentHeight()
+        Column(
+            modifier = Modifier.fillMaxSize()
         ) {
-            item {
-                AndroidView(factory = {
-                    WebView(it).apply {
-                        clearCache(true)
-                        settings.javaScriptEnabled = true
-                        this.layoutParams = ViewGroup.LayoutParams(
-                            ViewGroup.LayoutParams.MATCH_PARENT,
-                            ViewGroup.LayoutParams.WRAP_CONTENT
-                        )
-                        this.webViewClient = object : WebViewClient() {
-                            override fun shouldOverrideUrlLoading(
-                                view: WebView?,
-                                request: WebResourceRequest?
-                            ): Boolean {
+            AndroidView(modifier = Modifier.fillMaxSize(), factory = {
+                WebView(it).apply {
+                    clearCache(true)
+                    settings.javaScriptEnabled = true
+                    this.layoutParams = ViewGroup.LayoutParams(
+                        ViewGroup.LayoutParams.MATCH_PARENT,
+                        ViewGroup.LayoutParams.WRAP_CONTENT
+                    )
+                    this.webViewClient = object : WebViewClient() {
+                        override fun shouldOverrideUrlLoading(
+                            view: WebView?,
+                            request: WebResourceRequest?
+                        ): Boolean {
 
-                                request?.let {
+                            request?.let {
 
-                                    if (request.url.toString().startsWith("$redirectUrl/")) {
-                                        request.url.getQueryParameter("code")?.let { code ->
-                                            onDone(code)
-                                        }
+                                if (request.url.toString().startsWith("$redirectUrl/")) {
+                                    request.url.getQueryParameter("code")?.let { code ->
+                                        onDone(code)
                                     }
                                 }
-                                return super.shouldOverrideUrlLoading(view, request)
                             }
+                            return super.shouldOverrideUrlLoading(view, request)
                         }
                     }
-                }, update = {
-                    it.loadUrl(url)
-                })
-            }
+                }
+            }, update = {
+                it.loadUrl(url)
+            })
         }
     }
 }