diff --git a/extensions/query-store/src/common/utils.ts b/extensions/query-store/src/common/utils.ts index 94120c6e9b..72632c774a 100644 --- a/extensions/query-store/src/common/utils.ts +++ b/extensions/query-store/src/common/utils.ts @@ -36,7 +36,7 @@ export async function createOneComponentFlexContainer(view: azdata.ModelView, co * @param flexFlow row or column * @returns Flex container containing the two components */ -export async function createTwoComponentFlexContainer(view: azdata.ModelView, firstComponent: azdata.Component, secondComponent: azdata.Component, flexFlow: string): Promise { +export function createTwoComponentFlexContainer(view: azdata.ModelView, firstComponent: azdata.Component, secondComponent: azdata.Component, flexFlow: string): azdata.FlexContainer { const flexContainer = view.modelBuilder.flexContainer().component(); if (flexFlow === 'row') { diff --git a/extensions/query-store/src/reports/baseQueryStoreReport.ts b/extensions/query-store/src/reports/baseQueryStoreReport.ts index 8916c62c1a..9dbb8d1da9 100644 --- a/extensions/query-store/src/reports/baseQueryStoreReport.ts +++ b/extensions/query-store/src/reports/baseQueryStoreReport.ts @@ -61,18 +61,18 @@ export abstract class BaseQueryStoreReport { case 2: { // TODO: replace 800 to have the number be based on how big the window is // one container on top, one on the bottom - mainContainer = this.resizeable ? utils.createVerticalSplitView(view, containers[0], containers[1], 800) : await utils.createTwoComponentFlexContainer(view, containers[0], containers[1], 'column'); + mainContainer = this.resizeable ? utils.createVerticalSplitView(view, containers[0], containers[1], 800) : utils.createTwoComponentFlexContainer(view, containers[0], containers[1], 'column'); break; } case 3: { // 2 containers on top, one on the bottom // TODO: support portrait and landscape view. Right now it's landscape view only - mainContainer = this.resizeable ? utils.createVerticalSplitView(view, await utils.createTwoComponentFlexContainer(view, containers[0], containers[1], 'row'), containers[2], 800) - : await utils.createTwoComponentFlexContainer(view, await utils.createTwoComponentFlexContainer(view, containers[0], containers[1], 'row'), containers[2], 'column'); + mainContainer = this.resizeable ? utils.createVerticalSplitView(view, utils.createTwoComponentFlexContainer(view, containers[0], containers[1], 'row'), containers[2], 800) + : utils.createTwoComponentFlexContainer(view, utils.createTwoComponentFlexContainer(view, containers[0], containers[1], 'row'), containers[2], 'column'); break; } case 4: { // 2 containers on top, 2 on the bottom - mainContainer = this.resizeable ? utils.createVerticalSplitView(view, await utils.createTwoComponentFlexContainer(view, containers[0], containers[1], 'row'), await utils.createTwoComponentFlexContainer(view, containers[2], containers[3], 'row'), 800) - : await utils.createTwoComponentFlexContainer(view, await utils.createTwoComponentFlexContainer(view, containers[0], containers[1], 'row'), await utils.createTwoComponentFlexContainer(view, containers[2], containers[3], 'row'), 'column'); + mainContainer = this.resizeable ? utils.createVerticalSplitView(view, utils.createTwoComponentFlexContainer(view, containers[0], containers[1], 'row'), utils.createTwoComponentFlexContainer(view, containers[2], containers[3], 'row'), 800) + : utils.createTwoComponentFlexContainer(view, utils.createTwoComponentFlexContainer(view, containers[0], containers[1], 'row'), utils.createTwoComponentFlexContainer(view, containers[2], containers[3], 'row'), 'column'); break; } default: { throw new Error(`{views.length} number of views in a QDS report is not supported`);