Skip to content

Conversation

@samatar26
Copy link
Contributor

Fixes #967
Used the correct handler for closing the modal. I don't think I'l be able to demonstrate this using pictures so maybe try running it locally, sorry.

@kazup01 kazup01 self-requested a review October 13, 2017 00:35
Copy link
Member

@kazup01 kazup01 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kazup01 kazup01 merged commit af30838 into BoostIO:master Oct 13, 2017
@kazup01
Copy link
Member

kazup01 commented Oct 13, 2017

Merged. Thanks for your contribution @samatar26 !

@samatar26
Copy link
Contributor Author

@kazup01 どう致しまして, 頑張りましょう!

@kazup01
Copy link
Member

kazup01 commented Oct 13, 2017

@samatar26 ありがとうございます👍

@kazup01
Copy link
Member

kazup01 commented Oct 28, 2017

This improving feature has just released at v0.8.16! Thanks!
#1027

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants