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

When typing in editor, macOS Accessibility Keyboard's word suggestion fails #45096

Closed
blakewatson opened this issue Mar 5, 2018 · 8 comments
Closed
Assignees
Labels
accessibility Keyboard, mouse, ARIA, vision, screen readers (non-specific) issues bug Issue identified by VS Code Team member as probable bug editor-input Editor text input macos Issues with VS Code on MAC/OS X verified Verification succeeded
Milestone

Comments

@blakewatson
Copy link

Issue Type

Bug

Description

The Accessibility Keyboard in macOS High Sierra has a word prediction feature that allows the user to type a few characters and get suggestions on possible dictionary words. This feature is helpful for oncreen keyboard users because it saves precious key strokes/clicks.

When typing in VS Code's editor, the suggestion lists resets itself on each key stroke, offering suggestions only for the last key typed, instead of allowing the user to build up a word.

The behavior can be seen in this GIF.

https://www.blakewatson.com/externalfiles/vscode-accessibility-keyboard.gif

I tested in a SCSS file I had open, as well as in the interactive playground. I also tested with extensions off.

Note that the issue does not happen in the Monaco Editor online demo.

VS Code Info

VS Code version: Code 1.20.1 (f88bbf9, 2018-02-13T15:23:28.607Z)
OS version: Darwin x64 17.4.0

System Info
Item Value
CPUs Intel(R) Core(TM) i5-4690 CPU @ 3.50GHz (4 x 3500)
Load (avg) 2, 2, 2
Memory (System) 24.00GB (3.19GB free)
Process Argv /Applications/Visual Studio Code.app/Contents/MacOS/Electron
Screen Reader no
VM 0%
Extensions (2)
Extension Author (truncated) Version
format-indent Kas 1.3.0
php-intellisense fel 2.2.9

(1 theme extensions excluded)

Reproduces without extensions
@vscodebot vscodebot bot added editor editor-core Editor basic functionality labels Mar 5, 2018
@kieferrm kieferrm added the accessibility Keyboard, mouse, ARIA, vision, screen readers (non-specific) issues label Mar 6, 2018
@kieferrm
Copy link
Member

kieferrm commented Mar 6, 2018

Reproduced on macOS HighSierra with the built-in accessibility keyboard.

@rebornix
Copy link
Member

rebornix commented Mar 6, 2018

Technically it's about how we handle content in the textarea, we already did similar optimization for Screen Reader. @blakewatson you can set editor.accessibilitySupport to on and you should be able to see word based suggestions as expected. Let me know if that works.

We should see if there is a way to automatically detect Accessibility Keyboard and turn on this optimization without asking users to change the setting.

@rebornix rebornix added this to the March 2018 milestone Mar 6, 2018
@blakewatson
Copy link
Author

Thanks @rebornix and @kieferrm, that worked. What's the tradeoff, if any, to having editor.accessibilitySupport on vs not?

@rebornix
Copy link
Member

rebornix commented Mar 6, 2018

@alexandrudima can you help explain what's the side effect of turning on accessibilitySupport? Thanks ;)

@blakewatson
Copy link
Author

I found a dialog that says "Some editor features will have different behavior behavior: e.g. word wrapping, folding, etc."

@alexdima
Copy link
Member

alexdima commented Mar 6, 2018

By default, we only leave the last typed character in the <textarea> on OSX when we don't detect a screen reader is attached. That covers the cases when long pressing e into é. We could leave the entire word in the <textarea> to cover the Accessibility Keyboard case.

@alexdima alexdima self-assigned this Mar 6, 2018
@alexdima alexdima added bug Issue identified by VS Code Team member as probable bug and removed editor labels Mar 6, 2018
@blakewatson
Copy link
Author

@alexandrudima Ah, yeah from my perspective as a user that would be a great solution if doable—then I could keep access to text wrapping and whatever other features get restricted when accessibilitySupport is on.

@rebornix rebornix modified the milestones: March 2018, Backlog Mar 28, 2018
@alexdima alexdima modified the milestones: Backlog, March 2018 Mar 29, 2018
@alexdima alexdima added editor-input Editor text input and removed editor-core Editor basic functionality labels Mar 29, 2018
@alexdima
Copy link
Member

To the verifier:
Please check that the "long press" input method still works. e.g. e ===> è, etc.

Before:

kapture 2018-03-29 at 12 37 43

After:

kapture 2018-03-29 at 12 43 35

@alexdima alexdima added the macos Issues with VS Code on MAC/OS X label Mar 29, 2018
@rebornix rebornix added the verified Verification succeeded label Mar 30, 2018
@vscodebot vscodebot bot locked and limited conversation to collaborators May 13, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
accessibility Keyboard, mouse, ARIA, vision, screen readers (non-specific) issues bug Issue identified by VS Code Team member as probable bug editor-input Editor text input macos Issues with VS Code on MAC/OS X verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

4 participants