Skip to content

Fixed issue where UI Responsive events were not fired#603

Merged
sanderl merged 2 commits intomicrosoft:masterfrom
sanderl:uiresponsive-bug
Jul 26, 2019
Merged

Fixed issue where UI Responsive events were not fired#603
sanderl merged 2 commits intomicrosoft:masterfrom
sanderl:uiresponsive-bug

Conversation

@sanderl
Copy link
Copy Markdown
Contributor

@sanderl sanderl commented Jul 26, 2019

Fixes #602

Description of the changes:

  • Updated the LogWindowCreated method to take in the windowId and update the WindowIdLog
  • Renamed UpdateWindowIdLog to IsWindowIdInLog so that it reflects what the method is actually doing.

How changes were validated:

  • Manually using internal diagnostics tools to ensure the UI Responsive events are firing now.

@sanderl sanderl merged commit 8106691 into microsoft:master Jul 26, 2019
Geveffe pushed a commit that referenced this pull request Jul 26, 2019
* Fixed the WindowIdLog so that it is updated when a new WindowCreated event is fired

* Updated the windowidlog check in LogWindowCreated to use IsWindowIdInLog
@sanderl sanderl deleted the uiresponsive-bug branch September 3, 2019 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Missing UI_Responsive Event in Latest Calculator Build

2 participants