theKashey / react-focus-on

🎯 Solution for WAI ARIA compatible modal dialogs or full-screen tasks, you were looking for
MIT License
333 stars 14 forks source link

Allow `style` prop on <FocusOn /> #46

Closed thompsongl closed 3 years ago

thompsongl commented 3 years ago

Similar to how className is applied, style could be applied to the main div (the one with the data-focus-lock-disabled attribute).

Happy to open a pull request, but wanted to ask your preference:

1) Allow style prop only. 2) Allow lockProps prop similar to react-focus-lock where any valid HTML or React attribute can be applied.

After switching to react-focus-on from react-focus-lock this has been the only API gap. Thanks for your work!

theKashey commented 3 years ago

Valid point. A good addition to #42