Tests corrections

Author: KonradPietocha

src/command-pallette/CommandPalletteUI/components/components.tsx

@@ -90,8 +90,8 @@ export function CommandPalletteHelp({
       <table className="help-text">
         <thead>
           <tr>
-            <th scope="col">{t('common.headers.name')}</th>
-            <th scope="col">{t('common.headers.description')}</th>
+            <th>{t('common.headers.name')}</th>
+            <th>{t('common.headers.description')}</th>
           </tr>
         </thead>
         <tbody>

src/shared/components/GenericList/GenericList.js

@@ -196,7 +196,7 @@ export const GenericList = ({
     if (serverDataError) {
       return (
         <BodyFallback key="tableErrorMessage">
-          <p role="text">{getErrorMessage(serverDataError)}</p>
+          <p>{getErrorMessage(serverDataError)}</p>
         </BodyFallback>
       );
     }
@@ -212,7 +212,7 @@ export const GenericList = ({
       if (searchQuery) {
         return (
           <BodyFallback>
-            <p role="text">
+            <p>
               {i18n.exists(searchSettings.noSearchResultMessage)
                 ? t(searchSettings.noSearchResultMessage)
                 : searchSettings.noSearchResultMessage}
@@ -236,7 +236,7 @@ export const GenericList = ({
                 image={emptyListProps?.image}
               />
             ) : (
-              <p role="text">
+              <p>
                 {emptyListProps?.titleText ? (
                   <Trans i18nKey={emptyListProps?.titleText} />
                 ) : i18n.exists(notFoundMessage) ? (

src/shared/components/GenericList/Pagination/Pagination.js

@@ -110,6 +110,7 @@ export const Pagination = ({
           isInteractable={currentPage !== 1}
           onClick={() => onChangePage(1)}
           accessibleName="First page"
+          accesskey="first-page-link"
         >
           <Icon
             accessibleName="first page icon"
@@ -122,6 +123,7 @@ export const Pagination = ({
           isInteractable={currentPage !== 1}
           onClick={() => onChangePage(currentPage - 1)}
           accessibleName="Previous page"
+          accesskey="previous-page-link"
         >
           <Icon
             accessibleName="previous page icon"
@@ -170,6 +172,7 @@ export const Pagination = ({
           isInteractable={currentPage !== pagesCount}
           onClick={() => onChangePage(currentPage + 1)}
           accessibleName="Next page"
+          accesskey="next-page-link"
         >
           <Icon
             accessibleName="next page icon"
@@ -182,6 +185,7 @@ export const Pagination = ({
           isInteractable={currentPage !== pagesCount}
           onClick={() => onChangePage(pagesCount)}
           accessibleName="Last page"
+          accesskey="last-page-link"
         >
           <Icon
             accessibleName="last page icon"

src/shared/components/GenericList/components.js

@@ -41,6 +41,7 @@ export const HeaderRenderer = ({
         importance={0}
         popinHidden={true}
         key="actions-column"
+        scope="col"
         aria-label="actions-column"
         minWidth={`${30 * actions.length}px`}
       >
@@ -77,6 +78,7 @@ export const HeaderRenderer = ({
         {headerRenderer()?.map((h, index) => {
           return (
             <TableHeaderCell
+              scope="col"
               key={`${typeof h === 'object' ? index : h}-column`}
               popinText={h === 'Popin' ? t('common.headers.specification') : h}
               popinHidden={h !== 'Popin' && !noHideFields?.includes(h)}

src/shared/components/PodTemplate/components.js

@@ -29,7 +29,7 @@ function PodTemplateTable({ className, items, columns, rowRenderer }) {
       headerRow={
         <TableHeaderRow style={{ width: '50%' }}>
           {columns.map(column => (
-            <TableHeaderCell>
+            <TableHeaderCell scope="col" key={column}>
               <Title level="H5">{column}</Title>
             </TableHeaderCell>
           ))}