From 6e2cb978428ba078ebe0d171a69322308ab48d48 Mon Sep 17 00:00:00 2001
From: SurenNeware <sneware9@in.ibm.com>
Date: Thu, 24 Dec 2020 20:58:16 +0530
Subject: Fixed icon or text only button accessibility

- Add sr-only element to icon only buttons
- Removed title from text only buttons

Signed-off-by: Suren Neware <sneware9@in.ibm.com>
Change-Id: Ica57dec70401c04a9d7506306a406fe394e52113
---
 src/views/AccessControl/ClientSessions/ClientSessions.vue            | 1 +
 src/views/Health/HardwareStatus/HardwareStatusTableBmcManager.vue    | 2 +-
 src/views/Health/HardwareStatus/HardwareStatusTableChassis.vue       | 2 +-
 src/views/Health/HardwareStatus/HardwareStatusTableDimmSlot.vue      | 2 +-
 src/views/Health/HardwareStatus/HardwareStatusTableFans.vue          | 2 +-
 src/views/Health/HardwareStatus/HardwareStatusTablePowerSupplies.vue | 2 +-
 src/views/Health/HardwareStatus/HardwareStatusTableProcessors.vue    | 2 +-
 src/views/Health/HardwareStatus/HardwareStatusTableStystem.vue       | 2 +-
 8 files changed, 8 insertions(+), 7 deletions(-)

(limited to 'src/views')

diff --git a/src/views/AccessControl/ClientSessions/ClientSessions.vue b/src/views/AccessControl/ClientSessions/ClientSessions.vue
index f15c63c7..1921c86a 100644
--- a/src/views/AccessControl/ClientSessions/ClientSessions.vue
+++ b/src/views/AccessControl/ClientSessions/ClientSessions.vue
@@ -73,6 +73,7 @@
               :value="action.value"
               :title="action.title"
               :row-data="row.item"
+              :btn-icon-only="false"
               :data-test-id="`sessionLogs-button-deleteRow-${row.index}`"
               @click-table-action="onTableRowAction($event, row.item)"
             ></table-row-action>
diff --git a/src/views/Health/HardwareStatus/HardwareStatusTableBmcManager.vue b/src/views/Health/HardwareStatus/HardwareStatusTableBmcManager.vue
index 29a6679b..2fdcb2ed 100644
--- a/src/views/Health/HardwareStatus/HardwareStatusTableBmcManager.vue
+++ b/src/views/Health/HardwareStatus/HardwareStatusTableBmcManager.vue
@@ -13,12 +13,12 @@
         <b-button
           variant="link"
           data-test-id="hardwareStatus-button-expandBmc"
-          :aria-label="expandRowLabel"
           :title="expandRowLabel"
           class="btn-icon-only"
           @click="toggleRowDetails(row)"
         >
           <icon-chevron />
+          <span class="sr-only">{{ expandRowLabel }}</span>
         </b-button>
       </template>
 
diff --git a/src/views/Health/HardwareStatus/HardwareStatusTableChassis.vue b/src/views/Health/HardwareStatus/HardwareStatusTableChassis.vue
index 231f7c67..2bf11f62 100644
--- a/src/views/Health/HardwareStatus/HardwareStatusTableChassis.vue
+++ b/src/views/Health/HardwareStatus/HardwareStatusTableChassis.vue
@@ -13,12 +13,12 @@
         <b-button
           variant="link"
           data-test-id="hardwareStatus-button-expandChassis"
-          :aria-label="expandRowLabel"
           :title="expandRowLabel"
           class="btn-icon-only"
           @click="toggleRowDetails(row)"
         >
           <icon-chevron />
+          <span class="sr-only">{{ expandRowLabel }}</span>
         </b-button>
       </template>
 
