From 3366b16c96e89bfd5bebe041ed6d7f3e9fb8e2b1 Mon Sep 17 00:00:00 2001
From: Shadowfacts <me@shadowfacts.net>
Date: Fri, 22 Feb 2019 10:20:22 -0500
Subject: [PATCH] Add option to attempt to open links in their respective apps

---
 Tusker/Preferences/Preferences.swift          |  3 +-
 .../BehaviorTableViewController.swift         | 15 ++++++--
 .../Preferences/Preferences.storyboard        | 38 ++++++++++++++++++-
 Tusker/TuskerNavigationDelegate.swift         | 10 ++++-
 4 files changed, 58 insertions(+), 8 deletions(-)

diff --git a/Tusker/Preferences/Preferences.swift b/Tusker/Preferences/Preferences.swift
index 08d386e3..f29bcc91 100644
--- a/Tusker/Preferences/Preferences.swift
+++ b/Tusker/Preferences/Preferences.swift
@@ -38,9 +38,10 @@ class Preferences: Codable {
     var avatarStyle = AvatarStyle.roundRect
     var hideCustomEmojiInUsernames = false
     
+    // MARK: - Behavior
     var defaultPostVisibility = Status.Visibility.public
-    
     var automaticallySaveDrafts = true
+    var openLinksInApps = true
     
     // MARK: - Advanced
     var silentActions: [String: Permission] = [:]
diff --git a/Tusker/Screens/Preferences/BehaviorTableViewController.swift b/Tusker/Screens/Preferences/BehaviorTableViewController.swift
index a30ec27a..c72ba06a 100644
--- a/Tusker/Screens/Preferences/BehaviorTableViewController.swift
+++ b/Tusker/Screens/Preferences/BehaviorTableViewController.swift
@@ -12,16 +12,14 @@ class BehaviorTableViewController: UITableViewController {
 
     @IBOutlet weak var defaultPostVisibilityLabel: UILabel!
     @IBOutlet weak var automaticallySaveDraftsSwitch: UISwitch!
+    @IBOutlet weak var openLinksInAppsSwitch: UISwitch!
     
     override func viewDidLoad() {
         super.viewDidLoad()
         
         defaultPostVisibilityLabel.text = Preferences.shared.defaultPostVisibility.displayName
         automaticallySaveDraftsSwitch.setOn(Preferences.shared.automaticallySaveDrafts, animated: false)
-    }
-    
-    @IBAction func automaticallySaveDraftsChanged(_ sender: Any) {
-        Preferences.shared.automaticallySaveDrafts = automaticallySaveDraftsSwitch.isOn
+        openLinksInAppsSwitch.setOn(Preferences.shared.openLinksInApps, animated: false)
     }
     
     override func tableView(_ tableView: UITableView, willSelectRowAt indexPath: IndexPath) -> IndexPath? {
@@ -43,4 +41,13 @@ class BehaviorTableViewController: UITableViewController {
         present(alertController, animated: true)
     }
     
+    @IBAction func automaticallySaveDraftsChanged(_ sender: Any) {
+        Preferences.shared.automaticallySaveDrafts = automaticallySaveDraftsSwitch.isOn
+    }
+    
+    @IBAction func openLinksInAppsChanged(_ sender: Any) {
+        Preferences.shared.openLinksInApps = openLinksInAppsSwitch.isOn
+    }
+    
+    
 }
diff --git a/Tusker/Screens/Preferences/Preferences.storyboard b/Tusker/Screens/Preferences/Preferences.storyboard
index bb6ea324..af8c2d09 100644
--- a/Tusker/Screens/Preferences/Preferences.storyboard
+++ b/Tusker/Screens/Preferences/Preferences.storyboard
@@ -1,10 +1,10 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<document type="com.apple.InterfaceBuilder3.CocoaTouch.Storyboard.XIB" version="3.0" toolsVersion="14460.30.1" targetRuntime="iOS.CocoaTouch" propertyAccessControl="none" useAutolayout="YES" useTraitCollections="YES" useSafeAreas="YES" colorMatched="YES" initialViewController="VJJ-jC-9g8">
+<document type="com.apple.InterfaceBuilder3.CocoaTouch.Storyboard.XIB" version="3.0" toolsVersion="14490.59" targetRuntime="iOS.CocoaTouch" propertyAccessControl="none" useAutolayout="YES" useTraitCollections="YES" useSafeAreas="YES" colorMatched="YES" initialViewController="VJJ-jC-9g8">
     <device id="retina4_7" orientation="portrait">
         <adaptation id="fullscreen"/>
     </device>
     <dependencies>
-        <plugIn identifier="com.apple.InterfaceBuilder.IBCocoaTouchPlugin" version="14460.19.1"/>
+        <plugIn identifier="com.apple.InterfaceBuilder.IBCocoaTouchPlugin" version="14490.45"/>
         <capability name="documents saved in the Xcode 8 format" minToolsVersion="8.0"/>
     </dependencies>
     <scenes>
@@ -292,6 +292,39 @@
                                     </tableViewCell>
                                 </cells>
                             </tableViewSection>
+                            <tableViewSection id="pnH-qu-zTX">
+                                <cells>
+                                    <tableViewCell clipsSubviews="YES" contentMode="scaleToFill" preservesSuperviewLayoutMargins="YES" selectionStyle="default" indentationWidth="10" id="A95-CC-eMR">
+                                        <rect key="frame" x="0.0" y="159" width="375" height="44"/>
+                                        <autoresizingMask key="autoresizingMask"/>
+                                        <tableViewCellContentView key="contentView" opaque="NO" clipsSubviews="YES" multipleTouchEnabled="YES" contentMode="center" preservesSuperviewLayoutMargins="YES" insetsLayoutMarginsFromSafeArea="NO" tableViewCell="A95-CC-eMR" id="0l5-fe-g8R">
+                                            <rect key="frame" x="0.0" y="0.0" width="375" height="43.5"/>
+                                            <autoresizingMask key="autoresizingMask"/>
+                                            <subviews>
+                                                <label opaque="NO" userInteractionEnabled="NO" contentMode="left" horizontalHuggingPriority="251" verticalHuggingPriority="251" text="Open Links in Apps" textAlignment="natural" lineBreakMode="tailTruncation" baselineAdjustment="alignBaselines" adjustsFontSizeToFit="NO" translatesAutoresizingMaskIntoConstraints="NO" id="pRl-1e-i4j">
+                                                    <rect key="frame" x="16" y="11.5" width="148" height="21"/>
+                                                    <fontDescription key="fontDescription" type="system" pointSize="17"/>
+                                                    <nil key="textColor"/>
+                                                    <nil key="highlightedColor"/>
+                                                </label>
+                                                <switch opaque="NO" contentMode="scaleToFill" horizontalHuggingPriority="750" verticalHuggingPriority="750" contentHorizontalAlignment="center" contentVerticalAlignment="center" on="YES" translatesAutoresizingMaskIntoConstraints="NO" id="85C-G0-cRk">
+                                                    <rect key="frame" x="310" y="6.5" width="51" height="31"/>
+                                                    <connections>
+                                                        <action selector="automaticallySaveDraftsChanged:" destination="Nk9-yO-78a" eventType="valueChanged" id="3jl-gq-XEX"/>
+                                                        <action selector="openLinksInAppsChanged:" destination="Nk9-yO-78a" eventType="valueChanged" id="THd-FD-XYN"/>
+                                                    </connections>
+                                                </switch>
+                                            </subviews>
+                                            <constraints>
+                                                <constraint firstItem="85C-G0-cRk" firstAttribute="centerY" secondItem="0l5-fe-g8R" secondAttribute="centerY" id="QuM-P5-nPV"/>
+                                                <constraint firstItem="pRl-1e-i4j" firstAttribute="leading" secondItem="0l5-fe-g8R" secondAttribute="leadingMargin" id="Xqp-IO-EiQ"/>
+                                                <constraint firstAttribute="trailingMargin" secondItem="85C-G0-cRk" secondAttribute="trailing" id="bKz-K4-r5w"/>
+                                                <constraint firstItem="pRl-1e-i4j" firstAttribute="centerY" secondItem="0l5-fe-g8R" secondAttribute="centerY" id="ns6-xZ-q0A"/>
+                                            </constraints>
+                                        </tableViewCellContentView>
+                                    </tableViewCell>
+                                </cells>
+                            </tableViewSection>
                         </sections>
                         <connections>
                             <outlet property="dataSource" destination="Nk9-yO-78a" id="bN3-mz-nCZ"/>
@@ -302,6 +335,7 @@
                     <connections>
                         <outlet property="automaticallySaveDraftsSwitch" destination="HV4-3m-Mal" id="eE1-Q4-za9"/>
                         <outlet property="defaultPostVisibilityLabel" destination="OBq-tA-USG" id="yFX-yQ-dN9"/>
+                        <outlet property="openLinksInAppsSwitch" destination="85C-G0-cRk" id="2lR-Jb-O6K"/>
                     </connections>
                 </tableViewController>
                 <placeholder placeholderIdentifier="IBFirstResponder" id="YN6-Jb-XJJ" userLabel="First Responder" sceneMemberID="firstResponder"/>
diff --git a/Tusker/TuskerNavigationDelegate.swift b/Tusker/TuskerNavigationDelegate.swift
index 13490bcf..0d9a41ea 100644
--- a/Tusker/TuskerNavigationDelegate.swift
+++ b/Tusker/TuskerNavigationDelegate.swift
@@ -60,7 +60,15 @@ extension TuskerNavigationDelegate where Self: UIViewController {
     }
     
     func selected(url: URL) {
-        present(SFSafariViewController(url: url), animated: true)
+        if (Preferences.shared.openLinksInApps) {
+            UIApplication.shared.open(url, options: [.universalLinksOnly: true]) { (success) in
+                if (!success) {
+                    self.present(SFSafariViewController(url: url), animated: true)
+                }
+            }
+        } else {
+            present(SFSafariViewController(url: url), animated: true)
+        }
     }
     
     func selected(status statusID: String) {