Merge pull request #3346 from danjm/i3228-event-not-defined

[old-ui] Define event locally in onClickOutside method in account-dropdowns.js
7 years ago · 40484b1c1f
parent 33373b676f 78f6a48664
commit 40484b1c1f
3 changed files with 3 additions and 3 deletions
--- a/old-ui/app/components/account-dropdowns.js
+++ b/old-ui/app/components/account-dropdowns.js
@ -173,7 +173,7 @@ class AccountDropdowns extends Component {
          minWidth: '180px',
        },
        isOpen: optionsMenuActive,
-        onClickOutside: () => {
+        onClickOutside: (event) => {
          const { classList } = event.target
          const isNotToggleElement = !classList.contains(this.optionsMenuToggleClassName)
          if (optionsMenuActive && isNotToggleElement) {
--- a/ui/app/components/account-dropdowns.js
+++ b/ui/app/components/account-dropdowns.js
@ -173,7 +173,7 @@ class AccountDropdowns extends Component {
          minWidth: '180px',
        },
        isOpen: optionsMenuActive,
-        onClickOutside: () => {
+        onClickOutside: (event) => {
          const { classList } = event.target
          const isNotToggleElement = !classList.contains(this.optionsMenuToggleClassName)
          if (optionsMenuActive && isNotToggleElement) {
--- a/ui/app/components/dropdowns/components/account-dropdowns.js
+++ b/ui/app/components/dropdowns/components/account-dropdowns.js
@ -281,7 +281,7 @@ class AccountDropdowns extends Component {
          dropdownWrapperStyle,
        ),
        isOpen: optionsMenuActive,
-        onClickOutside: () => {
+        onClickOutside: (event) => {
          const { classList } = event.target
          const isNotToggleElement = !classList.contains(this.optionsMenuToggleClassName)
          if (optionsMenuActive && isNotToggleElement) {