fix: use SafariView for all links instead of jumping out of app

Changed link opening behavior to match the "Open in Browser" button:
- All links (internal V2EX and external) now open in SafariView
- Stays within the app instead of jumping to Safari
- Provides consistent user experience across the app

Changes:
- NewsContentView: Added SafariView sheet presentation
- ReplyItemView: Added SafariView sheet presentation
- Both views now use openInSafari() method
- Removed UIApplication.shared.open() calls

This matches the UX pattern used in FeedDetailPage's toolbar.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: graycreate

V2er/View/FeedDetail/NewsContentView.swift

@@ -13,7 +13,8 @@ struct NewsContentView: View {
     @Binding var rendered: Bool
     @EnvironmentObject var store: Store
     @Environment(\.colorScheme) var colorScheme
-    @State private var navigationPath = NavigationPath()
+    @State private var showingSafari = false
+    @State private var safariURL: URL?
 
     init(_ contentInfo: FeedDetailInfo.ContentInfo?, rendered: Binding<Bool>) {
         self.contentInfo = contentInfo
@@ -42,11 +43,15 @@ struct NewsContentView: View {
 
             Divider()
         }
+        .sheet(isPresented: $showingSafari) {
+            if let url = safariURL {
+                SafariView(url: url)
+            }
+        }
     }
 
     private func handleLinkTap(_ url: URL) {
         // Smart URL routing - parse V2EX URLs and route accordingly
-        let urlString = url.absoluteString
         let path = url.path
 
         // Check if it's a V2EX internal link
@@ -55,37 +60,42 @@ struct NewsContentView: View {
             if path.contains("/t/"), let topicId = extractTopicId(from: path) {
                 print("Navigate to topic: \(topicId)")
                 // TODO: Use proper navigation to FeedDetailPage(id: topicId)
-                // For now, open in Safari
-                UIApplication.shared.open(url)
+                // For now, open in SafariView
+                openInSafari(url)
                 return
             }
 
             // Member: /member/username
             if path.contains("/member/"), let username = extractUsername(from: path) {
                 print("Navigate to user: \(username)")
                 // TODO: Use proper navigation to UserDetailPage(userId: username)
-                // For now, open in Safari
-                UIApplication.shared.open(url)
+                // For now, open in SafariView
+                openInSafari(url)
                 return
             }
 
             // Node: /go/nodename
             if path.contains("/go/"), let nodeName = extractNodeName(from: path) {
                 print("Navigate to node: \(nodeName)")
                 // TODO: Use proper navigation to TagDetailPage
-                // For now, open in Safari
-                UIApplication.shared.open(url)
+                // For now, open in SafariView
+                openInSafari(url)
                 return
             }
 
-            // Other V2EX pages - open in Safari
-            UIApplication.shared.open(url)
+            // Other V2EX pages - open in SafariView
+            openInSafari(url)
         } else {
-            // External link - open in Safari
-            UIApplication.shared.open(url)
+            // External link - open in SafariView (stays in app)
+            openInSafari(url)
         }
     }
 
+    private func openInSafari(_ url: URL) {
+        safariURL = url
+        showingSafari = true
+    }
+
     private func extractTopicId(from path: String) -> String? {
         let pattern = "/t/(\\d+)"
         guard let regex = try? NSRegularExpression(pattern: pattern),

V2er/View/FeedDetail/ReplyItemView.swift

@@ -14,6 +14,8 @@ struct ReplyItemView: View {
     var info: FeedDetailInfo.ReplyInfo.Item
     @EnvironmentObject var store: Store
     @Environment(\.colorScheme) var colorScheme
+    @State private var showingSafari = false
+    @State private var safariURL: URL?
 
     var body: some View {
         HStack(alignment: .top) {
@@ -66,6 +68,11 @@ struct ReplyItemView: View {
             }
         }
         .padding(.horizontal, 12)
+        .sheet(isPresented: $showingSafari) {
+            if let url = safariURL {
+                SafariView(url: url)
+            }
+        }
     }
 
     private func handleLinkTap(_ url: URL) {
@@ -78,34 +85,39 @@ struct ReplyItemView: View {
             if path.contains("/t/"), let topicId = extractTopicId(from: path) {
                 print("Navigate to topic: \(topicId)")
                 // TODO: Use proper navigation to FeedDetailPage(id: topicId)
-                UIApplication.shared.open(url)
+                openInSafari(url)
                 return
             }
 
             // Member: /member/username
             if path.contains("/member/"), let username = extractUsername(from: path) {
                 print("Navigate to user: \(username)")
                 // TODO: Use proper navigation to UserDetailPage(userId: username)
-                UIApplication.shared.open(url)
+                openInSafari(url)
                 return
             }
 
             // Node: /go/nodename
             if path.contains("/go/"), let nodeName = extractNodeName(from: path) {
                 print("Navigate to node: \(nodeName)")
                 // TODO: Use proper navigation to TagDetailPage
-                UIApplication.shared.open(url)
+                openInSafari(url)
                 return
             }
 
-            // Other V2EX pages - open in Safari
-            UIApplication.shared.open(url)
+            // Other V2EX pages - open in SafariView
+            openInSafari(url)
         } else {
-            // External link - open in Safari
-            UIApplication.shared.open(url)
+            // External link - open in SafariView (stays in app)
+            openInSafari(url)
         }
     }
 
+    private func openInSafari(_ url: URL) {
+        safariURL = url
+        showingSafari = true
+    }
+
     private func extractTopicId(from path: String) -> String? {
         let pattern = "/t/(\\d+)"
         guard let regex = try? NSRegularExpression(pattern: pattern),