-
Notifications
You must be signed in to change notification settings - Fork 808
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
enable click to close in iOS (#301) #304
Conversation
} else { | ||
this.focusContent(); | ||
} | ||
handleOverlayClick(event) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lint: handleOverlayClick (event)
.
} | ||
|
||
handleContentMouseUp = () => { | ||
this.shouldClose = false; | ||
handleContentClick(event) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lint: handleContentClick (event)
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also...
Lint: static handleContentClick (event)
.
If a class method does not use this, it can safely be made a static function.
Hope this one is correct.
Sorry I don't have time to work through all the tests and this is my first time creating a pull request, perhaps I can figure it out this weekend. But so far everything works fine on my project. |
Fixes #[301].
Changes proposed:
Upgrade Path (for changed or removed APIs):
Acceptance Checklist:
CONTRIBUTING.md
.