fix(ButtonDropdown): add close event when Escape key is pressed (#1634)

[ftonato]

Hey there,

This pull-request add a small piece of code to close event when Escape key is pressed (#1634)

• Bug fix
• New feature
• Chore
• Breaking change
• There is an open issue which this change addresses
• I have read the CONTRIBUTING document.
• My commits follow the Git Commit Guidelines
• My code follows the code style of this project.
• My change requires a change to the documentation.
• • I have updated the documentation accordingly.
• I have added tests to cover my changes.
• All new and existing tests passed.

Fixes #1634

[TheSharpieOne]

Thanks for the PR.
I think a better place to make this change would be in part of the key down handler which handles when the key is on the button itself:

reactstrap/src/Dropdown.js

Lines 126 to 134 in dc0b8ae

	if (this.getMenuCtrl() === e.target) {
	if (
	!this.props.isOpen
	&& ([keyCodes.space, keyCodes.enter, keyCodes.up, keyCodes.down].indexOf(e.which) > -1)
	) {
	this.toggle(e);
	setTimeout(() => this.getMenuItems()[0].focus());
	}
	}

maybe something like:

   if ( 
     !this.props.isOpen 
     && ([keyCodes.space, keyCodes.enter, keyCodes.up, keyCodes.down].indexOf(e.which) > -1) 
   ) { 
     this.toggle(e); 
     setTimeout(() => this.getMenuItems()[0].focus()); 
   } else if (this.props.isOpen && e.which === keyCodes.esc) {
     this.toggle(e); 
   } 

[TheSharpieOne]

Awesome! Thanks again!