Fix #8526: Fix the issue of the share button disappearing

ns-vasilev · Dec 17, 2023 · fd063e3 · fd063e3
1 parent a9f0557
commit fd063e3
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/Sources/Brave/Frontend/Browser/BrowserViewController.swift b/Sources/Brave/Frontend/Browser/BrowserViewController.swift
@@ -651,7 +651,7 @@ public class BrowserViewController: UIViewController {
       let webView = tab.webView {
       updateURLBar()
       navigationToolbar.updateBackStatus(webView.canGoBack)
-      navigationToolbar.updateForwardStatus(webView.canGoForward)
+      updateForwardStatusIfNeeded(webView: webView)
       topToolbar.locationView.loading = tab.loading
     }
 
@@ -1792,7 +1792,7 @@ public class BrowserViewController: UIViewController {
       guard tab === tabManager.selectedTab, let canGoForward = change?[.newKey] as? Bool else {
         break
       }
-
+        
       navigationToolbar.updateForwardStatus(canGoForward)
     case .hasOnlySecureContent:
       guard let tab = tabManager[webView] else {
@@ -1913,6 +1913,14 @@ public class BrowserViewController: UIViewController {
     }
   }
 
+  func updateForwardStatusIfNeeded(webView: WKWebView) {
+    if let forwardListItem = webView.backForwardList.forwardList.first, forwardListItem.url.isReaderModeURL {
+      navigationToolbar.updateForwardStatus(false)
+    } else {
+      navigationToolbar.updateForwardStatus(webView.canGoForward)
+    }
+  }
+
   func updateUIForReaderHomeStateForTab(_ tab: Tab) {
     updateURLBar()
     toolbarVisibilityViewModel.toolbarState = .expanded

diff --git a/...e/Frontend/Browser/BrowserViewController/BrowserViewController+WKNavigationDelegate.swift b/...e/Frontend/Browser/BrowserViewController/BrowserViewController+WKNavigationDelegate.swift
@@ -666,6 +666,8 @@ extension BrowserViewController: WKNavigationDelegate {
     if tabManager.selectedTab === tab {
       updateUIForReaderHomeStateForTab(tab)
     }
+
+    updateForwardStatusIfNeeded(webView: webView)
   }
 
   public func webView(_ webView: WKWebView, didFinish navigation: WKNavigation!) {