diff --git a/src/views/Health/HardwareStatus/HardwareStatusTableDimmSlot.vue b/src/views/Health/HardwareStatus/HardwareStatusTableDimmSlot.vue
index 70df3466..ca9e949c 100644
--- a/src/views/Health/HardwareStatus/HardwareStatusTableDimmSlot.vue
+++ b/src/views/Health/HardwareStatus/HardwareStatusTableDimmSlot.vue
@@ -35,12 +35,12 @@
         <b-button
           variant="link"
           data-test-id="hardwareStatus-button-expandDimms"
-          :aria-label="expandRowLabel"
           :title="expandRowLabel"
           class="btn-icon-only"
           @click="toggleRowDetails(row)"
         >
           <icon-chevron />
+          <span class="sr-only">{{ expandRowLabel }}</span>
         </b-button>
       </template>
 
diff --git a/src/views/Health/HardwareStatus/HardwareStatusTableFans.vue b/src/views/Health/HardwareStatus/HardwareStatusTableFans.vue
index e16495b1..10b14a47 100644
--- a/src/views/Health/HardwareStatus/HardwareStatusTableFans.vue
+++ b/src/views/Health/HardwareStatus/HardwareStatusTableFans.vue
@@ -35,12 +35,12 @@
         <b-button
           variant="link"
           data-test-id="hardwareStatus-button-expandFans"
-          :aria-label="expandRowLabel"
           :title="expandRowLabel"
           class="btn-icon-only"
           @click="toggleRowDetails(row)"
         >
           <icon-chevron />
+          <span class="sr-only">{{ expandRowLabel }}</span>
         </b-button>
       </template>
 
diff --git a/src/views/Health/HardwareStatus/HardwareStatusTablePowerSupplies.vue b/src/views/Health/HardwareStatus/HardwareStatusTablePowerSupplies.vue
index 7b4e5a19..3de69198 100644
--- a/src/views/Health/HardwareStatus/HardwareStatusTablePowerSupplies.vue
+++ b/src/views/Health/HardwareStatus/HardwareStatusTablePowerSupplies.vue
@@ -35,12 +35,12 @@
         <b-button
           variant="link"
           data-test-id="hardwareStatus-button-expandPowerSupplies"
-          :aria-label="expandRowLabel"
           :title="expandRowLabel"
           class="btn-icon-only"
           @click="toggleRowDetails(row)"
         >
           <icon-chevron />
+          <span class="sr-only">{{ expandRowLabel }}</span>
         </b-button>
       </template>
 
diff --git a/src/views/Health/HardwareStatus/HardwareStatusTableProcessors.vue b/src/views/Health/HardwareStatus/HardwareStatusTableProcessors.vue
index 6e2ebdae..2f956a9f 100644
--- a/src/views/Health/HardwareStatus/HardwareStatusTableProcessors.vue
+++ b/src/views/Health/HardwareStatus/HardwareStatusTableProcessors.vue
@@ -34,12 +34,12 @@
         <b-button
           variant="link"
           data-test-id="hardwareStatus-button-expandProcessors"
-          :aria-label="expandRowLabel"
           :title="expandRowLabel"
           class="btn-icon-only"
           @click="toggleRowDetails(row)"
         >
           <icon-chevron />
+          <span class="sr-only">{{ expandRowLabel }}</span>
         </b-button>
       </template>
       <!-- Health -->
diff --git a/src/views/Health/HardwareStatus/HardwareStatusTableStystem.vue b/src/views/Health/HardwareStatus/HardwareStatusTableStystem.vue
index a89dacfe..d4087016 100644
--- a/src/views/Health/HardwareStatus/HardwareStatusTableStystem.vue
+++ b/src/views/Health/HardwareStatus/HardwareStatusTableStystem.vue
@@ -13,12 +13,12 @@
         <b-button
           variant="link"
           data-test-id="hardwareStatus-button-expandSystem"
-          :aria-label="expandRowLabel"
           :title="expandRowLabel"
           class="btn-icon-only"
           @click="toggleRowDetails(row)"
         >
           <icon-chevron />
+          <span class="sr-only">{{ expandRowLabel }}</span>
         </b-button>
       </template>
 
-- 
cgit v1.2.3