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

bugfix(splashscreen) splashscreen not visible in scenes app #623

Conversation

joeflateau
Copy link

Referenced by #621

@joeflateau joeflateau changed the title fix splashscreen in scenes app bugfix(splashscreen) splashscreen not visible in scenes app Sep 20, 2021
@jcesarmobile
Copy link
Member

Thanks for the PR!

I get this error on line 131

Variable used within its own initial value

Anyway, there is a similar pull request and I'm merging that one since I like it more
#631

@joeflateau joeflateau deleted the hotfix/splashscreen-bounds branch December 6, 2021 20:45
@joeflateau
Copy link
Author

@jcesarmobile thanks! want me to redo my other PR (#622) to find the window the way #631 does?

@jcesarmobile
Copy link
Member

Thanks for the offering, but I was already working on it.

If you can verify if my pull request fix your issue it would be helpful
#729

@joeflateau
Copy link
Author

@jcesarmobile I ran the app with #631 and #729 fixes installed and the app worked as expected. I had concerns about finding the window by looking at the first scene rather than finding the owner window from the cap web view, so I tried:

  • in info.plist, phone scene listed first before carplay, launch the phone scene first
  • in info.plist, carplay scene listed first, launch phone scene first
  • in info.plist, carplay scene listed first, launch carplay scene first
  • in info.plist, phone scene listed first, launch carplay first

All worked as expected, so does seem like there's some internal consistency that the phone scene will be the first scene and CarPlay second, but on iPads where you can have your own app side-by-side with itself I think you'd end up with the wrong dimensions if the cap view is not in the first scene. But for my own purposes, these resolve my issues.

@jcesarmobile
Copy link
Member

If it works for you then I think it's ok.
Once we drop iOS 12 support we might make the default template a scenes app as Xcode does and then more people could face issues, if that happens we will need to revisit.

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