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
Sextant >= 1.7.4 throws Exception on PushModal #94
Comments
Hey @xlbs-rm 👋, Thank you for opening an issue. We will get back to you as soon as we can. Also, check out our Open Collective and consider backing us. https://opencollective.com/reactiveui
|
This sounds like an issue with the 2.0 release, and should be resolved when it ships. I am trying to test out a few examples. The sample that works in this repository is fine. Also, Sextant doesn't currently support Xamarin.Forms 4.1 so I'm sure that's not helping. Closing this issue, as 1.7.4 is a prerelease package that is older and we are working out issues for the 2.0 release in our pre-release packages. Most recent packages on the feed is 1.7.14 |
@RLittlesII : the issue still appears also in Sextant 1.7.14. @see: https://github.com/xlbs-rm/TabbedPageSextantSample The Error also occures with XamarinForms 4.0.* (tested with: 4.0.0.540366) Exception:
|
Hey there, unfortunately this is very much still in development so it comes as is. The latest supported release is 1.5.5 which is the latest listed here https://www.nuget.org/packages/Sextant/
|
Thanks for the bug report anyway, will keep an eye out for it before the 2.0 release. |
And downgrading Xamarin.Forms to 3.6 doesn't resolve the issue? Again. Sextant does not currently support Xamarin.Forms 4.0 The error I see points to it being in Xamarin.Forms.Core |
@RLittlesII I don't see the error in Xamarin.Forms.Core due regular apps with push modal work quite fine. Thanks for re-opening the issue. As long as this is not fixed we stop using navigation parameters. |
@xlbs-rm I was able to reproduce this with the sample in this repository. I'm not quite sure how this ever worked. The offensive line of code seems to have been in the repository for a while, but I feel like this used to work. Regardless, thank you for reporting this issue. Apologies for closing it initially, I didn't realize it wasn't a regression issue at first. |
Thanks a lot for resolving this issue quickly! |
We are trying to ship 2.0. Anything that doesn't work that should is better fixed now. |
Updating above Sextant 1.7.3 throws an exception if page is pushed modal on android and ios (Tested by debugging on Simulator/Emulator)
The Exception is: "Page must not already have a parent..."
See small testproject updated with myget sextant:
bochos-bln/TabbedPageSextantSample@cf9f5aa
Steps To Reproduce
Start the App, press "First Modal Button"
Expected behavior
Modal Page Pushed (like with Sextant 1.7.3
... but App Crashes
Environment
Additional context
The text was updated successfully, but these errors were encountered: