Extension (IsCharacterString()) give false state when I enter space or tap

[AbdAlghaniAlbiek]

In Extensions session (IsCharacterString()) when I enter space or tap in textbox that check if all characters inside it are only letters not numbers or operators -> give me false state such as:
textbox.Text = Howareyou? -> true
textbox.Text = How are you? -> false

Hello AbdAlghaniAlbiek, thank you for opening an issue with us!

I have automatically added a "needs triage" label to help get things started. Our team will analyze and investigate the issue, and escalate it to the relevant team if possible. Other community members may also look into the issue and provide feedback 🙌

[Kyaa-dost]

@AbdAlghaniAlbiek can you please provide more detail on this? The steps you are taking or add a screenshot would be great.

[AbdAlghaniAlbiek]

I hope that but github doesn't support uploading images
I'll give you an example on my situation:

sting testText = Hello Kyaa;
if(testText.IsCharacterString()){
Debug.Writline("Valid") ;
}
else{
Debug.Writeline("Not valid");
}

==> Not valid

you can see that there isn't numbers or operators it should be valid but when I enter on space or tap it give me Not valid
I want to ignore spaces and tabs in text

[Kyaa-dost]

@AbdAlghaniAlbiek is that the code you are using? Seems like 'Writeline' is missing an e in the third line and string in first?

[AbdAlghaniAlbiek]

No this is just example I have written on github

[Kyaa-dost]

@AbdAlghaniAlbiek That makes sense. Thanks for clarifying 👍

[michael-hawker]

@AbdAlghaniAlbiek the purpose of this method was to validate letter characters only (a 'space' is not a letter); it doesn't accept spaces as valid input as you've discovered by design.

The RegEx driving this is here:

https://github.com/windows-toolkit/WindowsCommunityToolkit/blob/c65d6486952dec37f4a219ec177eb2709bb51c79/Microsoft.Toolkit/Extensions/StringExtensions.cs#L25

If you'd like, you could open a PR to add an additional method to accept whitespace like:

public static bool ContainsCharactersOrWhiteSpace(this string str) ...

We should probably deprecate IsCharacterString in 6.1 and add an equivalent ContainsCharacters method for matching name and clarity and remove IsCharacterString in 7.0? Thoughts?

@Sergio0694 curious on your optimization expertise for these cases? (especially as these aren't really localized as well... though larger changes to the behavior should probably go in the 7.0 release)