Skip to content

Commit

Permalink
Fixes the issue of navigation bar hiding on tab switch resulting in c…
Browse files Browse the repository at this point in the history
…ontent visiblity under navigatoin bar in Search tab.
  • Loading branch information
chandra9302 committed Feb 13, 2025
1 parent 092a8a6 commit e30d9f5
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion Wikipedia/Code/SearchViewController.swift
Original file line number Diff line number Diff line change
Expand Up @@ -84,6 +84,8 @@ class SearchViewController: ArticleCollectionViewController, WMFNavigationBarCon

private let source: EventLoggingSource

private var presentingSearchResults: Bool = false

// MARK: - Funcs

@objc required init(source: EventLoggingSource) {
Expand Down Expand Up @@ -196,7 +198,7 @@ class SearchViewController: ArticleCollectionViewController, WMFNavigationBarCon

let searchBarConfig = WMFNavigationBarSearchConfig(searchResultsController: nil, searchControllerDelegate: self, searchResultsUpdater: self, searchBarDelegate: self, searchBarPlaceholder: WMFLocalizedString("search-field-placeholder-text", value: "Search Wikipedia", comment: "Search field placeholder text"), showsScopeBar: false, scopeButtonTitles: nil)

configureNavigationBar(titleConfig: titleConfig, closeButtonConfig: nil, profileButtonConfig: profileButtonConfig, searchBarConfig: searchBarConfig, hideNavigationBarOnScroll: true)
configureNavigationBar(titleConfig: titleConfig, closeButtonConfig: nil, profileButtonConfig: profileButtonConfig, searchBarConfig: searchBarConfig, hideNavigationBarOnScroll: !presentingSearchResults)
}

private func updateProfileButton() {
Expand Down Expand Up @@ -663,6 +665,7 @@ extension SearchViewController: UISearchControllerDelegate {
func willPresentSearchController(_ searchController: UISearchController) {
needsAnimateLanguageBarMovement = true
navigationController?.hidesBarsOnSwipe = false
presentingSearchResults = true
}

func willDismissSearchController(_ searchController: UISearchController) {
Expand All @@ -672,6 +675,7 @@ extension SearchViewController: UISearchControllerDelegate {
func didDismissSearchController(_ searchController: UISearchController) {
needsAnimateLanguageBarMovement = false
navigationController?.hidesBarsOnSwipe = true
presentingSearchResults = false
}
}

Expand Down

0 comments on commit e30d9f5

Please sign in to comment.