Refactor dialog handling and improve AlertDialog UI

Refactored the showDialog function to accept a modifier and updated all AlertDialog usages to use RectangleShape and the modifier parameter. Improved dialog sizing and positioning by dynamically adjusting the window size based on content, and set additional window properties for better integration on macOS.

Author: Stefterv

app/src/processing/app/Messages.kt

@@ -19,23 +19,19 @@
 package processing.app
 
 import androidx.compose.foundation.layout.Column
-import androidx.compose.foundation.layout.fillMaxSize
 import androidx.compose.foundation.layout.padding
 import androidx.compose.material.icons.Icons
-import androidx.compose.material.icons.filled.Alarm
 import androidx.compose.material.icons.filled.Error
 import androidx.compose.material.icons.filled.Info
 import androidx.compose.material.icons.filled.Warning
-import androidx.compose.material3.AlertDialog
-import androidx.compose.material3.Button
-import androidx.compose.material3.ButtonDefaults
-import androidx.compose.material3.Icon
-import androidx.compose.material3.MaterialTheme
-import androidx.compose.material3.OutlinedButton
-import androidx.compose.material3.Text
+import androidx.compose.material3.*
 import androidx.compose.runtime.Composable
+import androidx.compose.ui.ExperimentalComposeUiApi
 import androidx.compose.ui.Modifier
 import androidx.compose.ui.awt.ComposeDialog
+import androidx.compose.ui.graphics.RectangleShape
+import androidx.compose.ui.layout.onSizeChanged
+import androidx.compose.ui.platform.LocalDensity
 import androidx.compose.ui.text.font.FontWeight
 import androidx.compose.ui.unit.DpSize
 import androidx.compose.ui.unit.dp
@@ -46,14 +42,13 @@ import com.formdev.flatlaf.FlatLightLaf
 import processing.app.ui.Toolkit
 import processing.app.ui.theme.PDETheme
 import java.awt.Dimension
-import java.awt.EventQueue
 import java.awt.Frame
 import java.io.PrintWriter
 import java.io.StringWriter
-import javax.swing.JFrame
 import javax.swing.JOptionPane
 import javax.swing.UIManager
 
+
 class Messages {
     companion object {
         /**
@@ -65,9 +60,11 @@ class Messages {
             if (Base.isCommandLine()) {
                 println("$title: $message")
             } else {
-                showDialog(title) { dismiss ->
+                showDialog(title) { modifier, dismiss ->
                     AlertDialog(
+                        modifier = modifier,
                         onDismissRequest = {  },
+                        shape = RectangleShape,
                         icon = { Icon(Icons.Default.Info, contentDescription = "Info!") },
                         title = { Text(title) },
                         text = { Text(message) },
@@ -96,9 +93,11 @@ class Messages {
             if (Base.isCommandLine()) {
                 println("$title: $message")
             } else {
-                showDialog(title){ dismiss ->
+                showDialog(title){ modifier, dismiss ->
                     AlertDialog(
+                        modifier = modifier,
                         onDismissRequest = {  },
+                        shape = RectangleShape,
                         icon = { Icon(Icons.Default.Warning, contentDescription = "Alert!") },
                         iconContentColor = MaterialTheme.colorScheme.tertiary,
                         title = { Text(title) },
@@ -134,9 +133,11 @@ class Messages {
                 //      proper parsing on the command line. Many have \n in them.
                 println("$title: $primary\n$secondary")
             } else {
-                showDialog(title){ dismiss ->
+                showDialog(title){ modifier, dismiss ->
                     AlertDialog(
+                        modifier = modifier,
                         onDismissRequest = {  },
+                        shape = RectangleShape,
                         icon = { Icon(Icons.Default.Warning, contentDescription = "Alert!") },
                         iconContentColor = MaterialTheme.colorScheme.tertiary,
                         title = { Text(title) },
@@ -173,9 +174,11 @@ class Messages {
             if (Base.isCommandLine()) {
                 System.err.println("$title: $message")
             } else {
-                showDialog(title){ dismiss ->
+                showDialog(title){ modifier, dismiss ->
                     AlertDialog(
+                        modifier = modifier,
                         onDismissRequest = {  },
+                        shape = RectangleShape,
                         icon = { Icon(Icons.Default.Error, contentDescription = "Alert!") },
                         iconContentColor = MaterialTheme.colorScheme.error,
                         title = { Text(title) },
@@ -360,17 +363,24 @@ class Messages {
     }
 }
 
-fun showDialog(title: String, content: @Composable (dismiss: () -> Unit) -> Unit) {
+@OptIn(ExperimentalComposeUiApi::class)
+fun showDialog(title: String, content: @Composable (modifier: Modifier,  dismiss: () -> Unit) -> Unit) {
     ComposeDialog().apply {
-//        isUndecorated = true
-//        isTransparent = true
         isModal = true
         setTitle(title)
-        size = Dimension(400, 200)
+        size = Dimension(400, 400)
+        rootPane.putClientProperty("apple.awt.fullWindowContent", true)
+        rootPane.putClientProperty("apple.awt.transparentTitleBar", true)
+        rootPane.putClientProperty("apple.awt.windowTitleVisible", false);
+
 
-        setContent {
+       setContent {
             PDETheme {
-                content(::dispose)
+                val density = LocalDensity.current
+                content(Modifier.onSizeChanged{
+                    size = Dimension((it.width / density.density).toInt(), (it.height / density.density).toInt())
+                    setLocationRelativeTo(null)
+                },::dispose)
             }
         }
         setLocationRelativeTo(null)