From 9aba847a92932180b61d11303db35926c22f955b Mon Sep 17 00:00:00 2001 From: Ben Gotow Date: Fri, 18 Mar 2016 14:03:37 -0700 Subject: [PATCH] fix(onboarding): Focus first field when showing settings #1696 --- .../onboarding/lib/account-settings-page.cjsx | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/internal_packages/onboarding/lib/account-settings-page.cjsx b/internal_packages/onboarding/lib/account-settings-page.cjsx index f06747e0a..00e235a40 100644 --- a/internal_packages/onboarding/lib/account-settings-page.cjsx +++ b/internal_packages/onboarding/lib/account-settings-page.cjsx @@ -81,6 +81,19 @@ class AccountSettingsPage extends React.Component + componentDidMount: => + @_applyFocus() + + componentDidUpdate: => + @_applyFocus() + + _applyFocus: => + firstInput = React.findDOMNode(@).querySelector('input') + anyInputFocused = document.activeElement and document.activeElement.nodeName is 'INPUT' + + if firstInput and not anyInputFocused + firstInput.focus() + _onSettingsChanged: (event) => # NOTE: This code is largely duplicated in _onValueChanged. TODO Fix! {field, format} = event.target.dataset