From 75d0273d53fa569c64488839d3e5f4c49a49e869 Mon Sep 17 00:00:00 2001 From: Jon Gunderson Date: Wed, 3 May 2017 12:55:53 -0500 Subject: [PATCH] Menu Button Examples: fixe left and right arrow and misspelling of "label" (pull #392) 1. To fix bug raised by @racer2207 in issue #384, removed event support for LEFT and RIGHT arrow keys. 2. Changed "lable" to "label" in keyboard table. --- examples/menu-button/js/MenuItemAction.js | 12 ------------ examples/menu-button/js/MenuItemLinks.js | 12 ------------ .../js/PopupMenuActionActivedescendant.js | 12 ------------ .../menu-button-actions-active-descendant.html | 2 +- examples/menu-button/menu-button-actions.html | 2 +- 5 files changed, 2 insertions(+), 38 deletions(-) diff --git a/examples/menu-button/js/MenuItemAction.js b/examples/menu-button/js/MenuItemAction.js index 377d9160f7..3ad7e3a948 100644 --- a/examples/menu-button/js/MenuItemAction.js +++ b/examples/menu-button/js/MenuItemAction.js @@ -123,18 +123,6 @@ MenuItem.prototype.handleKeydown = function (event) { flag = true; break; - case this.keyCode.LEFT: - this.menu.setFocusToController('previous'); - this.menu.close(true); - flag = true; - break; - - case this.keyCode.RIGHT: - this.menu.setFocusToController('next'); - this.menu.close(true); - flag = true; - break; - case this.keyCode.HOME: case this.keyCode.PAGEUP: this.menu.setFocusToFirstItem(); diff --git a/examples/menu-button/js/MenuItemLinks.js b/examples/menu-button/js/MenuItemLinks.js index 1d0e50be81..ca4991f9e5 100644 --- a/examples/menu-button/js/MenuItemLinks.js +++ b/examples/menu-button/js/MenuItemLinks.js @@ -101,18 +101,6 @@ MenuItemLinks.prototype.handleKeydown = function (event) { flag = true; break; - case this.keyCode.LEFT: - this.menu.setFocusToController('previous'); - this.menu.close(true); - flag = true; - break; - - case this.keyCode.RIGHT: - this.menu.setFocusToController('next'); - this.menu.close(true); - flag = true; - break; - case this.keyCode.HOME: case this.keyCode.PAGEUP: this.menu.setFocusToFirstItem(); diff --git a/examples/menu-button/js/PopupMenuActionActivedescendant.js b/examples/menu-button/js/PopupMenuActionActivedescendant.js index d769a6290f..62bd9d5563 100644 --- a/examples/menu-button/js/PopupMenuActionActivedescendant.js +++ b/examples/menu-button/js/PopupMenuActionActivedescendant.js @@ -195,18 +195,6 @@ PopupMenuActionActivedescendant.prototype.handleKeydown = function (event) { flag = true; break; - case this.keyCode.LEFT: - this.setFocusToPreviousItem(); - this.close(true); - flag = true; - break; - - case this.keyCode.RIGHT: - this.setFocusToNextItem(); - this.close(true); - flag = true; - break; - case this.keyCode.HOME: case this.keyCode.PAGEUP: this.setFocusToFirstItem(); diff --git a/examples/menu-button/menu-button-actions-active-descendant.html b/examples/menu-button/menu-button-actions-active-descendant.html index 60c443200e..6ffc0cbe84 100644 --- a/examples/menu-button/menu-button-actions-active-descendant.html +++ b/examples/menu-button/menu-button-actions-active-descendant.html @@ -161,7 +161,7 @@

Menu

A-Z
a-z diff --git a/examples/menu-button/menu-button-actions.html b/examples/menu-button/menu-button-actions.html index 060ad525b6..0ab22c2b5a 100644 --- a/examples/menu-button/menu-button-actions.html +++ b/examples/menu-button/menu-button-actions.html @@ -161,7 +161,7 @@

Menu

A-Z
a-z