Fix #4717 - Provide feedback when pinning a site (#6785)

* Fix #4717 - Provide feedback when pinning a site from inside the Library panel Display toast notification on the Library panel Display toast notification on the Bookmarks panel Display toast notification on the Website tab * Change to remove Co-authored-by: Vlad Dramaretskyi <vlad@Vlads-MacBook-Pro.local>
mozilla-mobile · Jun 24, 2020 · c51aab1 · c51aab1
1 parent 7ed0ddf
commit c51aab1
Show file tree

Hide file tree

Showing 4 changed files with 24 additions and 5 deletions.
diff --git a/Client/Frontend/Library/BookmarksPanel.swift b/Client/Frontend/Library/BookmarksPanel.swift
@@ -527,7 +527,11 @@ extension BookmarksPanel: LibraryPanelContextMenu {
         }
 
         let pinTopSite = PhotonActionSheetItem(title: Strings.PinTopsiteActionTitle, iconString: "action_pin", handler: { _, _ in
-            _ = self.profile.history.addPinnedTopSite(site)
+            _ = self.profile.history.addPinnedTopSite(site).uponQueue(.main) { result in
+                if result.isSuccess {
+                    SimpleToast().showAlertWithText(Strings.AppMenuAddPinToTopSitesConfirmMessage, bottomContainer: self.view)
+                }
+            }
         })
         actions.append(pinTopSite)
 

diff --git a/Client/Frontend/Library/HistoryPanel.swift b/Client/Frontend/Library/HistoryPanel.swift
@@ -232,7 +232,11 @@ class HistoryPanel: SiteTableViewController, LibraryPanel {
     }
 
     func pinToTopSites(_ site: Site) {
-        _ = profile.history.addPinnedTopSite(site).value
+        _ = profile.history.addPinnedTopSite(site).uponQueue(.main) { result in
+            if result.isSuccess {
+                SimpleToast().showAlertWithText(Strings.AppMenuAddPinToTopSitesConfirmMessage, bottomContainer: self.view)
+            }
+        }
     }
 
     func navigateToRecentlyClosed() {

diff --git a/Client/Frontend/Strings.swift b/Client/Frontend/Strings.swift
@@ -536,6 +536,8 @@ extension Strings {
     public static let AppMenuAddBookmarkConfirmMessage = NSLocalizedString("Menu.AddBookmark.Confirm", value: "Bookmark Added", comment: "Toast displayed to the user after a bookmark has been added.")
     public static let AppMenuTabSentConfirmMessage = NSLocalizedString("Menu.TabSent.Confirm", value: "Tab Sent", comment: "Toast displayed to the user after a tab has been sent successfully.")
     public static let AppMenuRemoveBookmarkConfirmMessage = NSLocalizedString("Menu.RemoveBookmark.Confirm", value: "Bookmark Removed", comment: "Toast displayed to the user after a bookmark has been removed.")
+    public static let AppMenuAddPinToTopSitesConfirmMessage = NSLocalizedString("Menu.AddPin.Confirm", value: "Pinned To Top Sites", comment: "Toast displayed to the user after adding the item to the Top Sites.")
+    public static let AppMenuRemovePinFromTopSitesConfirmMessage = NSLocalizedString("Menu.RemovePin.Confirm", value: "Removed From Top Sites", comment: "Toast displayed to the user after removing the item from the Top Sites.")
     public static let AppMenuAddToReadingListConfirmMessage = NSLocalizedString("Menu.AddToReadingList.Confirm", value: "Added To Reading List", comment: "Toast displayed to the user after adding the item to their reading list.")
     public static let SendToDeviceTitle = NSLocalizedString("Send to Device", tableName: "3DTouchActions", comment: "Label for preview action on Tab Tray Tab to send the current tab to another device")
     public static let PageActionMenuTitle = NSLocalizedString("Menu.PageActions.Title", value: "Page Actions", comment: "Label for title in page action menu.")

diff --git a/Client/Frontend/Widgets/PhotonActionSheet/PageActionMenu.swift b/Client/Frontend/Widgets/PhotonActionSheet/PageActionMenu.swift
@@ -11,6 +11,8 @@ enum ButtonToastAction {
     case bookmarkPage
     case removeBookmark
     case copyUrl
+    case pinPage
+    case removePinPage
 }
 
 extension PhotonActionSheetProtocol {
@@ -97,9 +99,12 @@ extension PhotonActionSheetProtocol {
                 guard let site = val.successValue?.asArray().first?.flatMap({ $0 }) else {
                     return succeed()
                 }
-
                 return self.profile.history.addPinnedTopSite(site)
-            }.uponQueue(.main) { _ in }
+            }.uponQueue(.main) { result in
+                if result.isSuccess {
+                    success(Strings.AppMenuAddPinToTopSitesConfirmMessage, .pinPage)
+                }
+            }
         }
 
         let removeTopSitesPin = PhotonActionSheetItem(title: Strings.RemovePinTopsiteActionTitle, iconString: "action_unpin") { _, _ in
@@ -111,7 +116,11 @@ extension PhotonActionSheetProtocol {
                 }
 
                 return self.profile.history.removeFromPinnedTopSites(site)
-            }.uponQueue(.main) { _ in }
+            }.uponQueue(.main) { result in
+                if result.isSuccess {
+                    success(Strings.AppMenuRemovePinFromTopSitesConfirmMessage, .removePinPage)
+                }
+            }
         }
 
         let sendToDevice = PhotonActionSheetItem(title: Strings.SendToDeviceTitle, iconString: "menu-Send-to-Device") { _, _ in