Reader: Add Favorites (#23815)

* Add support for adding favorites * Add analytics * Save in background * Fix
wordpress-mobile · Nov 19, 2024 · d61e178 · d61e178
1 parent 7ab94fd
commit d61e178
Show file tree

Hide file tree

Showing 5 changed files with 60 additions and 15 deletions.
diff --git a/WordPress/Classes/Models/ReaderAbstractTopic.swift b/WordPress/Classes/Models/ReaderAbstractTopic.swift
@@ -11,6 +11,7 @@ import CoreData
     @NSManaged open var following: Bool
     @NSManaged open var lastSynced: Date?
     @NSManaged open var path: String
+    /// Repurposed for "isFavorite".
     @NSManaged open var showInMenu: Bool
     @NSManaged open var title: String
     @NSManaged open var type: String

diff --git a/WordPress/Classes/Services/ReaderTopicService.m b/WordPress/Classes/Services/ReaderTopicService.m
@@ -735,7 +735,6 @@ - (ReaderSiteTopic *)siteTopicForRemoteSiteInfo:(RemoteReaderSiteInfo *)siteInfo
     topic.organizationID = [siteInfo.organizationID integerValue];
     topic.path = siteInfo.postsEndpoint;
     topic.postCount = siteInfo.postCount;
-    topic.showInMenu = NO;
     topic.siteBlavatar = siteInfo.siteBlavatar;
     topic.siteDescription = siteInfo.siteDescription;
     topic.siteID = siteInfo.siteID;

diff --git a/WordPress/Classes/Utility/Analytics/WPAnalyticsEvent.swift b/WordPress/Classes/Utility/Analytics/WPAnalyticsEvent.swift
@@ -116,6 +116,7 @@ import Foundation
     case readerCommentTextHighlighted
     case readerCommentTextCopied
     case readerPostContextMenuButtonTapped
+    case readerAddSiteToFavoritesTapped
 
     // Stats - Empty Stats nudges
     case statsPublicizeNudgeShown
@@ -816,6 +817,8 @@ import Foundation
             return "reader_comment_text_copied"
         case .readerPostContextMenuButtonTapped:
             return "reader_post_context_menu_button_tapped"
+        case .readerAddSiteToFavoritesTapped:
+            return "reader_add_site_to_favorites_tapped"
 
         // Stats - Empty Stats nudges
         case .statsPublicizeNudgeShown:

diff --git a/WordPress/Classes/ViewRelated/Reader/Sidebar/ReaderSidebarSubscriptionsSection.swift b/WordPress/Classes/ViewRelated/Reader/Sidebar/ReaderSidebarSubscriptionsSection.swift
@@ -13,19 +13,8 @@ struct ReaderSidebarSubscriptionsSection: View {
     private var subscriptions: FetchedResults<ReaderSiteTopic>
 
     var body: some View {
-        ForEach(subscriptions, id: \.self) { site in
-            Label {
-                Text(site.title)
-            } icon: {
-                ReaderSiteIconView(site: site, size: .small)
-            }
-            .lineLimit(1)
-            .tag(ReaderSidebarItem.subscription(TaggedManagedObjectID(site)))
-            .swipeActions(edge: .trailing) {
-                Button(SharedStrings.Reader.unfollow, role: .destructive) {
-                    ReaderSubscriptionHelper().unfollow(site)
-                }.tint(.red)
-            }
+        ForEach(subscriptions, id: \.self) {
+            ReaderSidebarSubscriptionCell(site: $0)
         }
         .onDelete(perform: delete)
     }
@@ -37,3 +26,42 @@ struct ReaderSidebarSubscriptionsSection: View {
         }
     }
 }
+
+struct ReaderSidebarSubscriptionCell: View {
+    @ObservedObject var site: ReaderSiteTopic
+    @Environment(\.editMode) var editMode
+
+    var body: some View {
+        HStack {
+            Label {
+                Text(site.title)
+            } icon: {
+                ReaderSiteIconView(site: site, size: .small)
+            }
+            if editMode?.wrappedValue.isEditing == true {
+                Spacer()
+                Button {
+                    if !site.showInMenu {
+                        WPAnalytics.track(.readerAddSiteToFavoritesTapped)
+                    }
+
+                    let siteObjectID = TaggedManagedObjectID(site)
+                    ContextManager.shared.performAndSave({ managedObjectContext in
+                        let site = try managedObjectContext.existingObject(with: siteObjectID)
+                        site.showInMenu.toggle()
+                    }, completion: nil, on: DispatchQueue.main)
+                } label: {
+                    Image(systemName: site.showInMenu ? "star.fill" : "star")
+                        .foregroundStyle(site.showInMenu ? .pink : .secondary)
+                }.buttonStyle(.plain)
+            }
+        }
+        .lineLimit(1)
+        .tag(ReaderSidebarItem.subscription(TaggedManagedObjectID(site)))
+        .swipeActions(edge: .trailing) {
+            Button(SharedStrings.Reader.unfollow, role: .destructive) {
+                ReaderSubscriptionHelper().unfollow(site)
+            }.tint(.red)
+        }
+    }
+}
diff --git a/WordPress/Classes/ViewRelated/Reader/Sidebar/ReaderSidebarViewController.swift b/WordPress/Classes/ViewRelated/Reader/Sidebar/ReaderSidebarViewController.swift
@@ -50,13 +50,20 @@ private struct ReaderSidebarView: View {
     @ObservedObject var viewModel: ReaderSidebarViewModel
 
     @AppStorage("reader_sidebar_organization_expanded") var isSectionOrganizationExpanded = true
+    @AppStorage("reader_sidebar_favorites_expanded") var isSectionFavoritesExpanded = true
     @AppStorage("reader_sidebar_subscriptions_expanded") var isSectionSubscriptionsExpanded = true
     @AppStorage("reader_sidebar_lists_expanded") var isSectionListsExpanded = true
     @AppStorage("reader_sidebar_tags_expanded") var isSectionTagsExpanded = true
 
     @FetchRequest(sortDescriptors: [SortDescriptor(\.title, order: .forward)])
     private var teams: FetchedResults<ReaderTeamTopic>
 
+    @FetchRequest(
+        sortDescriptors: [SortDescriptor(\.title, order: .forward)],
+        predicate: NSPredicate(format: "following = YES AND showInMenu = YES")
+    )
+    private var favorites: FetchedResults<ReaderSiteTopic>
+
     @Environment(\.layoutDirection) var layoutDirection
     @Environment(\.editMode) var editMode
 
@@ -101,7 +108,13 @@ private struct ReaderSidebarView: View {
                     .withDisabledSelection(isEditing)
             }
         }
-
+        if !favorites.isEmpty || isEditing {
+            makeSection(Strings.favorites, isExpanded: $isSectionFavoritesExpanded) {
+                ForEach(favorites, id: \.self) {
+                    ReaderSidebarSubscriptionCell(site: $0)
+                }
+            }
+        }
         if !teams.isEmpty {
             makeSection(Strings.organization, isExpanded: $isSectionOrganizationExpanded) {
                 ReaderSidebarOrganizationSection(viewModel: viewModel, teams: teams)
@@ -213,6 +226,7 @@ private extension View {
 private struct Strings {
     static let reader = SharedStrings.Reader.title
     static let subscriptions = NSLocalizedString("reader.sidebar.section.subscriptions.title", value: "Subscriptions", comment: "Reader sidebar section title")
+    static let favorites = NSLocalizedString("reader.sidebar.section.favorites.title", value: "Favorites", comment: "Reader sidebar section title")
     static let lists = NSLocalizedString("reader.sidebar.section.lists.title", value: "Lists", comment: "Reader sidebar section title")
     static let tags = NSLocalizedString("reader.sidebar.section.tags.title", value: "Tags", comment: "Reader sidebar section title")
     static let organization = NSLocalizedString("reader.sidebar.section.organization.title", value: "Organization", comment: "Reader sidebar section title")