From f8033f4083fb9c961273ee16410dea75e29bb92f Mon Sep 17 00:00:00 2001
From: Sanjana Kapur <sanjanakapur@microsoft.com>
Date: Wed, 12 Oct 2022 21:57:05 -0700
Subject: [PATCH 1/5] Fix AB#686: Add suggestion for Configure Proxy box

---
 .../manual-proxy-configuration-window.html                  | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/client/proxy/manual-proxy-configuration-window/manual-proxy-configuration-window.html b/src/client/proxy/manual-proxy-configuration-window/manual-proxy-configuration-window.html
index e7cb8f0383..fdd01be6ee 100644
--- a/src/client/proxy/manual-proxy-configuration-window/manual-proxy-configuration-window.html
+++ b/src/client/proxy/manual-proxy-configuration-window/manual-proxy-configuration-window.html
@@ -52,15 +52,15 @@ <h1>Enter proxy configuration for BatchExplorer</h1>
     </div>
     <div class="form-element">
         <label for="port">Port</label>
-        <input id="port">
+        <input id="port" placeholder="ex: 80">
     </div>
     <div class="form-element">
         <label for="username">Username</label>
-        <input id="username">
+        <input id="username" placeholder="ex: sample_username">
     </div>
     <div class="form-element">
         <label for="password">Password</label>
-        <input id="password" type="password">
+        <input id="password" type="password" placeholder="ex: samplepassword">
     </div>
     <br>
     <button class="save-button" onclick="submit()">Save</button>

From b7529298e936d1eee0219a9f1c3d7c4f8c355d89 Mon Sep 17 00:00:00 2001
From: Sanjana Kapur <sanjanakapur@microsoft.com>
Date: Wed, 12 Oct 2022 22:01:44 -0700
Subject: [PATCH 2/5] Fix AB#790: Make callout accessible via keyboard

---
 src/@batch-flask/ui/callout/callout.directive.ts | 12 ++++++++++++
 src/@batch-flask/ui/callout/callout.html         |  2 +-
 2 files changed, 13 insertions(+), 1 deletion(-)

diff --git a/src/@batch-flask/ui/callout/callout.directive.ts b/src/@batch-flask/ui/callout/callout.directive.ts
index baa9111077..0cba29710b 100644
--- a/src/@batch-flask/ui/callout/callout.directive.ts
+++ b/src/@batch-flask/ui/callout/callout.directive.ts
@@ -3,6 +3,7 @@ import { TemplatePortal } from "@angular/cdk/portal";
 import { Directive, ElementRef,  HostListener, Input, ViewContainerRef } from "@angular/core";
 import { Subscription } from "rxjs";
 import { CalloutComponent } from "./callout.component";
