diff --git a/iOS/AppDefaults.swift b/iOS/AppDefaults.swift index 8fc8e11ae..35ffe3df4 100644 --- a/iOS/AppDefaults.swift +++ b/iOS/AppDefaults.swift @@ -169,13 +169,17 @@ final class AppDefaults { var articleFullscreenEnabled: Bool { get { - return AppDefaults.bool(for: Key.articleFullscreenEnabled) + return articleFullscreenAvailable && AppDefaults.bool(for: Key.articleFullscreenEnabled) } set { AppDefaults.setBool(for: Key.articleFullscreenEnabled, newValue) } } + var logicalArticleFullscreenEnabled: Bool { + articleFullscreenAvailable && articleFullscreenEnabled + } + var confirmMarkAllAsRead: Bool { get { return AppDefaults.bool(for: Key.confirmMarkAllAsRead) diff --git a/iOS/Article/ArticleViewController.swift b/iOS/Article/ArticleViewController.swift index e84167bb4..ae6e73837 100644 --- a/iOS/Article/ArticleViewController.swift +++ b/iOS/Article/ArticleViewController.swift @@ -147,7 +147,7 @@ class ArticleViewController: UIViewController { articleExtractorButton.buttonState = controller.articleExtractorButtonState self.pageViewController.setViewControllers([controller], direction: .forward, animated: false, completion: nil) - if AppDefaults.shared.articleFullscreenEnabled { + if AppDefaults.shared.logicalArticleFullscreenEnabled { controller.hideBars() } @@ -163,7 +163,7 @@ class ArticleViewController: UIViewController { } override func viewWillAppear(_ animated: Bool) { - let hideToolbars = AppDefaults.shared.articleFullscreenEnabled + let hideToolbars = AppDefaults.shared.logicalArticleFullscreenEnabled if hideToolbars { currentWebViewController?.hideBars() } else { @@ -255,7 +255,7 @@ class ArticleViewController: UIViewController { @objc func willEnterForeground(_ note: Notification) { // The toolbar will come back on you if you don't hide it again - if AppDefaults.shared.articleFullscreenEnabled { + if AppDefaults.shared.logicalArticleFullscreenEnabled { currentWebViewController?.hideBars() } } diff --git a/iOS/Article/WebViewController.swift b/iOS/Article/WebViewController.swift index 7b9702be4..c5048bf09 100644 --- a/iOS/Article/WebViewController.swift +++ b/iOS/Article/WebViewController.swift @@ -675,7 +675,7 @@ private extension WebViewController { topShowBarsView.translatesAutoresizingMaskIntoConstraints = false view.addSubview(topShowBarsView) - if AppDefaults.shared.articleFullscreenEnabled { + if AppDefaults.shared.logicalArticleFullscreenEnabled { topShowBarsViewConstraint = view.topAnchor.constraint(equalTo: topShowBarsView.bottomAnchor, constant: -44.0) } else { topShowBarsViewConstraint = view.topAnchor.constraint(equalTo: topShowBarsView.bottomAnchor, constant: 0.0) @@ -695,7 +695,7 @@ private extension WebViewController { topShowBarsView.backgroundColor = .clear bottomShowBarsView.translatesAutoresizingMaskIntoConstraints = false view.addSubview(bottomShowBarsView) - if AppDefaults.shared.articleFullscreenEnabled { + if AppDefaults.shared.logicalArticleFullscreenEnabled { bottomShowBarsViewConstraint = view.bottomAnchor.constraint(equalTo: bottomShowBarsView.topAnchor, constant: 44.0) } else { bottomShowBarsViewConstraint = view.bottomAnchor.constraint(equalTo: bottomShowBarsView.topAnchor, constant: 0.0)