Skip to content

Fix FocusZone to stop eating spacebars in textarea#2054

Merged
dzearing merged 1 commit intomicrosoft:masterfrom
antonf:fix-focuszone-textarea
Jun 23, 2017
Merged

Fix FocusZone to stop eating spacebars in textarea#2054
dzearing merged 1 commit intomicrosoft:masterfrom
antonf:fix-focuszone-textarea

Conversation

@antonf
Copy link
Copy Markdown
Contributor

@antonf antonf commented Jun 22, 2017

Pull request checklist

Addresses an existing issue: Fixes #2053

Description of changes

FocusZone._tryInvokeClickForFocusable should abort not only for tags, but also for <textarea> tag targets.
Modified _isElementInput which should treat textarea as input element also.

Focus areas to test

Check that spaces are inserted into textarea: https://codepen.io/antonf/pen/RgZbpO

@msftclas
Copy link
Copy Markdown

@antonf,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.

It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot

@msftclas
Copy link
Copy Markdown

@antonf, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.

Thanks, Microsoft Pull Request Bot

@dzearing dzearing merged commit 60d27e1 into microsoft:master Jun 23, 2017
@microsoft microsoft locked as resolved and limited conversation to collaborators Aug 31, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Textarea item under DetailsList eating space button press

3 participants