Skip to content

Commit edfa066

Browse files
author
Momo Ozawa
committed
Fix: hide site menu spotlight if element isn’t site menu
1 parent 683485f commit edfa066

File tree

1 file changed

+3
-5
lines changed

1 file changed

+3
-5
lines changed

WordPress/Classes/ViewRelated/Blog/My Site/MySiteViewController+QuickStart.swift

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -8,19 +8,17 @@ extension MySiteViewController {
88
if let info = notification.userInfo,
99
let element = info[QuickStartTourGuide.notificationElementKey] as? QuickStartTourElement {
1010

11+
self?.siteMenuSpotlightIsShown = element == .siteMenu
12+
1113
switch element {
1214
case .noSuchElement, .newpost:
1315
self?.additionalSafeAreaInsets = .zero
1416

1517
case .siteIcon, .siteTitle, .viewSite:
1618
self?.scrollView.scrollToTop(animated: true)
17-
self?.additionalSafeAreaInsets = Constants.quickStartNoticeInsets
18-
19-
case .siteMenu:
20-
self?.siteMenuSpotlightIsShown = true
2119
fallthrough
2220

23-
case .pages, .sharing, .stats, .readerTab, .notifications:
21+
case .siteMenu, .pages, .sharing, .stats, .readerTab, .notifications:
2422
self?.additionalSafeAreaInsets = Constants.quickStartNoticeInsets
2523

2624
default:

0 commit comments

Comments
 (0)