Skip to content

Commit

Permalink
Update: move popover trigger to MySiteVC to enable the popover on W…
Browse files Browse the repository at this point in the history
…ordPress
  • Loading branch information
hassaanelgarem committed Jul 12, 2023
1 parent c1665e0 commit 9af977c
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,6 @@ final class BlogDashboardViewController: UIViewController {
BlogDashboardViewModel(viewController: self, blog: blog)
}()

private var complianceCoordinator: CompliancePopoverCoordinator?

lazy var collectionView: DynamicHeightCollectionView = {
let collectionView = DynamicHeightCollectionView(frame: .zero, collectionViewLayout: createLayout())
Expand Down Expand Up @@ -81,9 +80,6 @@ final class BlogDashboardViewController: UIViewController {
startAlertTimer()

WPAnalytics.track(.mySiteDashboardShown)

complianceCoordinator = CompliancePopoverCoordinator(viewController: self)
complianceCoordinator?.presentIfNeeded()
}

override func viewWillDisappear(_ animated: Bool) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -91,6 +91,7 @@ class MySiteViewController: UIViewController, NoResultsViewHost {
var willDisplayPostSignupFlow: Bool = false

private var createButtonCoordinator: CreateButtonCoordinator?
private var complianceCoordinator: CompliancePopoverCoordinator?

private let meScenePresenter: ScenePresenter
private let blogService: BlogService
Expand Down Expand Up @@ -208,6 +209,9 @@ class MySiteViewController: UIViewController, NoResultsViewHost {

createFABIfNeeded()
fetchPrompt(for: blog)

complianceCoordinator = CompliancePopoverCoordinator(viewController: self)
complianceCoordinator?.presentIfNeeded()
}

override func viewDidLayoutSubviews() {
Expand Down

0 comments on commit 9af977c

Please sign in to comment.