Fix enrolled stream popup actions and card link behavior.

Replace the comment row with a clearer open-link footer action and make announcement/coursework cards open their specific Classroom links when clicked.

Made-with: Cursor

classroom_app/ViewController.swift

@@ -5654,11 +5654,10 @@ private final class EnrolledClassDetailsViewController: NSViewController {
             container.addArrangedSubview(makeAttachmentPreview(item.attachments[0]))
         }
         if let link = item.alternateLink {
-            let open = makeInlineActionButton(title: "Open announcement", url: link)
+            let open = makeStreamFooterActionButton(title: "Open announcement", systemSymbol: "arrow.up.right.square", url: link)
             container.addArrangedSubview(open)
         }
-        container.addArrangedSubview(makeCommentRow())
-        return wrapInStreamCard(container)
+        return wrapInStreamCard(container, url: item.alternateLink)
     }
 
     private func makeCourseworkCard(_ item: ClassroomCourseWork) -> NSView {
@@ -5675,10 +5674,10 @@ private final class EnrolledClassDetailsViewController: NSViewController {
             container.addArrangedSubview(makeAttachmentPreview(item.attachments[0]))
         }
         if let link = item.alternateLink {
-            let open = makeInlineActionButton(title: "Open item", url: link)
+            let open = makeStreamFooterActionButton(title: "Open item", systemSymbol: "arrow.up.right.square", url: link)
             container.addArrangedSubview(open)
         }
-        return wrapInStreamCard(container)
+        return wrapInStreamCard(container, url: item.alternateLink)
     }
 
     private func makeAttachmentsList(_ attachments: [ClassroomAttachment]) -> NSView {
@@ -5747,32 +5746,40 @@ private final class EnrolledClassDetailsViewController: NSViewController {
         return body
     }
 
-    private func makeCommentRow() -> NSView {
+    private func makeStreamFooterActionButton(title: String, systemSymbol: String, url: URL) -> NSView {
         let divider = NSBox()
         divider.boxType = .separator
         divider.translatesAutoresizingMaskIntoConstraints = false
-
-        let comment = NSTextField(labelWithString: "Add comment")
-        comment.font = NSFont.systemFont(ofSize: 13, weight: .semibold)
-        comment.textColor = palette.primaryBlue
-
-        let icon = NSImageView()
-        icon.translatesAutoresizingMaskIntoConstraints = false
-        icon.image = NSImage(systemSymbolName: "bubble.left.and.bubble.right", accessibilityDescription: nil)
-        icon.symbolConfiguration = NSImage.SymbolConfiguration(pointSize: 13, weight: .medium)
-        icon.contentTintColor = palette.primaryBlue
-        icon.widthAnchor.constraint(equalToConstant: 15).isActive = true
-        icon.heightAnchor.constraint(equalToConstant: 15).isActive = true
-
-        let row = NSStackView(views: [icon, comment])
-        row.orientation = .horizontal
-        row.alignment = .centerY
-        row.spacing = 8
-
-        let container = NSStackView(views: [divider, row])
+        let actionButton = HoverButton(title: title, target: self, action: #selector(openLinkPressed(_:)))
+        actionButton.translatesAutoresizingMaskIntoConstraints = false
+        actionButton.isBordered = false
+        actionButton.bezelStyle = .regularSquare
+        actionButton.identifier = NSUserInterfaceItemIdentifier(url.absoluteString)
+        actionButton.wantsLayer = true
+        actionButton.layer?.cornerRadius = 8
+        actionButton.layer?.backgroundColor = NSColor.clear.cgColor
+        actionButton.font = NSFont.systemFont(ofSize: 13, weight: .semibold)
+        actionButton.contentTintColor = palette.primaryBlue
+        actionButton.image = NSImage(systemSymbolName: systemSymbol, accessibilityDescription: title)
+        actionButton.symbolConfiguration = NSImage.SymbolConfiguration(pointSize: 13, weight: .semibold)
+        actionButton.imagePosition = .imageLeading
+        actionButton.imageHugsTitle = true
+        actionButton.alignment = .left
+        actionButton.lineBreakMode = .byTruncatingTail
+        actionButton.heightAnchor.constraint(equalToConstant: 28).isActive = true
+
+        let baseHover = NSColor.clear
+        let hoverBg = palette.inputBackground.blended(withFraction: 0.20, of: palette.primaryBlue) ?? palette.inputBackground
+        actionButton.onHoverChanged = { [weak actionButton] hovering in
+            actionButton?.layer?.backgroundColor = (hovering ? hoverBg : baseHover).cgColor
+        }
+        actionButton.onHoverChanged?(false)
+
+        let container = NSStackView(views: [divider, actionButton])
+        container.translatesAutoresizingMaskIntoConstraints = false
         container.orientation = .vertical
         container.alignment = .width
-        container.spacing = 10
+        container.spacing = 8
         return container
     }
 
@@ -5834,26 +5841,6 @@ private final class EnrolledClassDetailsViewController: NSViewController {
         return preview
     }
 
-    private func makeInlineActionButton(title: String, url: URL) -> NSButton {
-        let open = HoverButton(title: title, target: self, action: #selector(openLinkPressed(_:)))
-        open.translatesAutoresizingMaskIntoConstraints = false
-        open.bezelStyle = .inline
-        open.identifier = NSUserInterfaceItemIdentifier(url.absoluteString)
-        open.font = NSFont.systemFont(ofSize: 12, weight: .semibold)
-        open.wantsLayer = true
-        open.layer?.cornerRadius = 8
-        open.layer?.backgroundColor = NSColor.clear.cgColor
-        open.onHoverChanged = { [weak open] hovering in
-            guard let open else { return }
-            let bg = hovering
-                ? (self.palette.inputBackground.blended(withFraction: 0.18, of: self.palette.primaryBlue) ?? self.palette.inputBackground)
-                : NSColor.clear
-            open.layer?.backgroundColor = bg.cgColor
-        }
-        open.onHoverChanged?(false)
-        return open
-    }
-
     private func makeStreamCardShell() -> NSView {
         let card = HoverTrackingView()
         card.translatesAutoresizingMaskIntoConstraints = false
@@ -5876,7 +5863,7 @@ private final class EnrolledClassDetailsViewController: NSViewController {
         return card
     }
 
-    private func wrapInStreamCard(_ content: NSView) -> NSView {
+    private func wrapInStreamCard(_ content: NSView, url: URL? = nil) -> NSView {
         let card = makeStreamCardShell()
         card.addSubview(content)
         NSLayoutConstraint.activate([
@@ -5886,7 +5873,22 @@ private final class EnrolledClassDetailsViewController: NSViewController {
             content.bottomAnchor.constraint(equalTo: card.bottomAnchor, constant: -10),
             card.widthAnchor.constraint(greaterThanOrEqualToConstant: 220)
         ])
-        return card
+        guard let url else { return card }
+
+        let hit = HoverButton(title: "", target: self, action: #selector(openLinkPressed(_:)))
+        hit.translatesAutoresizingMaskIntoConstraints = false
+        hit.isBordered = false
+        hit.bezelStyle = .regularSquare
+        hit.identifier = NSUserInterfaceItemIdentifier(url.absoluteString)
+        hit.toolTip = url.absoluteString
+        hit.addSubview(card)
+        NSLayoutConstraint.activate([
+            card.leadingAnchor.constraint(equalTo: hit.leadingAnchor),
+            card.trailingAnchor.constraint(equalTo: hit.trailingAnchor),
+            card.topAnchor.constraint(equalTo: hit.topAnchor),
+            card.bottomAnchor.constraint(equalTo: hit.bottomAnchor)
+        ])
+        return hit
     }
 
     private func timeText(_ date: Date?) -> String {