Tasks and Activity Viewer UI Improvements (#1831)

### Description

This PR refines the UI around Tasks and the Activity Viewer.

### Screenshots

https://github.com/user-attachments/assets/2edbd761-b09b-4a06-ad74-45825d1effa3

<img width="309" alt="image" src="https://github.com/user-attachments/assets/dfa6cd2b-3191-4a6c-80a3-28e16e654283">
<br />
<img width="303" alt="image" src="https://github.com/user-attachments/assets/8d77d592-7cbf-46b5-a9c2-c155bd3d0eda">


https://github.com/user-attachments/assets/6fae003d-244c-42eb-8ec1-22c27b6b9e9c

### Commits

* Made a few minor adjustments to the activity bar UI.

* Cleaned up activity viewer menus

* Gave spacers a minLength

* Unified TaskView duplication. Refined workspace settings and add and edit task sheet design

* Displaying workspace settings in a sheet instead of a window. Added minimum width to workspace settings bottons.

* Improved the activity viewer notification details popover design

* Conditional display and animation of stop toolbar button

* Added activity viewer notification animations.

* Fix Task Renew Race Condition, Stop Button Listeners

* Fixed SwiftLint error and reorganized Task folder structure

* Resolved PR issues. When running a task, the task output appears in the utility area and drawer opens if closed.

* Fixed SwiftLint errors

* PR issues

* Removed statusSubject and statusPublisher

* Fixed bug where you could not rerun a task that was finished

---------

Co-authored-by: Khan Winter <35942988+thecoolwinter@users.noreply.github.com>

Author: austincondiff

CodeEdit.xcodeproj/project.pbxproj

@@ -46,11 +46,14 @@ struct ActivityViewer: View {
                 .fixedSize()
         }
         .fixedSize(horizontal: false, vertical: false)
-        .padding(.horizontal, 10)
+        .padding(.horizontal, 5)
+        .padding(.vertical, 1.5)
+        .frame(height: 22)
+        .clipped()
         .background {
             if colorScheme == .dark {
                 RoundedRectangle(cornerRadius: 5)
-                    .opacity(0.10)
+                    .opacity(0.1)
             } else {
                 RoundedRectangle(cornerRadius: 5)
                     .opacity(0.1)

CodeEdit/Features/ActivityViewer/ActivityViewer.swift

@@ -1,19 +1,21 @@
 //
-//  CustomLoadingRingView.swift
+//  CECircularProgressView.swift
 //  CodeEdit
 //
 //  Created by Tommy Ludwig on 21.06.24.
 //
 
 import SwiftUI
 
-struct CustomLoadingRingView: View {
+struct CECircularProgressView: View {
     @State private var isAnimating = false
     @State private var previousValue: Bool = false
+
     var progress: Double?
-    var currentTaskCount: Int
+    var currentTaskCount: Int = 1
 
     let lineWidth: CGFloat = 2
+
     var body: some View {
         Circle()
             .stroke(style: StrokeStyle(lineWidth: lineWidth))
@@ -22,12 +24,12 @@ struct CustomLoadingRingView: View {
                 if let progress = progress {
                     Circle()
                         .trim(from: 0, to: progress)
-                        .stroke(Color.blue.gradient, style: StrokeStyle(lineWidth: lineWidth, lineCap: .round))
+                        .stroke(Color.accentColor, style: StrokeStyle(lineWidth: lineWidth, lineCap: .round))
                         .animation(.easeInOut, value: progress)
                 } else {
                     Circle()
                         .trim(from: 0, to: 0.5)
-                        .stroke(Color.blue.gradient, style: StrokeStyle(lineWidth: lineWidth, lineCap: .round))
+                        .stroke(Color.accentColor, style: StrokeStyle(lineWidth: lineWidth, lineCap: .round))
                         .rotationEffect(
                             previousValue ?
                                 .degrees(isAnimating ?  0 : -360)
@@ -41,6 +43,7 @@ struct CustomLoadingRingView: View {
                 }
             }
             .rotationEffect(.degrees(-90))
+            .padding(lineWidth/2)
             .overlay {
                 if currentTaskCount > 1 {
                     Text("\(currentTaskCount)")
@@ -52,10 +55,10 @@ struct CustomLoadingRingView: View {
 
 #Preview {
     Group {
-        CustomLoadingRingView(currentTaskCount: 1)
+        CECircularProgressView(currentTaskCount: 1)
             .frame(width: 22, height: 22)
 
-        CustomLoadingRingView(progress: 0.65, currentTaskCount: 1)
+        CECircularProgressView(progress: 0.65, currentTaskCount: 1)
             .frame(width: 22, height: 22)
     }
     .padding()

CodeEdit/Features/ActivityViewer/Notificaitons/TaskNotificationView.swift

@@ -0,0 +1,71 @@
+//
+//  TaskNotificationView.swift
+//  CodeEdit
+//
+//  Created by Tommy Ludwig on 21.06.24.
+//
+
+import SwiftUI
+
+struct TaskNotificationView: View {
+    @ObservedObject var taskNotificationHandler: TaskNotificationHandler
+    @State private var isPresented: Bool = false
+    @State var notification: TaskNotificationModel?
+
+    var body: some View {
+        ZStack {
+            if let notification {
+                HStack {
+                    Text(notification.title)
+                        .font(.subheadline)
+                        .transition(
+                            .asymmetric(insertion: .move(edge: .top), removal: .move(edge: .bottom))
+                            .combined(with: .opacity)
+                        )
+                        .id("NotificationTitle" + notification.title)
+
+                    if notification.isLoading {
+                        CECircularProgressView(
+                            progress: notification.percentage,
+                            currentTaskCount: taskNotificationHandler.notifications.count
+                        )
+                        .padding(.horizontal, -1)
+                        .frame(height: 16)
+                    } else {
+                        if taskNotificationHandler.notifications.count > 1 {
+                            Text("\(taskNotificationHandler.notifications.count)")
+                                .font(.caption)
+                                .padding(5)
+                                .background(
+                                    Circle()
+                                        .foregroundStyle(.gray)
+                                        .opacity(0.2)
+                                )
+                                .padding(-5)
+                        }
+                    }
+                }
+                .transition(.opacity.combined(with: .move(edge: .trailing)))
+                .padding(3)
+                .padding(-3)
+                .padding(.trailing, 3)
+                .popover(isPresented: $isPresented, arrowEdge: .bottom) {
+                    TaskNotificationsDetailView(taskNotificationHandler: taskNotificationHandler)
+                }.onTapGesture {
+                    self.isPresented.toggle()
+                }
+            }
+        }
+        .animation(.easeInOut, value: notification)
+        .onChange(of: taskNotificationHandler.notifications) { newValue in
+            withAnimation {
+                notification = newValue.first
+            }
+        }
+    }
+
+}
+
+#Preview {
+    TaskNotificationView(taskNotificationHandler: TaskNotificationHandler())
+}

CodeEdit/Features/ActivityViewer/Notificaitons/TaskNotificationsDetailView.swift

@@ -0,0 +1,48 @@
+//
+//  TaskNotificationsDetailView.swift
+//  CodeEdit
+//
+//  Created by Tommy Ludwig on 21.06.24.
+//
+
+import SwiftUI
+
+struct TaskNotificationsDetailView: View {
+    @ObservedObject var taskNotificationHandler: TaskNotificationHandler
+    @State private var selectedTaskNotificationIndex: Int = 0
+    var body: some View {
+        ScrollView {
+            VStack(alignment: .leading, spacing: 15) {
+                ForEach(taskNotificationHandler.notifications, id: \.id) { notification in
+                    HStack(alignment: .center, spacing: 8) {
+                        CECircularProgressView(progress: notification.percentage)
+                            .frame(width: 16, height: 16)
+                        VStack(alignment: .leading) {
+                            Text(notification.title)
+                                .fixedSize(horizontal: false, vertical: true)
+                                .transition(.identity)
+
+                            if let message = notification.message, !message.isEmpty {
+                                Text(message)
+                                    .font(.subheadline)
+                                    .foregroundStyle(.secondary)
+                            }
+                        }
+                        Spacer()
+                    }
+                }
+            }
+        }
+        .padding(15)
+        .frame(minWidth: 320)
+        .onChange(of: taskNotificationHandler.notifications) { newValue in
+            if selectedTaskNotificationIndex >= newValue.count {
+                selectedTaskNotificationIndex = 0
+            }
+        }
+    }
+}
+
+#Preview {
+    TaskNotificationsDetailView(taskNotificationHandler: TaskNotificationHandler())
+}

…otificaitons/CustomLoadingRingView.swift …tifications/CECircularProgressView.swiftCodeEdit/Features/ActivityViewer/Notificaitons/CustomLoadingRingView.swift renamed to CodeEdit/Features/ActivityViewer/Notifications/CECircularProgressView.swift CodeEdit/Features/ActivityViewer/Notificaitons/CustomLoadingRingView.swift renamed to CodeEdit/Features/ActivityViewer/Notifications/CECircularProgressView.swift

@@ -0,0 +1,21 @@
+//
+//  ActiveTaskView.swift
+//  CodeEdit
+//
+//  Created by Austin Condiff on 8/4/24.
+//
+
+import SwiftUI
+
+// We need to observe each active task individually because:
+// 1. Active tasks are nested inside TaskManager.
+// 2. Reference types (like objects) do not notify observers when their internal state changes.
+/// `ActiveTaskView` represents a single active task and observes its state.
+/// - Parameter activeTask: The active task to be displayed and observed.
+struct ActiveTaskView: View {
+    @ObservedObject var activeTask: CEActiveTask
+
+    var body: some View {
+        TaskView(task: activeTask.task, status: activeTask.status)
+    }
+}