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

Add autocorrect content attribute to HTMLElement #3595

Open
rniwa opened this Issue Mar 28, 2018 · 9 comments

Comments

8 participants
@rniwa
Copy link
Collaborator

rniwa commented Mar 28, 2018

Like autocapitalize, autocorrect would turn on/off autocorrection on iOS. It would be good to have this iOS feature standardized.

@domenic

This comment has been minimized.

Copy link
Member

domenic commented Apr 1, 2018

@dtapuska any interest in helping with this?

@kojiishi

This comment has been minimized.

Copy link

kojiishi commented Apr 2, 2018

+1, I'm interested in this in Blink. /cc @yosinch

@nixkuroi

This comment has been minimized.

Copy link

nixkuroi commented Apr 4, 2018

I recommend also adding "no-punctuate" option, bringing options to off|on|no-punctuate.

No-punctuate would imply that autocorrect is on, but removes the sometimes tedious problem of removing punctuation whenever two spaces are pressed.

Currently, to disable this behavior, it's all or nothing at the OS level (detailed here: http://osxdaily.com/2012/07/03/stop-the-period-automatically-typing-in-ios/)

@dtapuska

This comment has been minimized.

Copy link
Contributor

dtapuska commented Apr 4, 2018

Blink supports this on HTMLElement although the value isn't in the IDL; it is queried on the focused element.. See https://cs.chromium.org/chromium/src/third_party/WebKit/Source/core/editing/ime/InputMethodController.cpp?type=cs&l=1342

I support getting this spec'd. "on" and "off" are the only attribute values Blink supports (which appears to be case sensitive so there are likely some changes to Blink)

@jfbrennan

This comment has been minimized.

Copy link

jfbrennan commented Apr 12, 2018

Anyone know if an autoselect attr - selects input, textarea value upon focus, deselects on blur, probably applies to contenteditable elements as well...? Has this been considered?

Ready to get started if there isn't already work being done.

@annevk

This comment has been minimized.

Copy link
Member

annevk commented Apr 13, 2018

@jfbrennan that's probably best discussed in a separate issue. There's nothing on file that I know of.

@jfbrennan

This comment has been minimized.

Copy link

jfbrennan commented Apr 13, 2018

@annevk yes, will do. Thanks

@jfbrennan

This comment has been minimized.

Copy link

jfbrennan commented Apr 13, 2018

#3627, would appreciate feedback and sorry @rniwa for intruding here ;)

@r12a r12a added i18n-tracking and removed i18n-tracking labels Sep 4, 2018

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