ENH: Better looking status bar icon

MSG Desktop.xcodeproj/project.pbxproj

@@ -397,7 +397,7 @@
 				COMBINE_HIDPI_IMAGES = YES;
 				CURRENT_PROJECT_VERSION = 1;
 				DEVELOPMENT_ASSET_PATHS = "\"MSG Desktop/Preview Content\"";
-				DEVELOPMENT_TEAM = C8Z9PRF4VH;
+				DEVELOPMENT_TEAM = RN94J52F92;
 				ENABLE_HARDENED_RUNTIME = YES;
 				ENABLE_PREVIEWS = YES;
 				GENERATE_INFOPLIST_FILE = YES;
@@ -424,7 +424,7 @@
 				COMBINE_HIDPI_IMAGES = YES;
 				CURRENT_PROJECT_VERSION = 1;
 				DEVELOPMENT_ASSET_PATHS = "\"MSG Desktop/Preview Content\"";
-				DEVELOPMENT_TEAM = C8Z9PRF4VH;
+				DEVELOPMENT_TEAM = RN94J52F92;
 				ENABLE_HARDENED_RUNTIME = YES;
 				ENABLE_PREVIEWS = YES;
 				GENERATE_INFOPLIST_FILE = YES;

MSG Desktop/Assets.xcassets/AppStatusBar.imageset/Contents.json

@@ -0,0 +1,23 @@
+{
+  "images" : [
+    {
+      "filename" : "512@1x.png",
+      "idiom" : "universal",
+      "scale" : "1x"
+    },
+    {
+      "filename" : "512@2x.png",
+      "idiom" : "universal",
+      "scale" : "2x"
+    },
+    {
+      "filename" : "1024.png",
+      "idiom" : "universal",
+      "scale" : "3x"
+    }
+  ],
+  "info" : {
+    "author" : "xcode",
+    "version" : 1
+  }
+}

MSG Desktop/ContentView.swift

@@ -7,38 +7,61 @@
 
 import SwiftUI
 
