revert: "Fix tabbing in quick pick"

src/vs/base/parts/quickopen/browser/quickOpenWidget.ts

@@ -244,8 +244,7 @@ export class QuickOpenWidget implements IModelProvider {
 						alwaysFocused: true,
 						verticalScrollMode: ScrollbarVisibility.Visible,
 						ariaLabel: nls.localize('treeAriaLabel', "Quick Picker"),
-						keyboardSupport: this.options.keyboardSupport,
-						preventRootFocus: true
+						keyboardSupport: this.options.keyboardSupport
 					});
 
 				this.treeElement = this.tree.getHTMLElement();
@@ -337,16 +336,6 @@ export class QuickOpenWidget implements IModelProvider {
 
 		this.applyStyles();
 
-		// Allows focus to switch to next/previous entry after tab into an actionbar item
-		DOM.addDisposableListener(this.treeContainer.getHTMLElement(), DOM.EventType.KEY_DOWN, (e: KeyboardEvent) => {
-			const keyboardEvent: StandardKeyboardEvent = new StandardKeyboardEvent(e);
-			if (keyboardEvent.keyCode === KeyCode.DownArrow || keyboardEvent.keyCode === KeyCode.UpArrow || keyboardEvent.keyCode === KeyCode.PageDown || keyboardEvent.keyCode === KeyCode.PageUp) {
-				DOM.EventHelper.stop(e, true);
-				this.navigateInTree(keyboardEvent.keyCode, keyboardEvent.shiftKey);
-				this.inputBox.inputElement.focus();
-			}
-		});
-
 		return this.builder.getHTMLElement();
 	}
 

src/vs/base/parts/tree/browser/tree.ts

@@ -670,7 +670,6 @@ export interface ITreeOptions extends ITreeStyles {
 	paddingOnRow?: boolean;
 	ariaLabel?: string;
 	keyboardSupport?: boolean;
-	preventRootFocus?: boolean;
 }
 
 export interface ITreeStyles {

src/vs/base/parts/tree/browser/treeView.ts

@@ -444,9 +444,7 @@ export class TreeView extends HeightMap {
 
 		this.domNode = document.createElement('div');
 		this.domNode.className = `monaco-tree no-focused-item monaco-tree-instance-${this.instance}`;
-		if (!context.options.preventRootFocus) {
-			this.domNode.tabIndex = 0;
-		}
+		this.domNode.tabIndex = 0;
 
 		this.styleElement = DOM.createStyleSheet(this.domNode);