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.

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

Entry.IsSpellCheckEnabled="False" still shows squiggly line #8558

Closed
mobycorp opened this issue Jul 6, 2022 · 9 comments · Fixed by #15459
Closed

Entry.IsSpellCheckEnabled="False" still shows squiggly line #8558

mobycorp opened this issue Jul 6, 2022 · 9 comments · Fixed by #15459
Assignees
Labels
area-controls-entry Entry fixed-in-8.0.0-preview.7.8842 Look for this fix in 8.0.0-preview.7.8842! platform/windows 🪟 s/triaged Issue has been reviewed s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working

Comments

@mobycorp
Copy link

mobycorp commented Jul 6, 2022

Description

I have an entry field with IsSpellCheckEnabled="False". When text is entered into the field and it fails the spellchecker, it still draws a squiggly line under the text. I am expecting no squiggly line.

Screenshot 2022-07-06 071101

This is the xaml used to define the entry control:

<Entry
	x:Name="EmailAddressEntry"
	ClearButtonVisibility="WhileEditing"
	IsSpellCheckEnabled="False"
	Margin="0,2"
	Placeholder="{x:Static strings:AppResources.PlaceholderEnterEmailAddress}"
	Text="{Binding EmailAddress}" />

This field is used to enter an email address so there's a 99.9999% chance it will fail a spellchecker...

Steps to Reproduce

  1. In ANY Maui app, add an entry field and use the following definition:
<Entry
	x:Name="EmailAddressEntry"
	ClearButtonVisibility="WhileEditing"
	IsSpellCheckEnabled="False"
	Margin="0,2"
	Placeholder="{x:Static strings:AppResources.PlaceholderEnterEmailAddress}"
	Text="{Binding EmailAddress}" />
  1. Enter "asdefasdf@asdasdf.com"
  2. The dreaded squiggly will be drawn showing the "spelling error"

Version with bug

6.0.400 (current)

Last version that worked well

Unknown/Other

Affected platforms

Windows

Affected platform versions

windows10.0.19041.0

Did you find any workaround?

NO

Relevant log output

No response

@mobycorp mobycorp added s/needs-verification Indicates that this issue needs initial verification before further triage will happen t/bug Something isn't working labels Jul 6, 2022
@ghost ghost added the legacy-area-controls Label, Button, CheckBox, Slider, Stepper, Switch, Picker, Entry, Editor label Jul 6, 2022
@jsuarezruiz jsuarezruiz added platform/windows 🪟 and removed legacy-area-controls Label, Button, CheckBox, Slider, Stepper, Switch, Picker, Entry, Editor labels Jul 6, 2022
@Eilon Eilon added the legacy-area-controls Label, Button, CheckBox, Slider, Stepper, Switch, Picker, Entry, Editor label Jul 6, 2022
@jsuarezruiz
Copy link
Contributor

Getting spell checking also with Entry.IsSpellCheckEnabled="False".
image

Verified.

@jsuarezruiz jsuarezruiz added s/verified Verified / Reproducible Issue ready for Engineering Triage s/triaged Issue has been reviewed and removed s/needs-verification Indicates that this issue needs initial verification before further triage will happen labels Jul 7, 2022
@jsuarezruiz
Copy link
Contributor

This issue will be fixed by #7442

@BobAaldering
Copy link

BobAaldering commented Jan 6, 2023

For those who also have this problem. What can help is to also set the property Entry.IsTextPredictionEnabled to false within the Entry. Then text predictions and corrections are also disabled, and the squiggly line is gone (only verified on WinUI). See also the following link.

To disable spell checking, you can do the following:

<Entry ...
       IsSpellCheckEnabled="False"
       IsTextPredictionEnabled="False"
       ... />

@mattleibow mattleibow added this to the Backlog milestone Mar 20, 2023
@ghost
Copy link

ghost commented Mar 20, 2023

We've added this issue to our backlog, and we will work to address it as time and resources allow. If you have any additional information or questions about this issue, please leave a comment. For additional info about issue management, please read our Triage Process.

@mattleibow
Copy link
Member

This fix caused a few issues, see revert: #14147

@samhouts samhouts added fixed-in-8.0.0-preview.3.8149 Look for this fix in 8.0.0-preview.3.8149! and removed s/verified Verified / Reproducible Issue ready for Engineering Triage s/triaged Issue has been reviewed fixed-in-8.0.0-preview.3.8149 Look for this fix in 8.0.0-preview.3.8149! labels Apr 5, 2023
@edgiardina
Copy link

This is also the case in the searchbar control. even Setting

IsSpellCheckEnabled="False"
IsTextPredictionEnabled="False"

doesn't change the text prediction or spell check lines in iOS

@jknaudt21 jknaudt21 self-assigned this May 10, 2023
@XamlTest XamlTest added s/verified Verified / Reproducible Issue ready for Engineering Triage s/triaged Issue has been reviewed labels May 18, 2023
@XamlTest
Copy link

Verified this on Visual Studio Enterprise 17.7.0 Preview 1.0. Repro on Windows 11 with below Project:
MauiApp13.zip

image

Setting below, the squiggly line will disappear on Windows:

IsSpellCheckEnabled="False"
IsTextPredictionEnabled="False"

@samhouts samhouts modified the milestones: Backlog, .NET 8 Jul 11, 2023
@samhouts samhouts added the fixed-in-8.0.0-preview.7.8842 Look for this fix in 8.0.0-preview.7.8842! label Aug 8, 2023
@Richard-Dufour
Copy link

I confirm that squiggly line will disappear on Windows but will remain on iOS.

Microsoft Visual Studio Enterprise 2022 (64-bit) - Current
Version 17.6.5

Please don't close this ticket until resolved (even if there is no activity for a while :-)

@ghost
Copy link

ghost commented Aug 18, 2023

Hello lovely human, thank you for your comment on this issue. Because this issue has been closed for a period of time, please strongly consider opening a new issue linking to this issue instead to ensure better visibility of your comment. Thank you!

@ghost ghost locked as resolved and limited conversation to collaborators Oct 12, 2023
@Eilon Eilon removed the legacy-area-controls Label, Button, CheckBox, Slider, Stepper, Switch, Picker, Entry, Editor label May 10, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-controls-entry Entry fixed-in-8.0.0-preview.7.8842 Look for this fix in 8.0.0-preview.7.8842! platform/windows 🪟 s/triaged Issue has been reviewed s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

10 participants