+struct ViewHeightKey: PreferenceKey {
+    static var defaultValue: CGFloat = 0
+    static func reduce(value: inout CGFloat, nextValue: () -> CGFloat) {
+        value = nextValue()
+    }
+}
+
 struct ContentView: View {
     @ObservedObject var appState = AppState.shared
     private let maxLines = 300
     @State private var showCancelButton: Bool = false
     @State private var showReinitPopup = false
     @State private var showCancelPopup = false
+    @State private var imageHeight: CGFloat = 0
 
     var body: some View {
-        VStack {
-//            Image(systemName: "globe")
-//                .imageScale(.large)
-//                .foregroundStyle(.tint)
+        GeometryReader { geometry in
+            VStack(spacing: 10) {
+                // App icon
                 let appIcon = NSImage(named: NSImage.applicationIconName)
-
-                    // Use SwiftUI's Image to display the icon
                 Image(nsImage: appIcon ?? NSImage())
-                .resizable() // Optional: if you want to resize the icon
-                .scaledToFit() // Optional: makes sure the aspect ratio is maintained
+                    .   resizable()
+                    .scaledToFit()
+                    .frame(maxHeight: 150)
+                    .background(
+                        GeometryReader { proxy in
+                            Color.clear
+                                .preference(key: ViewHeightKey.self, value: proxy.size.height)
+                        }
+                    )
+                    .onPreferenceChange(ViewHeightKey.self) { height in
+                        self.imageHeight = height
+                    }
                 
+                ZStack {
+                    Color.black
+                        .ignoresSafeArea()
                     
+                    // ScrollView fills remaining height
                     ScrollViewReader { proxy in
                         ScrollView {
-                    VStack(alignment: .leading) {
-                        TextEditor(text: $appState.output)
-                            .font(.system(.body, design: .monospaced))
-                            .frame(minWidth: 750, minHeight: 300)
-                            .padding()
-                            .disabled(true)
+                            LazyVStack(alignment: .leading, spacing: 0) {
+                                let lines = appState.output.components(separatedBy: .newlines)
+                                
+                                ForEach(lines.indices, id: \.self) { index in
+                                    Text(lines[index])
+                                        .font(.system(.body, design: .monospaced))
+                                        .foregroundColor(.gray)
+                                        .frame(maxWidth: .infinity, alignment: .leading)
                                     
-                        Text("")
-                            .id("bottom")
                                 }
+                                
+                                Color.clear.frame(height: 1).id("bottom")
+                            }
+                            .padding()
                         }
                         .onChange(of: appState.output) { _, _ in
                             trimLinesIfNeeded()
@@ -46,52 +69,62 @@ struct ContentView: View {
                                 proxy.scrollTo("bottom", anchor: .bottom)
                             }
                         }
+                        
+                        //                    ScrollView {
+                        //                        VStack(alignment: .leading) {
+                        //                            TextEditor(text: $appState.output)
+                        //                                .font(.system(.body, design: .monospaced))
+                        //                                .frame(
+                        //                                    width: geometry.size.width - 60,
+                        //                                    height: max(0, geometry.size.height - imageHeight - 120) // - rough height of button row and padding
+                        //                                )
+                        //                                .padding()
+                        //                                .disabled(true)
+                        //
+                        //                            Text("")
+                        //                                .id("bottom")
+                        //                        }
+                        //                    }
+                        //                    .onChange(of: appState.output) { _, _ in
+                        //                        trimLinesIfNeeded()
+                        //                        withAnimation {
+                        //                            proxy.scrollTo("bottom", anchor: .bottom)
+                        //                        }
+                        //                    }
                     }
+                }
+
+                // Buttons
                 HStack {
-                Button(action: {
-                    DispatchQueue.main.async {
-                        self.showCancelButton = true
-                    }
+                    Button("Init") {
+                        DispatchQueue.main.async { self.showCancelButton = true }
                         runShellCommandAsync(command: "msg machine init", outputText: $appState.output, showButton: $showCancelButton)
-                     
-                }) {
-                    Text("Init")
                     }
 
-                
-                Button(action: {
+                    Button("Reinit") {
                         showReinitPopup.toggle()
-                                    }) {
-                    Text("Reinit")
-                }.confirmationDialog("Are you sure?", isPresented: $showReinitPopup, titleVisibility: .visible) {
+                    }
+                    .confirmationDialog("Are you sure?", isPresented: $showReinitPopup, titleVisibility: .visible) {
                         Button("Confirm", role: .destructive) {
-                        DispatchQueue.main.async {
                             self.showCancelButton = true
                             runShellCommandAsync(command: "rm -rf ~/.guix && msg machine init", outputText: $appState.output, showButton: $showCancelButton)
                         }
-                        
-                    }
                         Button("Cancel", role: .cancel) {}
                     }
 
-                
-                Button(action: {
-                    runShellCommandAsync(command: "msg machine start", outputText: $appState.output, showButton: nil)                }) {
-                    Text("Start")
+                    Button("Start") {
+                        runShellCommandAsync(command: "msg machine start", outputText: $appState.output, showButton: nil)
                     }
-                if showCancelButton {
-                    Button(action: {
-                        showCancelPopup.toggle()
-                        }) {
-                        Text("Cancel")
-                    }.confirmationDialog("Are you sure?", isPresented: $showCancelPopup, titleVisibility: .visible) {
-                        Button("Confirm", role: .destructive) {
 
+                    if showCancelButton {
+                        Button("Cancel") {
+                            showCancelPopup.toggle()
+                        }
+                        .confirmationDialog("Are you sure?", isPresented: $showCancelPopup, titleVisibility: .visible) {
+                            Button("Confirm", role: .destructive) {
                                 runShellCommandAsync(command: "kill $(ps aux | grep '[g]uile' | awk '{print $2}')", outputText: $appState.output, showButton: nil)
                                 runShellCommandAsync(command: "kill $(ps aux | grep '[q]emu' | awk '{print $2}')", outputText: $appState.output, showButton: nil)
-                
                                 DispatchQueue.main.async {
-                                
                                     self.showCancelButton = false
                                 }
                             }
@@ -99,35 +132,34 @@ struct ContentView: View {
                         }
                     }
 
-                
-                if !showCancelButton{
-                    Button(action: {
-                        runShellCommandAsync(command: "msg machine stop", outputText: $appState.output, showButton: nil)                }) {
-                        Text("Stop")
+                    if !showCancelButton {
+                        Button("Stop") {
+                            runShellCommandAsync(command: "msg machine stop", outputText: $appState.output, showButton: nil)
                         }
                     }
 
-                
-                Button(action: {
-                    runShellCommandAsync(command: "osascript -e 'tell application \"Terminal\" to do script \"msg shell\"' -e 'tell application \"Terminal\" to activate'", outputText: $appState.output, showButton: nil)
-                    
-                }) {
-                    Text("Shell")
+                    Button("Shell") {
+                        runShellCommandAsync(
+                            command: "osascript -e 'tell application \"Terminal\" to do script \"msg shell\"' -e 'tell application \"Terminal\" to activate'",
+                            outputText: $appState.output,
+                            showButton: nil
+                        )
                     }
                 }
-            
-            
             }
             .padding()
-        .frame(width: 800, height: 400)
+            .frame(maxHeight: .infinity)
+            .onAppear {
+                // Silences error due to metallib on resizing of window
+                let device = MTLCreateSystemDefaultDevice()
+                _ = device?.makeDefaultLibrary()
+            }
+        }
     }
-
 
     private func trimLinesIfNeeded() {
         let lines = AppState.shared.output.split(separator: "\n")
-            
         if lines.count > maxLines {
-                _ = lines.count - maxLines
             let trimmedLines = lines.suffix(maxLines)
             AppState.shared.output = trimmedLines.joined(separator: "\n")
         }

MSG Desktop/MSG_Desktop.metal

@@ -0,0 +1,11 @@
+//
+//  MGS_Desktop.metal
+//  MSG Desktop
+//
+//  Created by Etienne Roesch on 31/03/2025.
+//
+
+#include <metal_stdlib>
+using namespace metal;
+
+

MSG Desktop/MSG_DesktopApp.swift

@@ -34,6 +34,7 @@ func runShellCommandAsync(command: String, outputText: Binding<String>?, showBut
         
         var observer: NSObjectProtocol?
         
+        
         observer = NotificationCenter.default.addObserver(forName: .NSFileHandleDataAvailable, object: fileHandle, queue: nil) { _ in
             let data = fileHandle.availableData
             if let output = String(data: data, encoding: .utf8), !output.isEmpty {
@@ -59,6 +60,8 @@ func runShellCommandAsync(command: String, outputText: Binding<String>?, showBut
 }
 
 class AppDelegate: NSObject, NSApplicationDelegate {
+    var window: NSWindow!
+    
     func applicationWillTerminate(_ notification: Notification) {
         runShellCommandAsync(command: "msg machine stop", outputText: nil, showButton: nil)
         runShellCommandAsync(command: "kill $(ps aux | grep '[g]uile' | awk '{print $2}')", outputText: nil, showButton: nil)
@@ -75,9 +78,28 @@ class AppDelegate: NSObject, NSApplicationDelegate {
     func applicationDidFinishLaunching(_ notification: Notification) {
         statusBarItem = NSStatusBar.system.statusItem(withLength: NSStatusItem.variableLength)
         
+        let contentView = ContentView()
+        window = NSWindow(
+            contentRect: NSRect(x: 0, y: 0, width: 800, height: 600),
+            styleMask: [.titled, .closable, .resizable, .miniaturizable],
+            backing: .buffered,
+            defer: false
+        )
+        window.center()
+        //window.setFrameAutosaveName("MSG App")   // to retain size after quitting
+        window.contentView = NSHostingView(rootView: contentView)
+        window.makeKeyAndOrderFront(nil)
+        
         if let button = statusBarItem.button {
-                button.title = "MSG"
+            //button.title = "MSG"
             button.toolTip = "MSG App"
+            if let image = NSImage(named: "AppStatusBar") {
+                let newSize = NSSize(width: 18, height: 18)
+                image.size = newSize
+                image.isTemplate = true // Important: so it adapts to dark/light mode but we need to get B&W icons..
+                button.image = image
+            }
+            button.target = self
         }
         
         let menu = NSMenu()
@@ -105,8 +127,6 @@ class AppDelegate: NSObject, NSApplicationDelegate {
     
 }
 
-
-
 class AppState: ObservableObject {
     @Published var output: String = "Please select an option \n"
     
@@ -118,8 +138,6 @@ struct MSG_DesktopApp: App {
     @NSApplicationDelegateAdaptor(AppDelegate.self) var appDelegate
     
     var body: some Scene {
-        WindowGroup {
-            ContentView()
-        }
+        Settings {} // prevents crash on some macOS version
     }
 }

README.org

@@ -1,4 +1,4 @@
-[[./example-gifs/init-example.gif]]
+[[./doc/images/init-example.gif]]
 
 * Install from tap
 ** Add Tap
@@ -10,3 +10,11 @@ brew tap MSG/apps https://forge.superkamiguru.org/MSG/homebrew-apps
 #+begin_src sh
 brew install msg-desktop
 #+end_src
+
+** Security & Privacy
+The first time you run the app, you will need to allow it to run in the Security
+& Privacy settings of your Mac. This is because the app is not signed by Apple.
+You can do this by going to System Preferences > Security & Privacy > General
+and clicking "Open Anyway" next to the message about the app being blocked.
+
+[[./doc/images/macos15.3.2-security.png]]