Skip to content
This repository has been archived by the owner on May 1, 2024. It is now read-only.

PopupRenderer: check for isDisposed in OnElementPropertyChanged before taking action. Fixes https://github.com/xamarin/XamarinCommunityToolkit/issues/1947 #1968

Merged
merged 1 commit into from
May 6, 2023

Conversation

mfeingol
Copy link
Contributor

@mfeingol mfeingol commented May 4, 2023

Description of Bug

cf #1947

Issues Fixed

Add a check for isDisposed and make sure the code has the same safeguards in all versions of the renderer. No new tests or samples should be needed.

Behavioral Changes

None of significance.

PR Checklist

  • Has a linked Issue, and the Issue has been approved
  • Has tests (if omitted, state reason in description)
  • Has samples (if omitted, state reason in description)
  • Rebased on top of main at time of PR
  • Changes adhere to coding standard

Copy link
Contributor

@pictos pictos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@bijington bijington left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work thank you

@bijington bijington merged commit 9b80d79 into xamarin:main May 6, 2023
7 checks passed
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.

None yet

3 participants