Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Fix nav bar crash and empty settings screen after logout #2124

Merged
merged 2 commits into from
Jul 29, 2024

Conversation

ademozsayin
Copy link
Contributor

The settings screen is empty after logout as it seen in the video. It may be related dynamic tab view count.

In that video the buggy one:
https://github.com/Dimillian/IceCubesApp/assets/78085814/c7b3c868-90ca-465e-9a84-32a895bc7fce

Here is the resolved one:

fixed.mp4

There was a weird crash sometimes related UINavigationBar such as *** Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: 'Layout requested for visible navigation bar, <SwiftUI.UIKitNavigationBar: 0x109f91080; baseClass = UINavigationBar; frame = (0 24; 933.5 50); opaque = NO; autoresize = W; gestureRecognizers = <NSArray: 0x600000d7cea0>; layer = <CALayer: 0x6000005340a0>> delegate=0x135272800 standardAppearance=0x60000a62f180, when the top item belongs to a different navigation bar. topItem = <UINavigationItem: 0x12f5b3f10> title='@ozsayin' titleView=0x166b8b000 style=navigator leftItemsSupplementBackButton largeTitleDisplayMode=always, navigation bar = <SwiftUI.UIKitNavigationBar: 0x165ef79d0; baseClass = UINavigationBar; frame = (0 24; 533 50); opaque = NO; autoresize = W; userInteractionEnabled = NO; gestureRecognizers = <NSArray: 0x600000e1f060>; layer = <CALayer: 0x60000059acc0>> delegate=0x10acc9600 standardAppearance=0x60000a6cff00, possibly from a client attempt to nest wrapped navigation controllers.'

That is resolved too .

@Dimillian Dimillian merged commit 6766ed4 into Dimillian:main Jul 29, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants