Merge pull request #15222 from wordpress-mobile/fix/8224-add_edit_pag…

…e_option Pages: add Edit to options menu
wordpress-mobile · Nov 5, 2020 · 6a945ed · 6a945ed
2 parents e137b36 + 1a3f4df
commit 6a945ed
Showing 1 changed file with 17 additions and 0 deletions.
diff --git a/WordPress/Classes/ViewRelated/Pages/PageListViewController.swift b/WordPress/Classes/ViewRelated/Pages/PageListViewController.swift
@@ -610,6 +610,8 @@ class PageListViewController: AbstractPostListViewController, UIViewControllerRe
                     strongSelf.retryPage(page)
                 })
             } else {
+                addEditAction(to: alertController, for: page)
+
                 alertController.addActionWithTitle(viewButtonTitle, style: .default, handler: { [weak self] (action) in
                     guard let strongSelf = self,
                         let page = strongSelf.pageForObjectID(objectID) else {
@@ -652,6 +654,8 @@ class PageListViewController: AbstractPostListViewController, UIViewControllerRe
                     strongSelf.retryPage(page)
                 })
             } else {
+                addEditAction(to: alertController, for: page)
+
                 alertController.addActionWithTitle(viewButtonTitle, style: .default, handler: { [weak self] (action) in
                     guard let strongSelf = self,
                         let page = strongSelf.pageForObjectID(objectID) else {
@@ -695,6 +699,19 @@ class PageListViewController: AbstractPostListViewController, UIViewControllerRe
         }
     }
 
+    private func addEditAction(to controller: UIAlertController, for page: AbstractPost) {
+        if page.status == .trash {
+            return
+        }
+
+        let buttonTitle = NSLocalizedString("Edit", comment: "Label for a button that opens the Edit Page view controller")
+        controller.addActionWithTitle(buttonTitle, style: .default, handler: { [weak self] _ in
+            if let page = self?.pageForObjectID(page.objectID) {
+                self?.editPage(page)
+            }
+        })
+    }
+
     private func addSetParentAction(to controller: UIAlertController, for page: AbstractPost, at index: IndexPath?) {
         /// This button is disabled for trashed pages
         //