Remove brackets and update tabIndex

Author: youda97

src/dialog/overflow-menu/overflow-menu-pane.component.ts

@@ -46,7 +46,7 @@ export class OverflowMenuPane extends Dialog {
 		const listItems = this.listItems();
 
 		switch (event.key) {
-			case "ArrowDown": {
+			case "ArrowDown":
 				event.preventDefault();
 				if (!isFocusInLastItem(event, listItems))  {
 					const index = listItems.findIndex(item => item === event.target);
@@ -55,9 +55,9 @@ export class OverflowMenuPane extends Dialog {
 					listItems[0].focus();
 				}
 				break;
-			}
 
-			case "ArrowUp": {
+			case "Up": // IE specific value
+			case "ArrowUp":
 				event.preventDefault();
 				if (!isFocusInFirstItem(event, listItems))  {
 					const index = listItems.findIndex(item => item === event.target);
@@ -66,24 +66,21 @@ export class OverflowMenuPane extends Dialog {
 					listItems[listItems.length - 1].focus();
 				}
 				break;
-			}
 
-			case "Home": {
+			case "Home":
 				event.preventDefault();
 				listItems[0].focus();
 				break;
-			}
 
-			case"End": {
+			case"End":
 				event.preventDefault();
 				listItems[listItems.length - 1].focus();
 				break;
-			}
 
 			case "Enter":
-			case " ": {
+			case " ":
 				this.doClose();
-			}
+				break;
 		}
 	}
 

src/dialog/overflow-menu/overflow-menu.component.ts

@@ -20,8 +20,9 @@ import { Component } from "@angular/core";
 			[appendToBody]="true"
 			placement="bottom"
 			class="bx--overflow-menu"
-			style="display: block;">
-			<svg class="bx--overflow-menu__icon" width="3" height="15" viewBox="0 0 3 15" tabindex="0">
+			style="display: block;"
+			tabindex="0">
+			<svg class="bx--overflow-menu__icon" width="3" height="15" viewBox="0 0 3 15">
 				<g fill-rule="evenodd">
 					<circle cx="1.5" cy="1.5" r="1.5" />
 					<circle cx="1.5" cy="7.5" r="1.5" />

src/dialog/overflow-menu/overflow-menu.directive.ts

@@ -57,10 +57,9 @@ export class OverflowMenuDirective extends DialogDirective {
 	hostkeys(event: KeyboardEvent) {
 		switch (event.key) {
 			case "Enter":
-			case " ": {
+			case " ":
 				this.toggle();
 				break;
-			}
 		}
 	}
 }