Fix review comments

Author: youda97

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

@@ -34,7 +34,7 @@ import {
 })
 export class OverflowMenuOption {
 	@HostBinding("class") optionClass = "bx--overflow-menu-options__option";
-	@HostBinding("attr.role") role = "none";
+	@HostBinding("attr.role") role = "presentation";
 
 	@HostBinding("class.bx--overflow-menu-options__option--danger")
 	public get isDanger(): Boolean {

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

@@ -46,7 +46,16 @@ export class OverflowMenuPane extends Dialog {
 			this.dialogConfig.menuLabel = this.i18n.get().OVERFLOW_MENU.OVERFLOW;
 		}
 
-		setTimeout(() => this.listItems()[0].focus());
+		setTimeout(() =>  {
+			let options = Array.from<any>(this.dialog.nativeElement.querySelectorAll("*"));
+
+			options.forEach(element => {
+				if (element.tabIndex === 0) {
+					element.tabIndex = -1;
+				}
+			});
+			this.listItems()[0].focus();
+		}, 0);
 	}
 
 	@HostListener("keydown", ["$event"])
@@ -87,15 +96,15 @@ export class OverflowMenuPane extends Dialog {
 				break;
 
 			case "Esc": // IE specific value
-			case "Escape": {
+			case "Escape":
 				event.stopImmediatePropagation();
 				this.doClose();
 				break;
-			}
 		}
 	}
 
 	clickClose(event) {
+		// Opens menu when you click on the menu button and stays open when navigating through the options
 		if (this.dialogConfig.parentRef.nativeElement.firstChild.contains(event.target) ||
 			this.listItems().some(button => button === (event.relatedTarget)) ||
 			(event.type === "focusout" && event.relatedTarget === this.dialogConfig.parentRef.nativeElement)) {

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

@@ -18,7 +18,7 @@ storiesOf("Overflow Menu", module)
 				</ibm-overflow-menu-option>
 				<ibm-overflow-menu-option>Option 2</ibm-overflow-menu-option>
 				<li class="bx--overflow-menu-options__option">
-					<button class="bx--overflow-menu-options__btn" tabindex="-1">A fully custom option</button>
+					<button class="bx--overflow-menu-options__btn">A fully custom option</button>
 				</li>
 				<ibm-overflow-menu-option>Option 4</ibm-overflow-menu-option>
 				<ibm-overflow-menu-option disabled="true">Disabled</ibm-overflow-menu-option>