Allow developer to configure auto focus on the grid (addresses #813)

[supamanda]

Description

A PR to allow developer to configure whether the grid should steal focus when the browser is focused on the body. Addresses #813

Please check if the PR fulfills these requirements

• The commit message follows our guidelines: https://github.com/adazzle/react-data-grid/blob/master/CONTRIBUTING.md
• Tests for the changes have been added (for bug fixes / features)
• Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (check one with "x")

[ ] Bugfix
[x ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Other... Please describe:

What is the current behavior? (You can also link to an open issue here)
The grid steals focus when the browser is focused on the body, see #813

What is the new behavior?
Allow developer to configure whether grid should autofocus when browser is focused on the body

Does this PR introduce a breaking change? (check one with "x")

[ ] Yes
[x ] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...

Other information:

[coveralls]

Changes Unknown when pulling 9ab8687 on supamanda:ah-allow-configure-auto-focus into ** on adazzle:master**.

[coveralls]

Changes Unknown when pulling 68a6ddb on supamanda:ah-allow-configure-auto-focus into ** on adazzle:master**.

[theunixbeard]

👍 This PR looks great, Amanda, and would really help w/ my app as well.

It seems like any app w/ a complex enough UI to require scrolling would run into this issue, so it's definitely important.

Though referencing the solution in my older PR from a few months back (#704), this video demonstrates the problem in any easy to understand manner: https://www.youtube.com/watch?v=xejQdVGJX6w

With the configuration flag in Amanda's solution, app's like mine and Mike's (#813) could simply set the flag to false and get the behavior we want while leaving everyone else unaffected.

@malonecj any thoughts on this? Seems like there are no conflicts to a merge.

[thom801]

@supamanda I am also needing this feature. Can this be merged? Any way I can help?

[coveralls]

Changes Unknown when pulling 57588ed on supamanda:ah-allow-configure-auto-focus into ** on adazzle:master**.

[supamanda]

hi @thom801 , I just fixed the merge conflicts, so it's good to go again. In the meantime, I've come up with a workaround I'd be happy to share with you if you need it

[scrambled2k3]

Any time frame on when this can get merged in? I too am experiencing this and would love to disable it.

[cohuttaboy79]

I am also having this issue and having the ability to disable the functionality would be much appreciated.

[scrambled2k3]

@supamanda would you mind sharing with me your workaround?

[supamanda]

hi @scrambled2k3
My work around is to define my own row renderer, and override the checkFocus function; and then define my own Grid which uses the my row renderer. Code attached :)

noautofocus.zip

[coveralls]

Changes Unknown when pulling 3478fb2 on supamanda:ah-allow-configure-auto-focus into ** on adazzle:master**.