Skip to content

Commit 21e9761

Browse files
author
Momo Ozawa
committed
Refactor: add FAB to parent vc regardless of MSD feature flag
Previously, we were only adding the FAB to the parent vc (MySiteViewController) if the MSD feature flag was enabled.
1 parent 92cfc05 commit 21e9761

File tree

2 files changed

+18
-49
lines changed

2 files changed

+18
-49
lines changed

WordPress/Classes/ViewRelated/Blog/Blog Details/BlogDetailsViewController+FancyAlerts.swift

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -20,13 +20,12 @@ extension BlogDetailsViewController {
2020
if let info = notification.userInfo?[QuickStartTourGuide.notificationElementKey] as? QuickStartTourElement {
2121
switch info {
2222
case .noSuchElement:
23-
if FeatureFlag.mySiteDashboard.enabled,
24-
let parentVC = self?.parent as? MySiteViewController {
25-
parentVC.additionalSafeAreaInsets = .zero
26-
} else {
27-
self?.additionalSafeAreaInsets = .zero
23+
guard let parentVC = self?.parent as? MySiteViewController else {
24+
return
2825
}
2926

27+
parentVC.additionalSafeAreaInsets = .zero
28+
3029
case .siteIcon, .siteTitle:
3130
// handles the padding in case the element is not in the table view
3231
self?.additionalSafeAreaInsets = UIEdgeInsets(top: 0, left: 0, bottom: BlogDetailsViewController.bottomPaddingForQuickStartNotices, right: 0)

WordPress/Classes/ViewRelated/Blog/Blog Details/BlogDetailsViewController.m

Lines changed: 14 additions & 44 deletions
Original file line numberDiff line numberDiff line change
@@ -382,18 +382,12 @@ - (void)viewDidLoad
382382
[self startObservingQuickStart];
383383
[self addMeButtonToNavigationBarWithEmail:self.blog.account.email meScenePresenter:self.meScenePresenter];
384384

385-
if ([Feature enabled:FeatureFlagMySiteDashboard]) {
386-
MySiteViewController *parentVC = (MySiteViewController *)self.parentViewController;
387-
388-
[self.createButtonCoordinator addTo:parentVC.view
389-
trailingAnchor:parentVC.view.safeAreaLayoutGuide.trailingAnchor
390-
bottomAnchor:parentVC.view.safeAreaLayoutGuide.bottomAnchor];
391-
} else {
392-
[self.createButtonCoordinator addTo:self.view
393-
trailingAnchor:self.view.safeAreaLayoutGuide.trailingAnchor
394-
bottomAnchor:self.view.safeAreaLayoutGuide.bottomAnchor];
395-
}
385+
MySiteViewController *parentVC = (MySiteViewController *)self.parentViewController;
396386

387+
[self.createButtonCoordinator addTo:parentVC.view
388+
trailingAnchor:parentVC.view.safeAreaLayoutGuide.trailingAnchor
389+
bottomAnchor:parentVC.view.safeAreaLayoutGuide.bottomAnchor];
390+
397391
}
398392

399393
/// Resizes the `tableHeaderView` as necessary whenever its size changes.
@@ -419,25 +413,13 @@ - (void)viewDidLayoutSubviews {
419413
- (void)viewWillAppear:(BOOL)animated
420414
{
421415
[super viewWillAppear:animated];
416+
417+
MySiteViewController *parentVC = (MySiteViewController *)self.parentViewController;
422418

423419
if ([[QuickStartTourGuide shared] currentElementInt] != NSNotFound) {
424-
425-
if ([Feature enabled:FeatureFlagMySiteDashboard]) {
426-
MySiteViewController *parentVC = (MySiteViewController *)self.parentViewController;
427-
parentVC.additionalSafeAreaInsets = UIEdgeInsetsMake(0, 0, [BlogDetailsViewController bottomPaddingForQuickStartNotices], 0);
428-
} else {
429-
self.additionalSafeAreaInsets = UIEdgeInsetsMake(0, 0, [BlogDetailsViewController bottomPaddingForQuickStartNotices], 0);
430-
}
431-
420+
parentVC.additionalSafeAreaInsets = UIEdgeInsetsMake(0, 0, [BlogDetailsViewController bottomPaddingForQuickStartNotices], 0);
432421
} else {
433-
434-
if ([Feature enabled:FeatureFlagMySiteDashboard]) {
435-
MySiteViewController *parentVC = (MySiteViewController *)self.parentViewController;
436-
parentVC.additionalSafeAreaInsets = UIEdgeInsetsZero;
437-
} else {
438-
self.additionalSafeAreaInsets = UIEdgeInsetsZero;
439-
}
440-
422+
parentVC.additionalSafeAreaInsets = UIEdgeInsetsZero;
441423
}
442424

443425
if (self.splitViewControllerIsHorizontallyCompact) {
@@ -1717,18 +1699,10 @@ - (void)scrollToElement:(QuickStartTourElement) element
17171699
rowCount = 0;
17181700
for (BlogDetailsRow *row in section.rows) {
17191701
if (row.quickStartIdentifier == element) {
1720-
17211702
NSIndexPath *path = [NSIndexPath indexPathForRow:rowCount inSection:sectionCount];
1722-
1723-
if ([Feature enabled:FeatureFlagMySiteDashboard]) {
1724-
parentVC.additionalSafeAreaInsets = UIEdgeInsetsMake(0, 0, [BlogDetailsViewController bottomPaddingForQuickStartNotices], 0);
1725-
UITableViewCell *cell = [self.tableView cellForRowAtIndexPath:path];
1726-
[parentVC.scrollView scrollToView:cell animated:true];
1727-
} else {
1728-
self.additionalSafeAreaInsets = UIEdgeInsetsMake(0, 0, [BlogDetailsViewController bottomPaddingForQuickStartNotices], 0);
1729-
[self.tableView scrollToRowAtIndexPath:path atScrollPosition:UITableViewScrollPositionTop animated:true];
1730-
}
1731-
1703+
parentVC.additionalSafeAreaInsets = UIEdgeInsetsMake(0, 0, [BlogDetailsViewController bottomPaddingForQuickStartNotices], 0);
1704+
UITableViewCell *cell = [self.tableView cellForRowAtIndexPath:path];
1705+
[parentVC.scrollView scrollToView:cell animated:true];
17321706
}
17331707
rowCount++;
17341708
}
@@ -1948,12 +1922,8 @@ - (void)showViewSiteFromSource:(BlogDetailsNavigationSource)source
19481922
[[QuickStartTourGuide shared] completeViewSiteTourForBlog:self.blog];
19491923
}
19501924

1951-
if ([Feature enabled:FeatureFlagMySiteDashboard]) {
1952-
MySiteViewController *parentVC = (MySiteViewController *)self.parentViewController;
1953-
parentVC.additionalSafeAreaInsets = UIEdgeInsetsZero;
1954-
} else {
1955-
self.additionalSafeAreaInsets = UIEdgeInsetsZero;
1956-
}
1925+
MySiteViewController *parentVC = (MySiteViewController *)self.parentViewController;
1926+
parentVC.additionalSafeAreaInsets = UIEdgeInsetsZero;
19571927
}
19581928

19591929
- (void)showViewAdmin

0 commit comments

Comments
 (0)