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

WIP Handle CloseButton together with ShowDialogsOverTitleBar #3153

Merged

Conversation

punker76
Copy link
Member

@punker76 punker76 commented Jan 16, 2018

What changed?

  • Pass correct dialogs settings to HandleOverlayOnShow
  • New OwnerCanCloseWithDialog property at MetroDialogSettings which can be used to handle how the owner of the dialog can be closed.
  • New IsCloseButtonEnabledWithDialog property at MetroWindow which indicates if the close button should be enabled or not if a dialog is shown.
  • HandleOverlayOnHide sets the IsCloseButtonEnabledWithDialogPropertyKey and restores focus if needed
  • HandleOverlayOnShow sets the IsCloseButtonEnabledWithDialogPropertyKey
    Verified

Closed issues.

Closes #2882

- New `CanCloseDialogOwner` property at `MetroDialogSettings` which can be used to handle how the owner of the dialog can be closed.
- New `IsCloseButtonEnabledWithDialog` property at `MetroWindow` which indicates if the close button should be enabled or not if a dialog is shown.
@punker76 punker76 added this to the 1.6.0 milestone Jan 16, 2018
@punker76 punker76 self-assigned this Jan 16, 2018
- Add trigger for `IsCloseButtonEnabledWithDialog` at WindowButtonCommands for PART_Close button
- `HandleOverlayOnHide` sets the `IsCloseButtonEnabledWithDialogPropertyKey` and restores focus if needed
- `HandleOverlayOnShow` sets the `IsCloseButtonEnabledWithDialogPropertyKey`
@punker76 punker76 merged commit 6c58e81 into develop Jan 16, 2018
@punker76 punker76 deleted the feature/GH-2882_ShowDialogsOverTitleBar_Window_CloseButton branch January 16, 2018 12:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

Titlebar Close command does not work with ShowDialogsOverTitleBar=False
1 participant