Merge pull request #13 from kjantzer/fix/12

Fixing #12 - `shouldHideNavigationOnTap=true` was not working
FolioReader · Jan 1, 2016 · 4aa4d04 · 4aa4d04
2 parents 27c7ec5 + 01ab99f
commit 4aa4d04
Showing 1 changed file with 13 additions and 13 deletions.
diff --git a/Source/FolioReaderPage.swift b/Source/FolioReaderPage.swift
@@ -206,20 +206,20 @@ class FolioReaderPage: UICollectionViewCell, UIWebViewDelegate, UIGestureRecogni
         if FolioReader.sharedInstance.readerCenter.navigationController!.navigationBarHidden {
             let menuIsVisibleRef = menuIsVisible
 
-            if let selected = webView.js("getSelectedText()") {
-                if selected.characters.count == 0 {
-                    let seconds = 0.4
-                    let delay = seconds * Double(NSEC_PER_SEC)  // nanoseconds per seconds
-                    let dispatchTime = dispatch_time(DISPATCH_TIME_NOW, Int64(delay))
+            let selected = webView.js("getSelectedText()")
+
+            if selected == nil || selected!.characters.count == 0 {
+                let seconds = 0.4
+                let delay = seconds * Double(NSEC_PER_SEC)  // nanoseconds per seconds
+                let dispatchTime = dispatch_time(DISPATCH_TIME_NOW, Int64(delay))
+
+                dispatch_after(dispatchTime, dispatch_get_main_queue(), {
 
-                    dispatch_after(dispatchTime, dispatch_get_main_queue(), {
-
-                        if self.shouldShowBar && !menuIsVisibleRef {
-                            FolioReader.sharedInstance.readerCenter.toggleBars()
-                        }
-                        self.shouldShowBar = true
-                    })
-                }
+                    if self.shouldShowBar && !menuIsVisibleRef {
+                        FolioReader.sharedInstance.readerCenter.toggleBars()
+                    }
+                    self.shouldShowBar = true
+                })
             }
         } else if readerConfig.shouldHideNavigationOnTap == true {
             FolioReader.sharedInstance.readerCenter.hideBars()