Skip to content

Commit 30f6129

Browse files
authored
Merge pull request #286 from maxceem/feature/connectv2
fix for connect-app issue 1942 - Project status menu doesn't close on tap outside
2 parents 3764ab7 + 40e640c commit 30f6129

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

components/Dropdown/enhanceDropdown.js

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,9 +14,11 @@ const enhanceDropdown = (CompositeComponent) => class extends Component {
1414
refreshEventHandlers() {
1515
if (this.state.isOpen) {
1616
document.addEventListener('click', this.onClickOutside)
17+
document.addEventListener('touchstart', this.onClickOutside)
1718
document.addEventListener('dropdownClicked', this.onClickOtherDropdown)
1819
} else {
1920
document.removeEventListener('click', this.onClickOutside)
21+
document.removeEventListener('touchstart', this.onClickOutside)
2022
document.removeEventListener('dropdownClicked', this.onClickOtherDropdown)
2123
}
2224
}
@@ -71,16 +73,19 @@ const enhanceDropdown = (CompositeComponent) => class extends Component {
7173

7274
componentDidMount() {
7375
document.removeEventListener('click', this.onClickOutside)
76+
document.removeEventListener('touchstart', this.onClickOutside)
7477
document.removeEventListener('dropdownClicked', this.onClickOtherDropdown)
7578

7679
if (this.state.isOpen) {
7780
document.addEventListener('click', this.onClickOutside)
81+
document.addEventListener('touchstart', this.onClickOutside)
7882
document.addEventListener('dropdownClicked', this.onClickOtherDropdown)
7983
}
8084
}
8185

8286
componentWillUnmount() {
8387
document.removeEventListener('click', this.onClickOutside)
88+
document.removeEventListener('touchstart', this.onClickOutside)
8489
document.removeEventListener('dropdownClicked', this.onClickOtherDropdown)
8590
}
8691

0 commit comments

Comments
 (0)