Merge branch 'dev' into sjmarf/favorite-fix

mlemgroup · Jan 16, 2024 · 6c7f273 · 6c7f273
2 parents 7f875c8 + e4ca276
commit 6c7f273
Show file tree

Hide file tree

Showing 4 changed files with 20 additions and 10 deletions.
diff --git a/Mlem/Enums/Settings/PostSortType.swift b/Mlem/Enums/Settings/PostSortType.swift
@@ -85,9 +85,9 @@ enum PostSortType: String, Codable, CaseIterable, Identifiable {
         case .topHour:
             return "Top of the last hour"
         case .topSixHour:
-            return "Top of the last six hours"
+            return "Top of the last 6 hours"
         case .topTwelveHour:
-            return "Top of the last twelve hours"
+            return "Top of the last 12 hours"
         case .topDay:
             return "Top of today"
         case .topWeek:
@@ -164,9 +164,9 @@ extension PostSortType: SettingsOptions {
         case .topHour:
             return "Hour"
         case .topSixHour:
-            return "Six Hours"
+            return "6 Hours"
         case .topTwelveHour:
-            return "Twelve Hours"
+            return "12 Hours"
         case .topDay:
             return "Day"
         case .topWeek:

diff --git a/Mlem/Logic/URLHandler.swift b/Mlem/Logic/URLHandler.swift
@@ -30,10 +30,14 @@ class URLHandler {
         guard let scheme = url.scheme, scheme.hasPrefix("http") else {
             return .init(result: .systemAction, action: .error("This type of link is not currently supported 😞"))
         }
-
-        Task { @MainActor in
-            let viewController = SFSafariViewController(url: url, configuration: .default)
-            UIApplication.shared.firstKeyWindow?.rootViewController?.topMostViewController().present(viewController, animated: true)
+        let openLinksInBrowser = UserDefaults.standard.bool(forKey: "openLinksInBrowser")
+        if openLinksInBrowser {
+            UIApplication.shared.open(url)
+        } else {
+            Task { @MainActor in
+                let viewController = SFSafariViewController(url: url, configuration: .default)
+                UIApplication.shared.firstKeyWindow?.rootViewController?.topMostViewController().present(viewController, animated: true)
+            }
         }
 
         return .init(result: .handled, action: nil)

diff --git a/Mlem/Views/Shared/Comments/Comment Item Logic.swift b/Mlem/Views/Shared/Comments/Comment Item Logic.swift
@@ -151,7 +151,7 @@ extension CommentItem {
 
         // upvote
         let (upvoteText, upvoteImg) = hierarchicalComment.commentView.myVote == .upvote ?
-            ("Undo upvote", Icons.upvoteSquareFill) :
+            ("Undo Upvote", Icons.upvoteSquareFill) :
             ("Upvote", Icons.upvoteSquare)
         ret.append(MenuFunction.standardMenuFunction(
             text: upvoteText,
@@ -166,7 +166,7 @@ extension CommentItem {
 
         // downvote
         let (downvoteText, downvoteImg) = hierarchicalComment.commentView.myVote == .downvote ?
-            ("Undo downvote", Icons.downvoteSquareFill) :
+            ("Undo Downvote", Icons.downvoteSquareFill) :
             ("Downvote", Icons.downvoteSquare)
         ret.append(MenuFunction.standardMenuFunction(
             text: downvoteText,

diff --git a/Mlem/Views/Tabs/Settings/Components/Views/General/GeneralSettingsView.swift b/Mlem/Views/Tabs/Settings/Components/Views/General/GeneralSettingsView.swift
@@ -22,6 +22,7 @@ struct GeneralSettingsView: View {
     @AppStorage("upvoteOnSave") var upvoteOnSave: Bool = false
 
     @AppStorage("showSettingsIcons") var showSettingsIcons: Bool = false
+    @AppStorage("openLinksInBrowser") var openLinksInBrowser: Bool = false
 
     @EnvironmentObject var appState: AppState
 
@@ -30,6 +31,11 @@ struct GeneralSettingsView: View {
     var body: some View {
         List {
             Section {
+                SwitchableSettingsItem(
+                    settingPictureSystemName: Icons.browser,
+                    settingName: "Open Links in Browser",
+                    isTicked: $openLinksInBrowser
+                )
                 SelectableSettingsItem(
                     settingIconSystemName: Icons.haptics,
                     settingName: "Haptic Level",