From c32c09e1a74986cb1ee1cbd6b9c1c28704facf61 Mon Sep 17 00:00:00 2001
From: nasc17 <69922333+nasc17@users.noreply.github.com>
Date: Wed, 28 Jul 2021 12:28:40 -0700
Subject: [PATCH] Postgres Compute and Storage page accessibility fixes. 
 (#16203)

* focus, required, aria

* Fix logic for focusing buttons

* Remove buttons focus
---
 .../ui/dashboards/postgres/postgresComputeAndStoragePage.ts   | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/extensions/arc/src/ui/dashboards/postgres/postgresComputeAndStoragePage.ts b/extensions/arc/src/ui/dashboards/postgres/postgresComputeAndStoragePage.ts
index 1583856a44..c6856d169c 100644
--- a/extensions/arc/src/ui/dashboards/postgres/postgresComputeAndStoragePage.ts
+++ b/extensions/arc/src/ui/dashboards/postgres/postgresComputeAndStoragePage.ts
@@ -479,6 +479,7 @@ export class PostgresComputeAndStoragePage extends DashboardPage {
 
 		const keyComponent = this.modelView.modelBuilder.text().withProps({
 			value: loc.workerNodeCount,
+			requiredIndicator: true,
 			CSSStyles: { ...cssStyles.text, 'margin-block-start': '0px', 'margin-block-end': '0px' }
 		}).component();
 
@@ -488,6 +489,7 @@ export class PostgresComputeAndStoragePage extends DashboardPage {
 		const information = this.modelView.modelBuilder.button().withProps({
 			iconPath: IconPathHelper.information,
 			title: loc.workerNodesInformation,
+			ariaLabel: loc.workerNodesInformation,
 			width: '15px',
 			height: '15px',
 			enabled: false
@@ -547,6 +549,7 @@ export class PostgresComputeAndStoragePage extends DashboardPage {
 		if (component.value === originalValue) {
 			return false;
 		} else if ((!component.valid)) {
+			this.discardButton.enabled = true;
 			return false;
 		} else {
 			this.saveButton.enabled = true;
@@ -584,6 +587,7 @@ export class PostgresComputeAndStoragePage extends DashboardPage {
 		const information = this.modelView.modelBuilder.button().withProps({
 			iconPath: IconPathHelper.information,
 			title: description,
+			ariaLabel: description,
 			width: '15px',
 			height: '15px',
 			enabled: false