Skip to content
This repository was archived by the owner on Aug 30, 2023. It is now read-only.

change logic of closing open popups on move/size#30

Merged
azchohfi merged 1 commit intorel/6.0.0from
move-open-popups
Jan 15, 2019
Merged

change logic of closing open popups on move/size#30
azchohfi merged 1 commit intorel/6.0.0from
move-open-popups

Conversation

@kmelmon
Copy link
Copy Markdown
Contributor

@kmelmon kmelmon commented Jan 15, 2019

Issue: #25 Xaml Island popups should not be closed when WPF/WinForms window is moved/sized

PR Type

What kind of change does this PR introduce?

< - Bugfix >

What is the current behavior?

<In WPF/WinForms hosting Xaml Islands scenarios, all open popups are closed whenever the host window is moved or resized. This logic is problematic. In some cases popups that should not be closed are getting closed (eg ContentDialog)>

What is the new behavior?

When host window is moved, all open popups will be re-positioned to follow the new position of the host window. No action is taken when the host window is resized anymore.

PR Checklist

Please check if your PR fulfills the following requirements:

  • [ x] Tested code with current supported SDKs
  • Pull Request has been submitted to the documentation repository instructions. Link:
  • Sample in sample app has been added / updated (for bug fixes / features)
  • Tests for the changes have been added (for bug fixes / features) (if applicable)
  • Header has been added to all new source files (run build/UpdateHeaders.bat)
  • [x ] Contains NO breaking changes

Other information

@kmelmon kmelmon requested a review from gegao18 January 15, 2019 18:39
@dnfclas
Copy link
Copy Markdown

dnfclas commented Jan 15, 2019

CLA assistant check
All CLA requirements met.

@kmelmon kmelmon requested review from marb2000 and rjmurillo January 15, 2019 18:43
@rjmurillo rjmurillo requested review from azchohfi and removed request for rjmurillo January 15, 2019 18:50
@azchohfi azchohfi merged commit 3a5475f into rel/6.0.0 Jan 15, 2019
@delete-merged-branch delete-merged-branch bot deleted the move-open-popups branch January 15, 2019 20:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants