Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[MBL-1738] iPadOS 18 - Navbar overlay issue in "Follow Friends" screen #2164

Merged
merged 1 commit into from
Oct 1, 2024

Conversation

jovaniks
Copy link
Contributor

@jovaniks jovaniks commented Oct 1, 2024

📲 What

  • Moved the call to setNavigationBarHidden from viewWillAppear to viewDidAppear in the "Follow Friends" screen.

🤔 Why

The change was needed to fix an issue where the UINavigationBar was overlaying the content of the "Follow Friends" screen when running the app on a device with iPadOS 18 and Xcode 16.

This issue appears to be caused by layout changes introduced in iPadOS 18, where the tab bar is now at the top of the screen. By calling setNavigationBarHidden in viewDidAppear instead of viewWillAppear, we ensure that the layout has been fully constructed, which prevents the UINavigationBar from incorrectly overlaying the content.

🛠 How

The setNavigationBarHidden function was moved from viewWillAppear to viewDidAppear so that the layout is fully built before manipulating the visibility of the navigation bar. This change resolves the layout issue observed in iPadOS 18.

👀 See

Jira issue

Before 🐛 After 🦋
image image

✅ Acceptance criteria

  • Ensure the UINavigationBar no longer overlays the "Follow Friends" screen on iPadOS 18.

…etting it up in the viewDidAppear instead of the viewWillAppear
Copy link
Contributor

@ifosli ifosli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@jovaniks jovaniks merged commit 1c07bba into main Oct 1, 2024
4 checks passed
@jovaniks jovaniks deleted the jovaniks/mbl-1738/follow-friends-bugfix branch October 1, 2024 22:06
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants