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

Nullable fixes in DrawingView #1817

Merged
merged 4 commits into from
Feb 23, 2022
Merged

Nullable fixes in DrawingView #1817

merged 4 commits into from
Feb 23, 2022

Conversation

VladislavAntonyuk
Copy link
Contributor

@VladislavAntonyuk VladislavAntonyuk commented Feb 21, 2022

Description of Bug

Remove null-forgiving operator in drawing view.

I am not able to deploy the app to android device with VS 17.2, I was not able to test it on Android.
It works fine on UWP, WPF

Issues Fixed

Behavioral Changes

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

@bijington
Copy link
Contributor

Thanks @VladislavAntonyuk my comments may well be my not understanding how the DrawingView works but they stood out to me

@VladislavAntonyuk VladislavAntonyuk marked this pull request as ready for review February 23, 2022 12:39
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 thanks @VladislavAntonyuk

@bijington bijington merged commit 5bd4b5b into main Feb 23, 2022
@bijington bijington deleted the 1747-drawing-view-fix branch February 23, 2022 21:10
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.

[Bug] DrawingView In Expander (Android :☠️/ iOS : 🙈 )
2 participants