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.

Sign up for GitHub

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

Jump to bottom

TextBox.Text = null throws exception #11031

Closed
DierkDroth opened this issue Jan 23, 2023 · 3 comments · Fixed by #11113
Closed

TextBox.Text = null throws exception #11031

DierkDroth opened this issue Jan 23, 2023 · 3 comments · Fixed by #11113
Assignees
@Youssef1313
Labels
area/winui Categorizes an issue or PR as relevant to WinUI difficulty/tbd Categorizes an issue for which the difficulty level needs to be defined. kind/bug Something isn't working platform/all Categorizes an issue or PR as relevant to the all platforms triage/untriaged Indicates an issue requires triaging or verification

Comments

@DierkDroth
Copy link

Current behavior

Please see attched repo and search for '!!!'

TextBoxText.zip

WASM throws exception as per screenshot below
image

Expected behavior

WASM should work as WindowsAppSDK

How to reproduce it (as minimally and precisely as possible)

No response

Workaround

No response

Works on UWP/WinUI

Yes

Environment

Uno.UI / Uno.UI.WebAssembly / Uno.UI.Skia

NuGet package version(s)

No response

Affected platforms

No response

IDE

No response

IDE version

No response

Relevant plugins

No response

Anything else we need to know?

No response
@DierkDroth DierkDroth added difficulty/tbd Categorizes an issue for which the difficulty level needs to be defined. kind/bug Something isn't working triage/untriaged Indicates an issue requires triaging or verification labels Jan 23, 2023
@Youssef1313
Copy link
Member

Is WinUI behavior different than UWP?

I think UWP used to throw null when setting the text to null. #1303

@Youssef1313 Youssef1313 self-assigned this Jan 24, 2023
@marwalsch
Copy link

The issue is reproducible on iOS and Android as well.

@Youssef1313 It is, WinUI has no issue with null assignements to TextBox.Text.

@Youssef1313 Youssef1313 added the area/winui Categorizes an issue or PR as relevant to WinUI label Jan 31, 2023
@Youssef1313 Youssef1313 mentioned this issue Jan 31, 2023
Merged
6 tasks
@Youssef1313 Youssef1313 changed the title [WASM] TextBox.Text = null throws exception TextBox.Text = null throws exception Jan 31, 2023
@Youssef1313 Youssef1313 added the platform/all Categorizes an issue or PR as relevant to the all platforms label Jan 31, 2023
@DierkDroth
Copy link
Author

@jeromelaban @Youssef1313 thanks guys

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Youssef1313 Youssef1313

Labels
area/winui Categorizes an issue or PR as relevant to WinUI difficulty/tbd Categorizes an issue for which the difficulty level needs to be defined. kind/bug Something isn't working platform/all Categorizes an issue or PR as relevant to the all platforms triage/untriaged Indicates an issue requires triaging or verification
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(TextBox): Setting TextBox.Text to null should only throw in UWP Youssef1313/uno
3 participants
@DierkDroth @Youssef1313 @marwalsch