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

Activation and RoutedViewHost don't play nicely together #525

Closed
jen20 opened this issue Feb 21, 2014 · 5 comments
Closed

Activation and RoutedViewHost don't play nicely together #525

jen20 opened this issue Feb 21, 2014 · 5 comments
Labels

Comments

@jen20
Copy link
Contributor

jen20 commented Feb 21, 2014

RoutedViewHost being derived from TransitioningContentControl means that whenever you navigate to a new view model, the old one is reactivated as it is loaded again when it becomes the "previous" content in the TransitioningContentControl (at least I think that's what I'm seeing!)

If that's the case, it might be an idea to have a NonTransitioningRoutedViewHost as well (or ideally to have RoutedViewHost deriving from ContentControl and TransitioningRoutedViewHost deriving from TransitioningContentControl, thought that might be a breaking change too far).

I'd be happy to look into this, I have a version implemented in my project at the moment (which is for a fixed screen size so nixes the orientation stuff which makes it otherwise impossible to implement outside of the ReactiveUI assembly because of the internal IPlatformOperations).

Any thoughts?

@anaisbetts
Copy link
Member

Let's just fix the bug :) I think we can do something like combine Activated and HitTestVisible to make this work

@jen20
Copy link
Contributor Author

jen20 commented Feb 22, 2014

True that would also work ;-) will take a look.

@jen20
Copy link
Contributor Author

jen20 commented Feb 26, 2014

Actually irrespective of this, non-transitioning versions of these are useful for if you have annoying controls like WebBrowser which have airspace issues with TransitioningContentControl.

@anaisbetts
Copy link
Member

@jen20 I think you can just override the template, yes?

@jen20
Copy link
Contributor Author

jen20 commented Feb 26, 2014

That's probably a better solution.

@lock lock bot added the outdated label Jun 26, 2019
@lock lock bot locked and limited conversation to collaborators Jun 26, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants