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

SwiftUI 4 NavigationStack naming conflict #76

Open
razvanrujoiu opened this issue Jun 22, 2022 · 6 comments
Open

SwiftUI 4 NavigationStack naming conflict #76

razvanrujoiu opened this issue Jun 22, 2022 · 6 comments

Comments

@razvanrujoiu
Copy link

razvanrujoiu commented Jun 22, 2022

I'm using MacOS Ventura Beta with SwiftUI 4. The problem is that Apple introduced NavigationStack in SwiftUI 4 and it conflicts with the NavigationStack<T: NavigationCoordinatable> from stinsen library. Hence i'm getting the following error:
Generic type 'NavigationStack' specialized with too few type parameters (got 1, but expected 2)
As a quick fix i think you could rename NavigationStack to SNavigationStack or StinsenNavigationStack

@LePips
Copy link
Collaborator

LePips commented Jun 28, 2022

This is simply fixed in #75

@rundfunk47
Copy link
Owner

As @LePips says, this is temporary fixed with a namespace. However, this item should still be open - a long term solution should be to rename it completely. Any suggestions? I'm not a huge fan of adding a "Stinsen"-prefix...

Maybe just Stack would be fine?

@LePips
Copy link
Collaborator

LePips commented Jul 10, 2022

I think Stack would be problematic both in usage and in context. It would conflict with a basic Stack if a developer ever had one since NavigationStack is already public. For context, it's usually better to have the name indicate its domain or purpose, especially since it's for navigation.

@razvanrujoiu
Copy link
Author

I totally agree with @LePips , naming it just Stack would certainly create confusion

@LePips
Copy link
Collaborator

LePips commented Sep 12, 2022

With the release of Xcode 14 at time of writing, this is become an immediate "issue". I'm not bothered too much by my single usage of Stinsen.NavigationStack but something should be decided.

@rundfunk47 Have you been able to work on anything in regards to the latest comment in #29? If not, that's completely fine.

@LePips
Copy link
Collaborator

LePips commented Sep 12, 2022

A more pressing issue is that the navigation title styles (.inline, .large) are no longer consistent on push with Stinsen when targeting iOS 16. They will default to .large.

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

No branches or pull requests

3 participants