+import { ENTER, SPACE } from "@batch-flask/core/keys";
 
 @Directive({
     selector: "[blCallout]",
@@ -25,6 +26,17 @@ export class CalloutDirective {
         }
     }
 
+    @HostListener("keydown", ["$event"])
+    public onkeydown(event: KeyboardEvent) {
+        if (event.key === SPACE || event.key === ENTER) {
+            if (this._overlayRef) {
+                this.close();
+            } else {
+                this.open();
+            }
+        }
+    }
+
     public open() {
         this._overlayRef = this.overlay.create(this._getOverlayConfig());
         this._backDropClickSub = this._overlayRef.backdropClick().subscribe(() => {
diff --git a/src/@batch-flask/ui/callout/callout.html b/src/@batch-flask/ui/callout/callout.html
index 0c084b3f5f..f23afbbec7 100644
--- a/src/@batch-flask/ui/callout/callout.html
+++ b/src/@batch-flask/ui/callout/callout.html
@@ -1,5 +1,5 @@
 <ng-template>
-    <div class="bl-callout">
+    <div class="bl-callout" tabindex="0">
         <ng-content></ng-content>
     </div>
 </ng-template>

From 427c84241cb10eb036accc409d710b3e63ef0b25 Mon Sep 17 00:00:00 2001
From: Sanjana Kapur <sanjanakapur@microsoft.com>
Date: Wed, 12 Oct 2022 22:10:45 -0700
Subject: [PATCH 3/5] Fix AB#883: Make graph parts keyboard accessible

---
 .../account-monitoring-section.html                    |  6 +++---
 .../monitoring/monitor-chart/monitor-chart.html        | 10 +++++-----
 2 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/src/app/components/account/details/account-monitoring-section/account-monitoring-section.html b/src/app/components/account/details/account-monitoring-section/account-monitoring-section.html
index 7286385b4b..5918a832ae 100644
--- a/src/app/components/account/details/account-monitoring-section/account-monitoring-section.html
+++ b/src/app/components/account/details/account-monitoring-section/account-monitoring-section.html
@@ -22,9 +22,9 @@ <h1 class="section-title">{{'account-details.monitoring' | i18n }}</h1>
         role="application">
         <bl-monitor-chart [metrics]="metricType" [timeRange]="currentRange" [chartType]="chartType"></bl-monitor-chart>
     </a>
-    <div class="monitor-chart current-node-counts">
-        <h2>{{'account-monitoring.currentNodeStates' | i18n }}</h2>
-        <bl-pool-state-graph></bl-pool-state-graph>
+    <div class="monitor-chart current-node-counts" tabindex="0">
+        <h2 tabindex="0">{{'account-monitoring.currentNodeStates' | i18n }}</h2>
+        <bl-pool-state-graph tabindex="0" aria-label="graph"></bl-pool-state-graph>
     </div>
 </div>
 <div  *ngIf="!isArmAccount">
diff --git a/src/app/components/account/monitoring/monitor-chart/monitor-chart.html b/src/app/components/account/monitoring/monitor-chart/monitor-chart.html
index abfbb10e97..64d32815b7 100644
--- a/src/app/components/account/monitoring/monitor-chart/monitor-chart.html
+++ b/src/app/components/account/monitoring/monitor-chart/monitor-chart.html
@@ -1,20 +1,20 @@
 <ng-container *ngIf="!preview">
     <div class="header">
-        <h2>
+        <h2 tabindex="0">
             {{ 'account-monitoring.' + metrics | i18n }}
         </h2>
 
 
-        <div class="aggregation" *ngIf="aggregation">
+        <div class="aggregation" *ngIf="aggregation" tabindex="0">
             {{aggregation}}
         </div>
     </div>
     <ng-container *ngIf="isChartReady">
-        <div class="chart-container">
+        <div class="chart-container" tabindex="0" aria-label="graph">
             <canvas blChart [datasets]="datasets" [options]="options" [chartType]="chartType" [colors]="colors"></canvas>
         </div>
         <div class="legend">
-            <div class="legend-item" *ngFor="let dataset of datasets;trackBy: trackDataSet;let index=index">
+            <div class="legend-item" *ngFor="let dataset of datasets;trackBy: trackDataSet;let index=index" tabindex="0">
                 <div class="value">
                     {{total[index]}}
                 </div>
@@ -23,7 +23,7 @@ <h2>
                 </div>
             </div>
         </div>
-        <div class="interval-label" *ngIf="interval">
+        <div class="interval-label" *ngIf="interval" tabindex="0">
             {{'account-monitoring.aggregatedOver' | i18n: {interval: interval} }}
         </div>
     </ng-container>

From ec14a4872b07758481a1308ee3b22288b2bda773 Mon Sep 17 00:00:00 2001
From: Sanjana Kapur <sanjanakapur@microsoft.com>
Date: Wed, 12 Oct 2022 22:18:32 -0700
Subject: [PATCH 4/5] Fix AB#999: Add keyboard focus for Job Status pane

---
 src/app/components/account/details/account-details.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/app/components/account/details/account-details.html b/src/app/components/account/details/account-details.html
index 0b506d9d13..afcaf53945 100644
--- a/src/app/components/account/details/account-details.html
+++ b/src/app/components/account/details/account-details.html
@@ -9,7 +9,7 @@
         <bl-account-cost-card></bl-account-cost-card>
         <h1 class="section-title">{{'account-details.resources' | i18n }}</h1>
         <div class="account-cards">
-            <bl-card class="account-card">
+            <bl-card class="account-card" routerLink="/jobs" (click)="selectAccount(account.id)">
                 <div class="title">
                     <h2>Job status</h2>
                 </div>

From 1cb36504ba357da838ffa2be92e700092c33bf29 Mon Sep 17 00:00:00 2001
From: Sanjana Kapur <sanjanakapur@microsoft.com>
Date: Wed, 12 Oct 2022 22:28:58 -0700
Subject: [PATCH 5/5] Fix AB#917: Add asterisk to ID in Create Pool form

---
 .../components/pool/action/add/pool-create-basic-dialog.html | 2 +-
 .../components/pool/action/add/pool-create-basic-dialog.scss | 5 +++++
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/src/app/components/pool/action/add/pool-create-basic-dialog.html b/src/app/components/pool/action/add/pool-create-basic-dialog.html
index 774a2403ed..b5dbd05224 100644
--- a/src/app/components/pool/action/add/pool-create-basic-dialog.html
+++ b/src/app/components/pool/action/add/pool-create-basic-dialog.html
@@ -3,7 +3,7 @@
         <bl-form-section title="Pool details" subtitle="Basic information about the pool">
             <div class="grow">
                 <div class="gcol">
-                    <bl-form-field class="form-element">
+                    <bl-form-field class="form-element required">
                         <input blInput #idInput formControlName="id" placeholder="ID" maxlength="64">
 
                         <bl-hint align="end" attr.aria-label="{{idInput.value.length}} characters out of 64">{{idInput.value.length}} / 64</bl-hint>
diff --git a/src/app/components/pool/action/add/pool-create-basic-dialog.scss b/src/app/components/pool/action/add/pool-create-basic-dialog.scss
index a23d49e0da..07420468a6 100644
--- a/src/app/components/pool/action/add/pool-create-basic-dialog.scss
+++ b/src/app/components/pool/action/add/pool-create-basic-dialog.scss
@@ -26,3 +26,8 @@ bl-pool-create-basic-dialog {
         color: #1d54af;
     }
 }
+
+.required label:after {
+    content: "*";
+    color: $validation-error-color;
+}