-
-
Notifications
You must be signed in to change notification settings - Fork 18
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
Alt+F4 calls WizardPageViewModelBase.CancelAsync #330
Comments
The problem is that the The quick-and-dirty solution I came up with is to override the |
Great find. We should indeed fix this in the WizardWindow. Interested in a PR? |
Sure, I'll look into it.. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Please check all of the platforms you are having the issue on (if platform is not listed, it is not supported)
Component
WizardWindow / WizardViewModel
Version of Library
4.2.0 / 4.3.0
Version of OS(s) listed above with issue
win10
Steps to Reproduce
CancelAsync
method on anyWizardPageViewModel
(e.g.SkillsPageViewModel
)Expected Behavior
The behavior is identical to the one used for the
CancelCommand
- that is theWizardPageViewModel.CancelAsync
is only called after the user confirms the cancellation (by default).Actual Behavior
The
CancelAsync
on theWizardPageViewModel
is called before the confirmation dialog is displayed (from the respectiveCancelAsync
method on theWizardViewModel
).The text was updated successfully, but these errors were encountered: