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

Option to wrap a modal inside a NavigationPage #98

Closed
modplug opened this issue Jul 12, 2019 · 4 comments · Fixed by #103
Closed

Option to wrap a modal inside a NavigationPage #98

modplug opened this issue Jul 12, 2019 · 4 comments · Fixed by #103
Labels
bug Something isn't working outdated
Projects

Comments

@modplug
Copy link

modplug commented Jul 12, 2019

When I do PushModal from my viewmodel, the modal shows just fine. But as a good citizen I added a "Close" Toolbar button in the top right corner so that the user can opt out of the modal. The button is never shown because the modal doesn't have a NavigationPage.

I've used similiar frameworks before and I think MvvmCross has an option to "WrapInNavigationPage" when pushing modals.

An alternate solution would perhaps be to do this by using interactions, but that might trick the IViewStackService

There should be an option to wrap it in a NavigationPage.

Additional context

Screenshot nr 2 is the modal page without the NavigationBar.

Screenshot 2019-07-12 at 11 32 07
Screenshot 2019-07-12 at 11 32 26

@open-collective-bot
Copy link

open-collective-bot bot commented Jul 12, 2019

Hey @modplug 👋,

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

PS.: We offer priority support for all backers. Don't forget to add priority label when you start backing us 😄

An advanced, composable, functional reactive model-view-viewmodel framework for all .NET platforms!

@glennawatson glennawatson added the bug Something isn't working label Jul 12, 2019
@giusepe
Copy link
Member

giusepe commented Jul 12, 2019

Possibly a regression caused by #96 or even earlier in the 1.7.*
I tested with the version 1.5.5 and works as intended, but the view/viewmodel registration is different.

Thanks for trying the bleeding edge version and reporting the bug, just a heads up that version 1.7.* is a WIP that will become 2.0 and more suitable for production.

@RLittlesII RLittlesII added this to To do in v2.0.0 Jul 16, 2019
@RLittlesII RLittlesII moved this from To do to In progress in v2.0.0 Jul 16, 2019
@RLittlesII
Copy link
Member

@modplug I think I found the issue going to do a bit of checking around and verify that I haven't broken anything else. It was a regression issue with how we are registering the NavigationView.
👍 @giusepe for that thought.

v2.0.0 automation moved this from In progress to Done Jul 19, 2019
@RLittlesII
Copy link
Member

@modplug If you get the chance can you pull the beta bits and verify this solves your problem?

@lock lock bot added the outdated label Oct 17, 2019
@lock lock bot locked and limited conversation to collaborators Oct 17, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working outdated
Projects
No open projects
v2.0.0
  
Done
Development

Successfully merging a pull request may close this issue.

4 participants