diff --git a/accessible/block-options-modal.component.js b/accessible/block-options-modal.component.js
index 0a81f6029..e44c15244 100644
--- a/accessible/block-options-modal.component.js
+++ b/accessible/block-options-modal.component.js
@@ -64,15 +64,15 @@ blocklyApp.BlockOptionsModalComponent = ng.core.Component({
this.modalIsVisible = false;
this.actionButtonsInfo = [];
this.activeActionButtonIndex = 0;
- this.onCancelCallback = null;
+ this.onDismissCallback = null;
var that = this;
this.blockOptionsModalService.registerPreShowHook(
- function(newActionButtonsInfo, onCancelCallback) {
+ function(newActionButtonsInfo, onDismissCallback) {
that.modalIsVisible = true;
that.actionButtonsInfo = newActionButtonsInfo;
that.activeActionButtonIndex = 0;
- that.onCancelCallback = onCancelCallback;
+ that.onDismissCallback = onDismissCallback;
that.keyboardInputService.setOverride({
// Tab key: no-op.
@@ -83,20 +83,23 @@ blocklyApp.BlockOptionsModalComponent = ng.core.Component({
// Enter key: selects an action, performs it, and closes the
// modal.
'13': function(evt) {
+ evt.preventDefault();
+ evt.stopPropagation();
+
var button = document.getElementById(
that.getOptionId(that.activeActionButtonIndex));
if (that.activeActionButtonIndex <
that.actionButtonsInfo.length) {
that.actionButtonsInfo[that.activeActionButtonIndex].action();
} else {
- that.onCancelCallback();
+ that.onDismissCallback();
}
that.hideModal();
},
// Escape key: closes the modal.
'27': function() {
- that.onCancelCallback();
+ that.onDismissCallback();
that.hideModal();
},
// Up key: navigates to the previous item in the list.
diff --git a/accessible/block-options-modal.service.js b/accessible/block-options-modal.service.js
index 6b77b92bd..82e425c05 100644
--- a/accessible/block-options-modal.service.js
+++ b/accessible/block-options-modal.service.js
@@ -32,19 +32,19 @@ blocklyApp.BlockOptionsModalService = ng.core.Class({
'before it can be shown.');
};
this.modalIsShown = false;
- this.onCancelCallback = null;
+ this.onDismissCallback = null;
}],
registerPreShowHook: function(preShowHook) {
this.preShowHook = function() {
- preShowHook(this.actionButtonsInfo, this.onCancelCallback);
+ preShowHook(this.actionButtonsInfo, this.onDismissCallback);
};
},
isModalShown: function() {
return this.modalIsShown;
},
- showModal: function(actionButtonsInfo, onCancelCallback) {
+ showModal: function(actionButtonsInfo, onDismissCallback) {
this.actionButtonsInfo = actionButtonsInfo;
- this.onCancelCallback = onCancelCallback;
+ this.onDismissCallback = onDismissCallback;
this.preShowHook();
this.modalIsShown = true;
diff --git a/accessible/sidebar.component.js b/accessible/sidebar.component.js
index 225195b16..3df591fdd 100644
--- a/accessible/sidebar.component.js
+++ b/accessible/sidebar.component.js
@@ -37,17 +37,19 @@ blocklyApp.SidebarComponent = ng.core.Component({
{{buttonConfig.text}}
-
-
-