diff --git a/extensions/sql-migration/src/constants/strings.ts b/extensions/sql-migration/src/constants/strings.ts index e5e7a2563f..2b367a16b8 100644 --- a/extensions/sql-migration/src/constants/strings.ts +++ b/extensions/sql-migration/src/constants/strings.ts @@ -290,6 +290,7 @@ export const SUCCESSFULLY_MIGRATED_TO_AZURE_SQL = localize('sql.migration.succes export const MIGRATION_NOT_STARTED = localize('sql.migration.migration.not.started', "Migration not started"); export const CHOOSE_TO_MIGRATE_TO_AZURE_SQL = localize('sql.migration.choose.to.migrate.to.azure.sql', "Choose to migrate to Azure SQL"); export const COMING_SOON = localize('sql.migration.coming.soon', "Coming soon"); +export const SHOW_STATUS = localize('sql.migration.show.status', "Show status"); export function MIGRATION_INPROGRESS_WARNING(count: number) { switch (count) { case 1: diff --git a/extensions/sql-migration/src/dashboard/sqlServerDashboard.ts b/extensions/sql-migration/src/dashboard/sqlServerDashboard.ts index 15563613f9..abcee43a26 100644 --- a/extensions/sql-migration/src/dashboard/sqlServerDashboard.ts +++ b/extensions/sql-migration/src/dashboard/sqlServerDashboard.ts @@ -270,9 +270,9 @@ export class DashboardWidget { if (warningCount > 0) { this._inProgressWarningMigrationButton.warningText!.value = loc.MIGRATION_INPROGRESS_WARNING(warningCount); this._inProgressMigrationButton.container.display = 'none'; - this._inProgressWarningMigrationButton.container.display = 'inline'; + this._inProgressWarningMigrationButton.container.display = ''; } else { - this._inProgressMigrationButton.container.display = 'inline'; + this._inProgressMigrationButton.container.display = ''; this._inProgressWarningMigrationButton.container.display = 'none'; } @@ -286,8 +286,8 @@ export class DashboardWidget { const failedMigrations = filterMigrations(migrations, AdsMigrationStatus.FAILED); const failedCount = failedMigrations.length; if (failedCount > 0) { - this._failedMigrationButton.container.display = 'inline'; - this._failedMigrationButton.count.value = failedMigrations.length.toString(); + this._failedMigrationButton.container.display = ''; + this._failedMigrationButton.count.value = failedCount.toString(); } else { this._failedMigrationButton.container.display = 'none'; } @@ -295,7 +295,7 @@ export class DashboardWidget { const completingCutoverMigrations = filterMigrations(migrations, AdsMigrationStatus.COMPLETING); const cutoverCount = completingCutoverMigrations.length; if (cutoverCount > 0) { - this._completingMigrationButton.container.display = 'inline'; + this._completingMigrationButton.container.display = ''; this._completingMigrationButton.count.value = cutoverCount.toString(); } else { this._completingMigrationButton.container.display = 'none'; @@ -327,6 +327,7 @@ export class DashboardWidget { 'margin-bottom': '0px', 'width': '300px', 'font-size': '14px', + 'font-weight': 'bold' } }).component(); @@ -342,9 +343,7 @@ export class DashboardWidget { const flex = this._view.modelBuilder.flexContainer().withProps({ CSSStyles: { 'width': '400px', - 'height': '50px', - 'margin-top': '10px', - 'border': '1px solid', + 'height': '50px' } }).component(); @@ -374,8 +373,14 @@ export class DashboardWidget { const compositeButton = this._view.modelBuilder.divContainer().withItems([flex]).withProps({ ariaRole: 'button', - ariaLabel: 'show status', - clickable: true + ariaLabel: loc.SHOW_STATUS, + clickable: true, + CSSStyles: { + 'width': '400px', + 'border': '1px solid', + 'margin-top': '10px', + 'height': '50px' + } }).component(); return { container: compositeButton, @@ -456,8 +461,6 @@ export class DashboardWidget { CSSStyles: { 'width': '400px', 'height': '70px', - 'margin-top': '10px', - 'border': '1px solid' } }).component(); @@ -488,7 +491,13 @@ export class DashboardWidget { const compositeButton = this._view.modelBuilder.divContainer().withItems([flex]).withProps({ ariaRole: 'button', ariaLabel: 'show status', - clickable: true + clickable: true, + CSSStyles: { + 'width': '400px', + 'height': '70px', + 'margin-top': '10px', + 'border': '1px solid' + } }).component(); return { container: compositeButton, diff --git a/src/sql/workbench/browser/modelComponents/divContainer.component.ts b/src/sql/workbench/browser/modelComponents/divContainer.component.ts index e3939d1028..ebc836b3d8 100644 --- a/src/sql/workbench/browser/modelComponents/divContainer.component.ts +++ b/src/sql/workbench/browser/modelComponents/divContainer.component.ts @@ -18,6 +18,8 @@ import { KeyCode } from 'vs/base/common/keyCodes'; import { IComponentDescriptor, IComponent, IModelStore, ComponentEventType } from 'sql/platform/dashboard/browser/interfaces'; import { convertSize } from 'sql/base/browser/dom'; import { ILogService } from 'vs/platform/log/common/log'; +import { registerThemingParticipant, IColorTheme, IThemeService, ICssStyleCollector } from 'vs/platform/theme/common/themeService'; +import * as colorRegistry from 'vs/platform/theme/common/colorRegistry'; class DivItem { constructor(public descriptor: IComponentDescriptor, public config: azdata.DivItemLayout) { } @@ -25,7 +27,7 @@ class DivItem { @Component({ template: ` -