npm - @patternfly/react-core - Versions diffs - 5.0.0-alpha.26 → 5.0.0-alpha.28 - Mend

@patternfly/react-core 5.0.0-alpha.26 → 5.0.0-alpha.28

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (292) hide show

package/src/components/ProgressStepper/examples/ProgressStepperHelpPopover.tsx CHANGED Viewed

@@ -2,7 +2,7 @@ import React from 'react';
 import { ProgressStepper, ProgressStep, Popover } from '@patternfly/react-core';
 export const PopoverProgressStep = () => (
-  <ProgressStepper>
+  <ProgressStepper aria-label="Progress stepper with help popover">
     <ProgressStep
       variant="success"
       id="popover-step1"

package/src/components/SimpleList/SimpleList.tsx CHANGED Viewed

@@ -76,7 +76,11 @@ export class SimpleList extends React.Component<SimpleListProps, SimpleListState
       >
         <div className={css(styles.simpleList, className)} {...props}>
           {isGrouped && children}
-          {!isGrouped && <ul aria-label={ariaLabel}>{children}</ul>}
+          {!isGrouped && (
+            <ul role="list" aria-label={ariaLabel}>
+              {children}
+            </ul>
+          )}
         </div>
       </SimpleListContext.Provider>
     );

package/src/components/SimpleList/SimpleListGroup.tsx CHANGED Viewed

@@ -27,7 +27,7 @@ export const SimpleListGroup: React.FunctionComponent<SimpleListGroupProps> = ({
     <h2 id={id} className={css(styles.simpleListTitle, titleClassName)} aria-hidden="true">
       {title}
     </h2>
-    <ul className={css(className)} aria-labelledby={id}>
+    <ul className={css(className)} role="list" aria-labelledby={id}>
       {children}
     </ul>
   </section>

package/src/components/SimpleList/__tests__/Generated/__snapshots__/SimpleListGroup.test.tsx.snap CHANGED Viewed

@@ -13,6 +13,7 @@ exports[`SimpleListGroup should match snapshot (auto-generated) 1`] = `
     <ul
       aria-labelledby="''"
       class="''"
+      role="list"
     >
       ReactNode
     </ul>

package/src/components/SimpleList/__tests__/__snapshots__/SimpleList.test.tsx.snap CHANGED Viewed

@@ -7,6 +7,7 @@ exports[`SimpleList renders aria-labelled content 1`] = `
   >
     <ul
       aria-label="This is a simple list"
+      role="list"
     >
       <li
         class=""
@@ -48,7 +49,9 @@ exports[`SimpleList renders content 1`] = `
   <div
     class="pf-c-simple-list"
   >
-    <ul>
+    <ul
+      role="list"
+    >
       <li
         class=""
       >
@@ -102,6 +105,7 @@ exports[`SimpleList renders grouped content 1`] = `
       <ul
         aria-labelledby=""
         class=""
+        role="list"
       >
         <li
           class=""

package/src/components/Wizard/WizardNav.tsx CHANGED Viewed

@@ -27,7 +27,11 @@ export const WizardNav: React.FunctionComponent<WizardNavProps> = ({
 }: WizardNavProps) => {
   const ouiaProps = useOUIAProps(WizardNav.displayName, ouiaId, ouiaSafe);
-  const innerList = <ol className={css(styles.wizardNavList)}>{children}</ol>;
+  const innerList = (
+    <ol className={css(styles.wizardNavList)} role="list">
+      {children}
+    </ol>
+  );
   if (returnList) {
     return innerList;
@@ -40,7 +44,9 @@ export const WizardNav: React.FunctionComponent<WizardNavProps> = ({
       aria-labelledby={ariaLabelledBy}
       {...ouiaProps}
     >
-      <ol className={css(styles.wizardNavList)}>{children}</ol>
+      <ol className={css(styles.wizardNavList)} role="list">
+        {children}
+      </ol>
     </nav>
   );
 };

package/src/components/Wizard/__tests__/Generated/__snapshots__/WizardNav.test.tsx.snap CHANGED Viewed

@@ -12,6 +12,7 @@ exports[`WizardNav should match snapshot (auto-generated) 1`] = `
   >
     <ol
       class="pf-c-wizard__nav-list"
+      role="list"
     >
       any
     </ol>

package/src/components/Wizard/__tests__/__snapshots__/Wizard.test.tsx.snap CHANGED Viewed

@@ -100,6 +100,7 @@ exports[`Wizard Expandable Nav Wizard should match snapshot 1`] = `
         >
           <ol
             class="pf-c-wizard__nav-list"
+            role="list"
           >
             <li
               class="pf-c-wizard__nav-item"
@@ -154,6 +155,7 @@ exports[`Wizard Expandable Nav Wizard should match snapshot 1`] = `
               </button>
               <ol
                 class="pf-c-wizard__nav-list"
+                role="list"
               >
                 <li
                   class="pf-c-wizard__nav-item"
@@ -367,6 +369,7 @@ exports[`Wizard Wizard should match snapshot 1`] = `
         >
           <ol
             class="pf-c-wizard__nav-list"
+            role="list"
           >
             <li
               class="pf-c-wizard__nav-item"
@@ -397,6 +400,7 @@ exports[`Wizard Wizard should match snapshot 1`] = `
               </button>
               <ol
                 class="pf-c-wizard__nav-list"
+                role="list"
               >
                 <li
                   class="pf-c-wizard__nav-item"
@@ -571,6 +575,7 @@ exports[`Wizard bare wiz 1`] = `
         >
           <ol
             class="pf-c-wizard__nav-list"
+            role="list"
           >
             <li
               class="pf-c-wizard__nav-item"

package/src/next/components/Wizard/WizardNav.tsx CHANGED Viewed

@@ -23,7 +23,11 @@ export const WizardNav: React.FunctionComponent<WizardNavProps> = ({
   isInnerList = false
 }: WizardNavProps) => {
   if (isInnerList) {
-    return <ol className={css(styles.wizardNavList)}>{children}</ol>;
+    return (
+      <ol className={css(styles.wizardNavList)} role="list">
+        {children}
+      </ol>
+    );
   }
   return (
@@ -32,7 +36,9 @@ export const WizardNav: React.FunctionComponent<WizardNavProps> = ({
       aria-label={ariaLabel}
       aria-labelledby={ariaLabelledBy}
     >
-      <ol className={css(styles.wizardNavList)}>{children}</ol>
+      <ol className={css(styles.wizardNavList)} role="list">
+        {children}
+      </ol>
     </nav>
   );
 };