@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.
- package/CHANGELOG.md +12 -0
- package/CONTRIBUTING.md +12 -1
- package/components/AlertGroup/AlertGroup.d.ts +2 -0
- package/components/AlertGroup/AlertGroup.d.ts.map +1 -1
- package/components/AlertGroup/AlertGroup.js +2 -2
- package/components/AlertGroup/AlertGroup.js.map +1 -1
- package/components/AlertGroup/AlertGroupInline.d.ts.map +1 -1
- package/components/AlertGroup/AlertGroupInline.js +1 -1
- package/components/AlertGroup/AlertGroupInline.js.map +1 -1
- package/components/Breadcrumb/Breadcrumb.js +1 -1
- package/components/Breadcrumb/Breadcrumb.js.map +1 -1
- package/components/ContextSelector/ContextSelector.d.ts +2 -0
- package/components/ContextSelector/ContextSelector.d.ts.map +1 -1
- package/components/ContextSelector/ContextSelector.js +2 -2
- package/components/ContextSelector/ContextSelector.js.map +1 -1
- package/components/ContextSelector/ContextSelectorMenuList.d.ts +5 -3
- package/components/ContextSelector/ContextSelectorMenuList.d.ts.map +1 -1
- package/components/ContextSelector/ContextSelectorMenuList.js.map +1 -1
- package/components/DataList/DataList.d.ts.map +1 -1
- package/components/DataList/DataList.js +1 -1
- package/components/DataList/DataList.js.map +1 -1
- package/components/HelperText/HelperText.d.ts +2 -0
- package/components/HelperText/HelperText.d.ts.map +1 -1
- package/components/HelperText/HelperText.js +2 -2
- package/components/HelperText/HelperText.js.map +1 -1
- package/components/JumpLinks/JumpLinks.d.ts +1 -1
- package/components/JumpLinks/JumpLinks.d.ts.map +1 -1
- package/components/JumpLinks/JumpLinks.js +2 -5
- package/components/JumpLinks/JumpLinks.js.map +1 -1
- package/components/JumpLinks/JumpLinksList.js +1 -1
- package/components/JumpLinks/JumpLinksList.js.map +1 -1
- package/components/List/List.d.ts +4 -1
- package/components/List/List.d.ts.map +1 -1
- package/components/List/List.js +1 -1
- package/components/List/List.js.map +1 -1
- package/components/LoginPage/LoginMainFooter.d.ts +3 -1
- package/components/LoginPage/LoginMainFooter.d.ts.map +1 -1
- package/components/LoginPage/LoginMainFooter.js +2 -2
- package/components/LoginPage/LoginMainFooter.js.map +1 -1
- package/components/LoginPage/LoginPage.d.ts +2 -0
- package/components/LoginPage/LoginPage.d.ts.map +1 -1
- package/components/LoginPage/LoginPage.js +2 -2
- package/components/LoginPage/LoginPage.js.map +1 -1
- package/components/MultipleFileUpload/MultipleFileUploadStatus.d.ts +2 -0
- package/components/MultipleFileUpload/MultipleFileUploadStatus.d.ts.map +1 -1
- package/components/MultipleFileUpload/MultipleFileUploadStatus.js +2 -2
- package/components/MultipleFileUpload/MultipleFileUploadStatus.js.map +1 -1
- package/components/Nav/NavExpandable.d.ts.map +1 -1
- package/components/Nav/NavExpandable.js +4 -4
- package/components/Nav/NavExpandable.js.map +1 -1
- package/components/Nav/NavGroup.d.ts.map +1 -1
- package/components/Nav/NavGroup.js +1 -1
- package/components/Nav/NavGroup.js.map +1 -1
- package/components/Nav/NavList.d.ts.map +1 -1
- package/components/Nav/NavList.js +1 -1
- package/components/Nav/NavList.js.map +1 -1
- package/components/NotificationDrawer/NotificationDrawerList.d.ts +2 -0
- package/components/NotificationDrawer/NotificationDrawerList.d.ts.map +1 -1
- package/components/NotificationDrawer/NotificationDrawerList.js +2 -2
- package/components/NotificationDrawer/NotificationDrawerList.js.map +1 -1
- package/components/ProgressStepper/ProgressStep.d.ts.map +1 -1
- package/components/ProgressStepper/ProgressStep.js +3 -1
- package/components/ProgressStepper/ProgressStep.js.map +1 -1
- package/components/ProgressStepper/ProgressStepper.d.ts +2 -0
- package/components/ProgressStepper/ProgressStepper.d.ts.map +1 -1
- package/components/ProgressStepper/ProgressStepper.js +2 -2
- package/components/ProgressStepper/ProgressStepper.js.map +1 -1
- package/components/SimpleList/SimpleList.d.ts.map +1 -1
- package/components/SimpleList/SimpleList.js +1 -1
- package/components/SimpleList/SimpleList.js.map +1 -1
- package/components/SimpleList/SimpleListGroup.js +1 -1
- package/components/SimpleList/SimpleListGroup.js.map +1 -1
- package/components/Wizard/WizardNav.d.ts.map +1 -1
- package/components/Wizard/WizardNav.js +2 -2
- package/components/Wizard/WizardNav.js.map +1 -1
- package/dist/esm/components/AlertGroup/AlertGroup.d.ts +2 -0
- package/dist/esm/components/AlertGroup/AlertGroup.d.ts.map +1 -1
- package/dist/esm/components/AlertGroup/AlertGroup.js +2 -2
- package/dist/esm/components/AlertGroup/AlertGroup.js.map +1 -1
- package/dist/esm/components/AlertGroup/AlertGroupInline.d.ts.map +1 -1
- package/dist/esm/components/AlertGroup/AlertGroupInline.js +1 -1
- package/dist/esm/components/AlertGroup/AlertGroupInline.js.map +1 -1
- package/dist/esm/components/Breadcrumb/Breadcrumb.js +1 -1
- package/dist/esm/components/Breadcrumb/Breadcrumb.js.map +1 -1
- package/dist/esm/components/ContextSelector/ContextSelector.d.ts +2 -0
- package/dist/esm/components/ContextSelector/ContextSelector.d.ts.map +1 -1
- package/dist/esm/components/ContextSelector/ContextSelector.js +2 -2
- package/dist/esm/components/ContextSelector/ContextSelector.js.map +1 -1
- package/dist/esm/components/ContextSelector/ContextSelectorMenuList.d.ts +5 -3
- package/dist/esm/components/ContextSelector/ContextSelectorMenuList.d.ts.map +1 -1
- package/dist/esm/components/ContextSelector/ContextSelectorMenuList.js.map +1 -1
- package/dist/esm/components/DataList/DataList.d.ts.map +1 -1
- package/dist/esm/components/DataList/DataList.js +1 -1
- package/dist/esm/components/DataList/DataList.js.map +1 -1
- package/dist/esm/components/HelperText/HelperText.d.ts +2 -0
- package/dist/esm/components/HelperText/HelperText.d.ts.map +1 -1
- package/dist/esm/components/HelperText/HelperText.js +2 -2
- package/dist/esm/components/HelperText/HelperText.js.map +1 -1
- package/dist/esm/components/JumpLinks/JumpLinks.d.ts +1 -1
- package/dist/esm/components/JumpLinks/JumpLinks.d.ts.map +1 -1
- package/dist/esm/components/JumpLinks/JumpLinks.js +2 -5
- package/dist/esm/components/JumpLinks/JumpLinks.js.map +1 -1
- package/dist/esm/components/JumpLinks/JumpLinksList.js +1 -1
- package/dist/esm/components/JumpLinks/JumpLinksList.js.map +1 -1
- package/dist/esm/components/List/List.d.ts +4 -1
- package/dist/esm/components/List/List.d.ts.map +1 -1
- package/dist/esm/components/List/List.js +1 -1
- package/dist/esm/components/List/List.js.map +1 -1
- package/dist/esm/components/LoginPage/LoginMainFooter.d.ts +3 -1
- package/dist/esm/components/LoginPage/LoginMainFooter.d.ts.map +1 -1
- package/dist/esm/components/LoginPage/LoginMainFooter.js +2 -2
- package/dist/esm/components/LoginPage/LoginMainFooter.js.map +1 -1
- package/dist/esm/components/LoginPage/LoginPage.d.ts +2 -0
- package/dist/esm/components/LoginPage/LoginPage.d.ts.map +1 -1
- package/dist/esm/components/LoginPage/LoginPage.js +2 -2
- package/dist/esm/components/LoginPage/LoginPage.js.map +1 -1
- package/dist/esm/components/MultipleFileUpload/MultipleFileUploadStatus.d.ts +2 -0
- package/dist/esm/components/MultipleFileUpload/MultipleFileUploadStatus.d.ts.map +1 -1
- package/dist/esm/components/MultipleFileUpload/MultipleFileUploadStatus.js +2 -2
- package/dist/esm/components/MultipleFileUpload/MultipleFileUploadStatus.js.map +1 -1
- package/dist/esm/components/Nav/NavExpandable.d.ts.map +1 -1
- package/dist/esm/components/Nav/NavExpandable.js +4 -4
- package/dist/esm/components/Nav/NavExpandable.js.map +1 -1
- package/dist/esm/components/Nav/NavGroup.d.ts.map +1 -1
- package/dist/esm/components/Nav/NavGroup.js +1 -1
- package/dist/esm/components/Nav/NavGroup.js.map +1 -1
- package/dist/esm/components/Nav/NavList.d.ts.map +1 -1
- package/dist/esm/components/Nav/NavList.js +1 -1
- package/dist/esm/components/Nav/NavList.js.map +1 -1
- package/dist/esm/components/NotificationDrawer/NotificationDrawerList.d.ts +2 -0
- package/dist/esm/components/NotificationDrawer/NotificationDrawerList.d.ts.map +1 -1
- package/dist/esm/components/NotificationDrawer/NotificationDrawerList.js +2 -2
- package/dist/esm/components/NotificationDrawer/NotificationDrawerList.js.map +1 -1
- package/dist/esm/components/ProgressStepper/ProgressStep.d.ts.map +1 -1
- package/dist/esm/components/ProgressStepper/ProgressStep.js +3 -1
- package/dist/esm/components/ProgressStepper/ProgressStep.js.map +1 -1
- package/dist/esm/components/ProgressStepper/ProgressStepper.d.ts +2 -0
- package/dist/esm/components/ProgressStepper/ProgressStepper.d.ts.map +1 -1
- package/dist/esm/components/ProgressStepper/ProgressStepper.js +2 -2
- package/dist/esm/components/ProgressStepper/ProgressStepper.js.map +1 -1
- package/dist/esm/components/SimpleList/SimpleList.d.ts.map +1 -1
- package/dist/esm/components/SimpleList/SimpleList.js +1 -1
- package/dist/esm/components/SimpleList/SimpleList.js.map +1 -1
- package/dist/esm/components/SimpleList/SimpleListGroup.js +1 -1
- package/dist/esm/components/SimpleList/SimpleListGroup.js.map +1 -1
- package/dist/esm/components/Wizard/WizardNav.d.ts.map +1 -1
- package/dist/esm/components/Wizard/WizardNav.js +2 -2
- package/dist/esm/components/Wizard/WizardNav.js.map +1 -1
- package/dist/esm/next/components/Wizard/WizardNav.d.ts.map +1 -1
- package/dist/esm/next/components/Wizard/WizardNav.js +2 -2
- package/dist/esm/next/components/Wizard/WizardNav.js.map +1 -1
- package/dist/js/components/AlertGroup/AlertGroup.d.ts +2 -0
- package/dist/js/components/AlertGroup/AlertGroup.d.ts.map +1 -1
- package/dist/js/components/AlertGroup/AlertGroup.js +2 -2
- package/dist/js/components/AlertGroup/AlertGroup.js.map +1 -1
- package/dist/js/components/AlertGroup/AlertGroupInline.d.ts.map +1 -1
- package/dist/js/components/AlertGroup/AlertGroupInline.js +1 -1
- package/dist/js/components/AlertGroup/AlertGroupInline.js.map +1 -1
- package/dist/js/components/Breadcrumb/Breadcrumb.js +1 -1
- package/dist/js/components/Breadcrumb/Breadcrumb.js.map +1 -1
- package/dist/js/components/ContextSelector/ContextSelector.d.ts +2 -0
- package/dist/js/components/ContextSelector/ContextSelector.d.ts.map +1 -1
- package/dist/js/components/ContextSelector/ContextSelector.js +2 -2
- package/dist/js/components/ContextSelector/ContextSelector.js.map +1 -1
- package/dist/js/components/ContextSelector/ContextSelectorMenuList.d.ts +5 -3
- package/dist/js/components/ContextSelector/ContextSelectorMenuList.d.ts.map +1 -1
- package/dist/js/components/ContextSelector/ContextSelectorMenuList.js.map +1 -1
- package/dist/js/components/DataList/DataList.d.ts.map +1 -1
- package/dist/js/components/DataList/DataList.js +1 -1
- package/dist/js/components/DataList/DataList.js.map +1 -1
- package/dist/js/components/HelperText/HelperText.d.ts +2 -0
- package/dist/js/components/HelperText/HelperText.d.ts.map +1 -1
- package/dist/js/components/HelperText/HelperText.js +2 -2
- package/dist/js/components/HelperText/HelperText.js.map +1 -1
- package/dist/js/components/JumpLinks/JumpLinks.d.ts +1 -1
- package/dist/js/components/JumpLinks/JumpLinks.d.ts.map +1 -1
- package/dist/js/components/JumpLinks/JumpLinks.js +2 -5
- package/dist/js/components/JumpLinks/JumpLinks.js.map +1 -1
- package/dist/js/components/JumpLinks/JumpLinksList.js +1 -1
- package/dist/js/components/JumpLinks/JumpLinksList.js.map +1 -1
- package/dist/js/components/List/List.d.ts +4 -1
- package/dist/js/components/List/List.d.ts.map +1 -1
- package/dist/js/components/List/List.js +1 -1
- package/dist/js/components/List/List.js.map +1 -1
- package/dist/js/components/LoginPage/LoginMainFooter.d.ts +3 -1
- package/dist/js/components/LoginPage/LoginMainFooter.d.ts.map +1 -1
- package/dist/js/components/LoginPage/LoginMainFooter.js +2 -2
- package/dist/js/components/LoginPage/LoginMainFooter.js.map +1 -1
- package/dist/js/components/LoginPage/LoginPage.d.ts +2 -0
- package/dist/js/components/LoginPage/LoginPage.d.ts.map +1 -1
- package/dist/js/components/LoginPage/LoginPage.js +2 -2
- package/dist/js/components/LoginPage/LoginPage.js.map +1 -1
- package/dist/js/components/MultipleFileUpload/MultipleFileUploadStatus.d.ts +2 -0
- package/dist/js/components/MultipleFileUpload/MultipleFileUploadStatus.d.ts.map +1 -1
- package/dist/js/components/MultipleFileUpload/MultipleFileUploadStatus.js +2 -2
- package/dist/js/components/MultipleFileUpload/MultipleFileUploadStatus.js.map +1 -1
- package/dist/js/components/Nav/NavExpandable.d.ts.map +1 -1
- package/dist/js/components/Nav/NavExpandable.js +4 -4
- package/dist/js/components/Nav/NavExpandable.js.map +1 -1
- package/dist/js/components/Nav/NavGroup.d.ts.map +1 -1
- package/dist/js/components/Nav/NavGroup.js +1 -1
- package/dist/js/components/Nav/NavGroup.js.map +1 -1
- package/dist/js/components/Nav/NavList.d.ts.map +1 -1
- package/dist/js/components/Nav/NavList.js +1 -1
- package/dist/js/components/Nav/NavList.js.map +1 -1
- package/dist/js/components/NotificationDrawer/NotificationDrawerList.d.ts +2 -0
- package/dist/js/components/NotificationDrawer/NotificationDrawerList.d.ts.map +1 -1
- package/dist/js/components/NotificationDrawer/NotificationDrawerList.js +2 -2
- package/dist/js/components/NotificationDrawer/NotificationDrawerList.js.map +1 -1
- package/dist/js/components/ProgressStepper/ProgressStep.d.ts.map +1 -1
- package/dist/js/components/ProgressStepper/ProgressStep.js +3 -1
- package/dist/js/components/ProgressStepper/ProgressStep.js.map +1 -1
- package/dist/js/components/ProgressStepper/ProgressStepper.d.ts +2 -0
- package/dist/js/components/ProgressStepper/ProgressStepper.d.ts.map +1 -1
- package/dist/js/components/ProgressStepper/ProgressStepper.js +2 -2
- package/dist/js/components/ProgressStepper/ProgressStepper.js.map +1 -1
- package/dist/js/components/SimpleList/SimpleList.d.ts.map +1 -1
- package/dist/js/components/SimpleList/SimpleList.js +1 -1
- package/dist/js/components/SimpleList/SimpleList.js.map +1 -1
- package/dist/js/components/SimpleList/SimpleListGroup.js +1 -1
- package/dist/js/components/SimpleList/SimpleListGroup.js.map +1 -1
- package/dist/js/components/Wizard/WizardNav.d.ts.map +1 -1
- package/dist/js/components/Wizard/WizardNav.js +2 -2
- package/dist/js/components/Wizard/WizardNav.js.map +1 -1
- package/dist/js/next/components/Wizard/WizardNav.d.ts.map +1 -1
- package/dist/js/next/components/Wizard/WizardNav.js +2 -2
- package/dist/js/next/components/Wizard/WizardNav.js.map +1 -1
- package/dist/umd/react-core.min.js +1 -1
- package/next/components/Wizard/WizardNav.d.ts.map +1 -1
- package/next/components/Wizard/WizardNav.js +2 -2
- package/next/components/Wizard/WizardNav.js.map +1 -1
- package/package.json +2 -2
- package/src/components/AlertGroup/AlertGroup.tsx +13 -1
- package/src/components/AlertGroup/AlertGroupInline.tsx +1 -0
- package/src/components/AlertGroup/__tests__/Generated/__snapshots__/AlertGroupInline.test.tsx.snap +1 -0
- package/src/components/AlertGroup/__tests__/__snapshots__/AlertGroup.test.tsx.snap +1 -0
- package/src/components/Breadcrumb/Breadcrumb.tsx +1 -1
- package/src/components/Breadcrumb/__tests__/__snapshots__/Breadcrumb.test.tsx.snap +4 -0
- package/src/components/ContextSelector/ContextSelector.tsx +6 -1
- package/src/components/ContextSelector/ContextSelectorMenuList.tsx +5 -3
- package/src/components/DataList/DataList.tsx +1 -0
- package/src/components/DataList/__tests__/__snapshots__/DataList.test.tsx.snap +11 -0
- package/src/components/Divider/examples/DividerUsingLi.tsx +1 -1
- package/src/components/HelperText/HelperText.tsx +4 -0
- package/src/components/HelperText/__tests__/__snapshots__/HelperText.test.tsx.snap +1 -0
- package/src/components/JumpLinks/JumpLinks.tsx +5 -8
- package/src/components/JumpLinks/JumpLinksList.tsx +1 -1
- package/src/components/JumpLinks/__tests__/__snapshots__/JumpLinks.test.tsx.snap +6 -0
- package/src/components/List/List.tsx +6 -1
- package/src/components/List/__tests__/__snapshots__/List.test.tsx.snap +1 -0
- package/src/components/LoginPage/LoginMainFooter.tsx +9 -2
- package/src/components/LoginPage/LoginPage.tsx +4 -0
- package/src/components/LoginPage/__tests__/__snapshots__/LoginPage.test.tsx.snap +1 -0
- package/src/components/LoginPage/examples/LoginPageBasic.tsx +1 -0
- package/src/components/LoginPage/examples/LoginPageLanguageSelect.tsx +1 -0
- package/src/components/LoginPage/examples/LoginPageShowHidePassword.tsx +1 -0
- package/src/components/MultipleFileUpload/MultipleFileUploadStatus.tsx +6 -1
- package/src/components/MultipleFileUpload/__tests__/__snapshots__/MultipleFileUploadStatus.test.tsx.snap +4 -0
- package/src/components/MultipleFileUpload/examples/MultipleFileUploadBasic.tsx +15 -14
- package/src/components/Nav/NavExpandable.tsx +6 -4
- package/src/components/Nav/NavGroup.tsx +3 -1
- package/src/components/Nav/NavList.tsx +1 -0
- package/src/components/Nav/__tests__/Generated/__snapshots__/NavExpandable.test.tsx.snap +1 -0
- package/src/components/Nav/__tests__/Generated/__snapshots__/NavGroup.test.tsx.snap +1 -0
- package/src/components/Nav/__tests__/Generated/__snapshots__/NavList.test.tsx.snap +1 -0
- package/src/components/Nav/__tests__/__snapshots__/Nav.test.tsx.snap +19 -0
- package/src/components/NotificationDrawer/NotificationDrawerList.tsx +10 -1
- package/src/components/NotificationDrawer/__tests__/__snapshots__/NotificationDrawerList.test.tsx.snap +2 -0
- package/src/components/NotificationDrawer/examples/NotificationDrawerBasic.tsx +1 -1
- package/src/components/NotificationDrawer/examples/NotificationDrawerGroups.tsx +3 -3
- package/src/components/NotificationDrawer/examples/NotificationDrawerLightweight.tsx +12 -3
- package/src/components/Page/__tests__/__snapshots__/Page.test.tsx.snap +12 -0
- package/src/components/ProgressStepper/ProgressStep.tsx +2 -0
- package/src/components/ProgressStepper/ProgressStepper.tsx +5 -0
- package/src/components/ProgressStepper/__tests__/__snapshots__/ProgressStep.test.tsx.snap +1 -0
- package/src/components/ProgressStepper/__tests__/__snapshots__/ProgressStepper.test.tsx.snap +1 -0
- package/src/components/ProgressStepper/examples/ProgressStepperBasic.tsx +1 -1
- package/src/components/ProgressStepper/examples/ProgressStepperBasicFailure.tsx +1 -1
- package/src/components/ProgressStepper/examples/ProgressStepperBasicIssue.tsx +1 -1
- package/src/components/ProgressStepper/examples/ProgressStepperBasicWithAlignment.tsx +5 -1
- package/src/components/ProgressStepper/examples/ProgressStepperBasicWithDescription.tsx +1 -1
- package/src/components/ProgressStepper/examples/ProgressStepperCompact.tsx +1 -1
- package/src/components/ProgressStepper/examples/ProgressStepperCustomIcons.tsx +1 -1
- package/src/components/ProgressStepper/examples/ProgressStepperHelpPopover.tsx +1 -1
- package/src/components/SimpleList/SimpleList.tsx +5 -1
- package/src/components/SimpleList/SimpleListGroup.tsx +1 -1
- package/src/components/SimpleList/__tests__/Generated/__snapshots__/SimpleListGroup.test.tsx.snap +1 -0
- package/src/components/SimpleList/__tests__/__snapshots__/SimpleList.test.tsx.snap +5 -1
- package/src/components/Wizard/WizardNav.tsx +8 -2
- package/src/components/Wizard/__tests__/Generated/__snapshots__/WizardNav.test.tsx.snap +1 -0
- package/src/components/Wizard/__tests__/__snapshots__/Wizard.test.tsx.snap +5 -0
- package/src/next/components/Wizard/WizardNav.tsx +8 -2
|
@@ -34,7 +34,7 @@ export const MultipleFileUploadBasic: React.FunctionComponent = () => {
|
|
|
34
34
|
React.useEffect(() => {
|
|
35
35
|
if (readFileData.length < currentFiles.length) {
|
|
36
36
|
setStatusIcon('inProgress');
|
|
37
|
-
} else if (readFileData.every(file => file.loadResult === 'success')) {
|
|
37
|
+
} else if (readFileData.every((file) => file.loadResult === 'success')) {
|
|
38
38
|
setStatusIcon('success');
|
|
39
39
|
} else {
|
|
40
40
|
setStatusIcon('danger');
|
|
@@ -44,13 +44,13 @@ export const MultipleFileUploadBasic: React.FunctionComponent = () => {
|
|
|
44
44
|
// remove files from both state arrays based on their name
|
|
45
45
|
const removeFiles = (namesOfFilesToRemove: string[]) => {
|
|
46
46
|
const newCurrentFiles = currentFiles.filter(
|
|
47
|
-
currentFile => !namesOfFilesToRemove.some(fileName => fileName === currentFile.name)
|
|
47
|
+
(currentFile) => !namesOfFilesToRemove.some((fileName) => fileName === currentFile.name)
|
|
48
48
|
);
|
|
49
49
|
|
|
50
50
|
setCurrentFiles(newCurrentFiles);
|
|
51
51
|
|
|
52
52
|
const newReadFiles = readFileData.filter(
|
|
53
|
-
readFile => !namesOfFilesToRemove.some(fileName => fileName === readFile.fileName)
|
|
53
|
+
(readFile) => !namesOfFilesToRemove.some((fileName) => fileName === readFile.fileName)
|
|
54
54
|
);
|
|
55
55
|
|
|
56
56
|
setReadFileData(newReadFiles);
|
|
@@ -60,34 +60,34 @@ export const MultipleFileUploadBasic: React.FunctionComponent = () => {
|
|
|
60
60
|
* only used in this example for demonstration purposes */
|
|
61
61
|
const updateCurrentFiles = (files: File[]) => {
|
|
62
62
|
if (fileUploadShouldFail) {
|
|
63
|
-
const corruptedFiles = files.map(file => ({ ...file, lastModified:
|
|
64
|
-
setCurrentFiles(prevFiles => [...prevFiles, ...corruptedFiles]);
|
|
63
|
+
const corruptedFiles = files.map((file) => ({ ...file, lastModified: 'foo' as unknown as number }));
|
|
64
|
+
setCurrentFiles((prevFiles) => [...prevFiles, ...corruptedFiles]);
|
|
65
65
|
} else {
|
|
66
|
-
setCurrentFiles(prevFiles => [...prevFiles, ...files]);
|
|
66
|
+
setCurrentFiles((prevFiles) => [...prevFiles, ...files]);
|
|
67
67
|
}
|
|
68
68
|
};
|
|
69
69
|
|
|
70
70
|
// callback that will be called by the react dropzone with the newly dropped file objects
|
|
71
71
|
const handleFileDrop = (droppedFiles: File[]) => {
|
|
72
72
|
// identify what, if any, files are re-uploads of already uploaded files
|
|
73
|
-
const currentFileNames = currentFiles.map(file => file.name);
|
|
74
|
-
const reUploads = droppedFiles.filter(droppedFile => currentFileNames.includes(droppedFile.name));
|
|
73
|
+
const currentFileNames = currentFiles.map((file) => file.name);
|
|
74
|
+
const reUploads = droppedFiles.filter((droppedFile) => currentFileNames.includes(droppedFile.name));
|
|
75
75
|
|
|
76
76
|
/** this promise chain is needed because if the file removal is done at the same time as the file adding react
|
|
77
77
|
* won't realize that the status items for the re-uploaded files needs to be re-rendered */
|
|
78
78
|
Promise.resolve()
|
|
79
|
-
.then(() => removeFiles(reUploads.map(file => file.name)))
|
|
79
|
+
.then(() => removeFiles(reUploads.map((file) => file.name)))
|
|
80
80
|
.then(() => updateCurrentFiles(droppedFiles));
|
|
81
81
|
};
|
|
82
82
|
|
|
83
83
|
// callback called by the status item when a file is successfully read with the built-in file reader
|
|
84
84
|
const handleReadSuccess = (data: string, file: File) => {
|
|
85
|
-
setReadFileData(prevReadFiles => [...prevReadFiles, { data, fileName: file.name, loadResult: 'success' }]);
|
|
85
|
+
setReadFileData((prevReadFiles) => [...prevReadFiles, { data, fileName: file.name, loadResult: 'success' }]);
|
|
86
86
|
};
|
|
87
87
|
|
|
88
88
|
// callback called by the status item when a file encounters an error while being read with the built-in file reader
|
|
89
89
|
const handleReadFail = (error: DOMException, file: File) => {
|
|
90
|
-
setReadFileData(prevReadFiles => [
|
|
90
|
+
setReadFileData((prevReadFiles) => [
|
|
91
91
|
...prevReadFiles,
|
|
92
92
|
{ loadError: error, fileName: file.name, loadResult: 'danger' }
|
|
93
93
|
]);
|
|
@@ -95,7 +95,7 @@ export const MultipleFileUploadBasic: React.FunctionComponent = () => {
|
|
|
95
95
|
|
|
96
96
|
// add helper text to a status item showing any error encountered during the file reading process
|
|
97
97
|
const createHelperText = (file: File) => {
|
|
98
|
-
const fileResult = readFileData.find(readFile => readFile.fileName === file.name);
|
|
98
|
+
const fileResult = readFileData.find((readFile) => readFile.fileName === file.name);
|
|
99
99
|
if (fileResult?.loadError) {
|
|
100
100
|
return (
|
|
101
101
|
<HelperText isLiveRegion>
|
|
@@ -105,7 +105,7 @@ export const MultipleFileUploadBasic: React.FunctionComponent = () => {
|
|
|
105
105
|
}
|
|
106
106
|
};
|
|
107
107
|
|
|
108
|
-
const successfullyReadFileCount = readFileData.filter(fileData => fileData.loadResult === 'success').length;
|
|
108
|
+
const successfullyReadFileCount = readFileData.filter((fileData) => fileData.loadResult === 'success').length;
|
|
109
109
|
|
|
110
110
|
return (
|
|
111
111
|
<>
|
|
@@ -131,8 +131,9 @@ export const MultipleFileUploadBasic: React.FunctionComponent = () => {
|
|
|
131
131
|
<MultipleFileUploadStatus
|
|
132
132
|
statusToggleText={`${successfullyReadFileCount} of ${currentFiles.length} files uploaded`}
|
|
133
133
|
statusToggleIcon={statusIcon}
|
|
134
|
+
aria-label="Current uploads"
|
|
134
135
|
>
|
|
135
|
-
{currentFiles.map(file => (
|
|
136
|
+
{currentFiles.map((file) => (
|
|
136
137
|
<MultipleFileUploadStatusItem
|
|
137
138
|
file={file}
|
|
138
139
|
key={file.name}
|
|
@@ -82,7 +82,7 @@ export class NavExpandable extends React.Component<NavExpandableProps, NavExpand
|
|
|
82
82
|
if (this.props.onExpand) {
|
|
83
83
|
this.props.onExpand(e, !expandedState);
|
|
84
84
|
} else {
|
|
85
|
-
this.setState(prevState => ({ expandedState: !prevState.expandedState }));
|
|
85
|
+
this.setState((prevState) => ({ expandedState: !prevState.expandedState }));
|
|
86
86
|
const { groupId } = this.props;
|
|
87
87
|
onToggle(e, groupId, !expandedState);
|
|
88
88
|
}
|
|
@@ -112,7 +112,7 @@ export class NavExpandable extends React.Component<NavExpandableProps, NavExpand
|
|
|
112
112
|
|
|
113
113
|
return (
|
|
114
114
|
<NavContext.Consumer>
|
|
115
|
-
{context => (
|
|
115
|
+
{(context) => (
|
|
116
116
|
<li
|
|
117
117
|
className={css(
|
|
118
118
|
styles.navItem,
|
|
@@ -129,7 +129,7 @@ export class NavExpandable extends React.Component<NavExpandableProps, NavExpand
|
|
|
129
129
|
<button
|
|
130
130
|
className={styles.navLink}
|
|
131
131
|
id={srText ? null : this.id}
|
|
132
|
-
onClick={e => this.onExpand(e, context.onToggle)}
|
|
132
|
+
onClick={(e) => this.onExpand(e, context.onToggle)}
|
|
133
133
|
aria-expanded={expandedState}
|
|
134
134
|
tabIndex={isNavOpen ? null : -1}
|
|
135
135
|
{...buttonProps}
|
|
@@ -149,7 +149,9 @@ export class NavExpandable extends React.Component<NavExpandableProps, NavExpand
|
|
|
149
149
|
{srText}
|
|
150
150
|
</h2>
|
|
151
151
|
)}
|
|
152
|
-
<ul className={css(styles.navList)}>
|
|
152
|
+
<ul className={css(styles.navList)} role="list">
|
|
153
|
+
{children}
|
|
154
|
+
</ul>
|
|
153
155
|
</section>
|
|
154
156
|
</li>
|
|
155
157
|
)}
|
|
@@ -35,7 +35,9 @@ export const NavGroup: React.FunctionComponent<NavGroupProps> = ({
|
|
|
35
35
|
{title}
|
|
36
36
|
</h2>
|
|
37
37
|
)}
|
|
38
|
-
<ul className={css(styles.navList, className)}>
|
|
38
|
+
<ul className={css(styles.navList, className)} role="list">
|
|
39
|
+
{children}
|
|
40
|
+
</ul>
|
|
39
41
|
</section>
|
|
40
42
|
);
|
|
41
43
|
};
|
|
@@ -11,6 +11,7 @@ exports[`Nav Dark Nav List 1`] = `
|
|
|
11
11
|
>
|
|
12
12
|
<ul
|
|
13
13
|
class="pf-c-nav__list test-nav-list-class"
|
|
14
|
+
role="list"
|
|
14
15
|
>
|
|
15
16
|
<li
|
|
16
17
|
class="pf-c-nav__item test-nav-item-class"
|
|
@@ -80,6 +81,7 @@ exports[`Nav Default Nav List - Trigger item active update 1`] = `
|
|
|
80
81
|
>
|
|
81
82
|
<ul
|
|
82
83
|
class="pf-c-nav__list"
|
|
84
|
+
role="list"
|
|
83
85
|
>
|
|
84
86
|
<li
|
|
85
87
|
class="pf-c-nav__item"
|
|
@@ -149,6 +151,7 @@ exports[`Nav Default Nav List 1`] = `
|
|
|
149
151
|
>
|
|
150
152
|
<ul
|
|
151
153
|
class="pf-c-nav__list test-nav-list-class"
|
|
154
|
+
role="list"
|
|
152
155
|
>
|
|
153
156
|
<li
|
|
154
157
|
class="pf-c-nav__item test-nav-item-class"
|
|
@@ -218,6 +221,7 @@ exports[`Nav Expandable Nav List - Trigger toggle 1`] = `
|
|
|
218
221
|
>
|
|
219
222
|
<ul
|
|
220
223
|
class="pf-c-nav__list"
|
|
224
|
+
role="list"
|
|
221
225
|
>
|
|
222
226
|
<li
|
|
223
227
|
class="pf-c-nav__item pf-m-expandable expandable-group"
|
|
@@ -260,6 +264,7 @@ exports[`Nav Expandable Nav List - Trigger toggle 1`] = `
|
|
|
260
264
|
>
|
|
261
265
|
<ul
|
|
262
266
|
class="pf-c-nav__list"
|
|
267
|
+
role="list"
|
|
263
268
|
>
|
|
264
269
|
<li
|
|
265
270
|
class="pf-c-nav__item"
|
|
@@ -332,6 +337,7 @@ exports[`Nav Expandable Nav List 1`] = `
|
|
|
332
337
|
>
|
|
333
338
|
<ul
|
|
334
339
|
class="pf-c-nav__list"
|
|
340
|
+
role="list"
|
|
335
341
|
>
|
|
336
342
|
<li
|
|
337
343
|
class="pf-c-nav__item pf-m-expandable"
|
|
@@ -374,6 +380,7 @@ exports[`Nav Expandable Nav List 1`] = `
|
|
|
374
380
|
>
|
|
375
381
|
<ul
|
|
376
382
|
class="pf-c-nav__list"
|
|
383
|
+
role="list"
|
|
377
384
|
>
|
|
378
385
|
<li
|
|
379
386
|
class="pf-c-nav__item"
|
|
@@ -446,6 +453,7 @@ exports[`Nav Expandable Nav List with aria label 1`] = `
|
|
|
446
453
|
>
|
|
447
454
|
<ul
|
|
448
455
|
class="pf-c-nav__list"
|
|
456
|
+
role="list"
|
|
449
457
|
>
|
|
450
458
|
<li
|
|
451
459
|
class="pf-c-nav__item pf-m-expandable"
|
|
@@ -493,6 +501,7 @@ exports[`Nav Expandable Nav List with aria label 1`] = `
|
|
|
493
501
|
</h2>
|
|
494
502
|
<ul
|
|
495
503
|
class="pf-c-nav__list"
|
|
504
|
+
role="list"
|
|
496
505
|
>
|
|
497
506
|
<li
|
|
498
507
|
class="pf-c-nav__item"
|
|
@@ -584,6 +593,7 @@ exports[`Nav Horizontal Nav List 1`] = `
|
|
|
584
593
|
</button>
|
|
585
594
|
<ul
|
|
586
595
|
class="pf-c-nav__list"
|
|
596
|
+
role="list"
|
|
587
597
|
>
|
|
588
598
|
<li
|
|
589
599
|
class="pf-c-nav__item"
|
|
@@ -691,6 +701,7 @@ exports[`Nav Horizontal SubNav List 1`] = `
|
|
|
691
701
|
</button>
|
|
692
702
|
<ul
|
|
693
703
|
class="pf-c-nav__list"
|
|
704
|
+
role="list"
|
|
694
705
|
>
|
|
695
706
|
<li
|
|
696
707
|
class="pf-c-nav__item"
|
|
@@ -789,9 +800,11 @@ exports[`Nav Nav Grouped List 1`] = `
|
|
|
789
800
|
</h2>
|
|
790
801
|
<ul
|
|
791
802
|
class="pf-c-nav__list"
|
|
803
|
+
role="list"
|
|
792
804
|
>
|
|
793
805
|
<ul
|
|
794
806
|
class="pf-c-nav__list"
|
|
807
|
+
role="list"
|
|
795
808
|
>
|
|
796
809
|
<li
|
|
797
810
|
class="pf-c-nav__item"
|
|
@@ -860,9 +873,11 @@ exports[`Nav Nav Grouped List 1`] = `
|
|
|
860
873
|
</h2>
|
|
861
874
|
<ul
|
|
862
875
|
class="pf-c-nav__list"
|
|
876
|
+
role="list"
|
|
863
877
|
>
|
|
864
878
|
<ul
|
|
865
879
|
class="pf-c-nav__list"
|
|
880
|
+
role="list"
|
|
866
881
|
>
|
|
867
882
|
<li
|
|
868
883
|
class="pf-c-nav__item"
|
|
@@ -953,6 +968,7 @@ exports[`Nav Nav List with custom item nodes 1`] = `
|
|
|
953
968
|
</button>
|
|
954
969
|
<ul
|
|
955
970
|
class="pf-c-nav__list"
|
|
971
|
+
role="list"
|
|
956
972
|
>
|
|
957
973
|
<li
|
|
958
974
|
class="pf-c-nav__item test-nav-item-class"
|
|
@@ -1020,6 +1036,7 @@ exports[`Nav Nav List with flyout 1`] = `
|
|
|
1020
1036
|
</button>
|
|
1021
1037
|
<ul
|
|
1022
1038
|
class="pf-c-nav__list"
|
|
1039
|
+
role="list"
|
|
1023
1040
|
>
|
|
1024
1041
|
<li
|
|
1025
1042
|
class="pf-c-nav__item pf-m-flyout test-nav-item-class"
|
|
@@ -1100,6 +1117,7 @@ exports[`Nav Renders nav list in strict mode 1`] = `
|
|
|
1100
1117
|
>
|
|
1101
1118
|
<ul
|
|
1102
1119
|
class="pf-c-nav__list test-nav-list-class"
|
|
1120
|
+
role="list"
|
|
1103
1121
|
>
|
|
1104
1122
|
<li
|
|
1105
1123
|
class="pf-c-nav__item test-nav-item-class"
|
|
@@ -1188,6 +1206,7 @@ exports[`Nav Tertiary Nav List 1`] = `
|
|
|
1188
1206
|
</button>
|
|
1189
1207
|
<ul
|
|
1190
1208
|
class="pf-c-nav__list"
|
|
1209
|
+
role="list"
|
|
1191
1210
|
>
|
|
1192
1211
|
<li
|
|
1193
1212
|
class="pf-c-nav__item"
|
|
@@ -9,15 +9,24 @@ export interface NotificationDrawerListProps extends React.HTMLProps<HTMLUListEl
|
|
|
9
9
|
className?: string;
|
|
10
10
|
/** Adds styling to the notification drawer list to indicate expand/hide state */
|
|
11
11
|
isHidden?: boolean;
|
|
12
|
+
/** Adds an accessible label to the notification drawer list. */
|
|
13
|
+
'aria-label'?: string;
|
|
12
14
|
}
|
|
13
15
|
|
|
14
16
|
export const NotificationDrawerList: React.FunctionComponent<NotificationDrawerListProps> = ({
|
|
15
17
|
children,
|
|
16
18
|
className = '',
|
|
17
19
|
isHidden = false,
|
|
20
|
+
'aria-label': ariaLabel,
|
|
18
21
|
...props
|
|
19
22
|
}: NotificationDrawerListProps) => (
|
|
20
|
-
<ul
|
|
23
|
+
<ul
|
|
24
|
+
{...props}
|
|
25
|
+
className={css('pf-c-notification-drawer__list', className)}
|
|
26
|
+
hidden={isHidden}
|
|
27
|
+
role="list"
|
|
28
|
+
aria-label={ariaLabel}
|
|
29
|
+
>
|
|
21
30
|
{children}
|
|
22
31
|
</ul>
|
|
23
32
|
);
|
|
@@ -5,6 +5,7 @@ exports[`NotificationDrawerList drawer list with hidden applied 1`] = `
|
|
|
5
5
|
<ul
|
|
6
6
|
class="pf-c-notification-drawer__list"
|
|
7
7
|
hidden=""
|
|
8
|
+
role="list"
|
|
8
9
|
/>
|
|
9
10
|
</DocumentFragment>
|
|
10
11
|
`;
|
|
@@ -13,6 +14,7 @@ exports[`NotificationDrawerList renders with PatternFly Core styles 1`] = `
|
|
|
13
14
|
<DocumentFragment>
|
|
14
15
|
<ul
|
|
15
16
|
class="pf-c-notification-drawer__list"
|
|
17
|
+
role="list"
|
|
16
18
|
/>
|
|
17
19
|
</DocumentFragment>
|
|
18
20
|
`;
|
|
@@ -57,7 +57,7 @@ export const NotificationDrawerBasic: React.FunctionComponent = () => {
|
|
|
57
57
|
/>
|
|
58
58
|
</NotificationDrawerHeader>
|
|
59
59
|
<NotificationDrawerBody>
|
|
60
|
-
<NotificationDrawerList>
|
|
60
|
+
<NotificationDrawerList aria-label="Notifications in the basic example">
|
|
61
61
|
<NotificationDrawerListItem variant="info">
|
|
62
62
|
<NotificationDrawerListItemHeader
|
|
63
63
|
variant="info"
|
|
@@ -94,7 +94,7 @@ export const NotificationDrawerGroups: React.FunctionComponent = () => {
|
|
|
94
94
|
count={2}
|
|
95
95
|
onExpand={toggleFirstDrawer}
|
|
96
96
|
>
|
|
97
|
-
<NotificationDrawerList isHidden={!firstGroupExpanded}>
|
|
97
|
+
<NotificationDrawerList isHidden={!firstGroupExpanded} aria-label="Notifications in the first group">
|
|
98
98
|
<NotificationDrawerListItem variant="info">
|
|
99
99
|
<NotificationDrawerListItemHeader
|
|
100
100
|
variant="info"
|
|
@@ -205,7 +205,7 @@ export const NotificationDrawerGroups: React.FunctionComponent = () => {
|
|
|
205
205
|
count={2}
|
|
206
206
|
onExpand={toggleSecondDrawer}
|
|
207
207
|
>
|
|
208
|
-
<NotificationDrawerList isHidden={!secondGroupExpanded}>
|
|
208
|
+
<NotificationDrawerList isHidden={!secondGroupExpanded} aria-label="Notifications in the second group">
|
|
209
209
|
<NotificationDrawerListItem variant="info">
|
|
210
210
|
<NotificationDrawerListItemHeader
|
|
211
211
|
variant="info"
|
|
@@ -317,7 +317,7 @@ export const NotificationDrawerGroups: React.FunctionComponent = () => {
|
|
|
317
317
|
onExpand={toggleThirdDrawer}
|
|
318
318
|
truncateTitle={1}
|
|
319
319
|
>
|
|
320
|
-
<NotificationDrawerList isHidden={!thirdGroupExpanded}>
|
|
320
|
+
<NotificationDrawerList isHidden={!thirdGroupExpanded} aria-label="Notifications in the third group">
|
|
321
321
|
<EmptyState variant={EmptyStateVariant.full}>
|
|
322
322
|
<EmptyStateIcon icon={SearchIcon} />
|
|
323
323
|
<Title headingLevel="h2" size="lg">
|
|
@@ -58,7 +58,10 @@ export const NotificationDrawerLightweight: React.FunctionComponent = () => {
|
|
|
58
58
|
isRead={true}
|
|
59
59
|
onExpand={toggleFirstDrawer}
|
|
60
60
|
>
|
|
61
|
-
<NotificationDrawerList
|
|
61
|
+
<NotificationDrawerList
|
|
62
|
+
isHidden={!firstGroupExpanded}
|
|
63
|
+
aria-label="Notifications in the first lightweight group"
|
|
64
|
+
>
|
|
62
65
|
<NotificationDrawerListItem variant="info">
|
|
63
66
|
<NotificationDrawerListItemHeader
|
|
64
67
|
variant="info"
|
|
@@ -109,7 +112,10 @@ export const NotificationDrawerLightweight: React.FunctionComponent = () => {
|
|
|
109
112
|
isRead={true}
|
|
110
113
|
onExpand={toggleSecondDrawer}
|
|
111
114
|
>
|
|
112
|
-
<NotificationDrawerList
|
|
115
|
+
<NotificationDrawerList
|
|
116
|
+
isHidden={!secondGroupExpanded}
|
|
117
|
+
aria-label="Notifications in the second lightweight group"
|
|
118
|
+
>
|
|
113
119
|
<NotificationDrawerListItem variant="info">
|
|
114
120
|
<NotificationDrawerListItemHeader
|
|
115
121
|
variant="info"
|
|
@@ -162,7 +168,10 @@ export const NotificationDrawerLightweight: React.FunctionComponent = () => {
|
|
|
162
168
|
isRead={true}
|
|
163
169
|
onExpand={toggleThirdDrawer}
|
|
164
170
|
>
|
|
165
|
-
<NotificationDrawerList
|
|
171
|
+
<NotificationDrawerList
|
|
172
|
+
isHidden={!thirdGroupExpanded}
|
|
173
|
+
aria-label="Notifications in the third lightweight group"
|
|
174
|
+
>
|
|
166
175
|
<EmptyState variant={EmptyStateVariant.full}>
|
|
167
176
|
<EmptyStateIcon icon={SearchIcon} />
|
|
168
177
|
<Title headingLevel="h2" size="lg">
|
|
@@ -176,6 +176,7 @@ exports[`Page Check page to verify breadcrumb is created - PageBreadcrumb syntax
|
|
|
176
176
|
>
|
|
177
177
|
<ol
|
|
178
178
|
class="pf-c-breadcrumb__list"
|
|
179
|
+
role="list"
|
|
179
180
|
>
|
|
180
181
|
<li
|
|
181
182
|
class="pf-c-breadcrumb__item"
|
|
@@ -343,6 +344,7 @@ exports[`Page Check page to verify breadcrumb is created 1`] = `
|
|
|
343
344
|
>
|
|
344
345
|
<ol
|
|
345
346
|
class="pf-c-breadcrumb__list"
|
|
347
|
+
role="list"
|
|
346
348
|
>
|
|
347
349
|
<li
|
|
348
350
|
class="pf-c-breadcrumb__item"
|
|
@@ -513,6 +515,7 @@ exports[`Page Check page to verify grouped nav and breadcrumb - new components s
|
|
|
513
515
|
>
|
|
514
516
|
<ol
|
|
515
517
|
class="pf-c-breadcrumb__list"
|
|
518
|
+
role="list"
|
|
516
519
|
>
|
|
517
520
|
<li
|
|
518
521
|
class="pf-c-breadcrumb__item"
|
|
@@ -635,6 +638,7 @@ exports[`Page Check page to verify grouped nav and breadcrumb - new components s
|
|
|
635
638
|
</button>
|
|
636
639
|
<ul
|
|
637
640
|
class="pf-c-nav__list"
|
|
641
|
+
role="list"
|
|
638
642
|
>
|
|
639
643
|
<li
|
|
640
644
|
class="pf-c-nav__item"
|
|
@@ -821,6 +825,7 @@ exports[`Page Check page to verify grouped nav and breadcrumb - old / props synt
|
|
|
821
825
|
</button>
|
|
822
826
|
<ul
|
|
823
827
|
class="pf-c-nav__list"
|
|
828
|
+
role="list"
|
|
824
829
|
>
|
|
825
830
|
<li
|
|
826
831
|
class="pf-c-nav__item"
|
|
@@ -920,6 +925,7 @@ exports[`Page Check page to verify grouped nav and breadcrumb - old / props synt
|
|
|
920
925
|
>
|
|
921
926
|
<ol
|
|
922
927
|
class="pf-c-breadcrumb__list"
|
|
928
|
+
role="list"
|
|
923
929
|
>
|
|
924
930
|
<li
|
|
925
931
|
class="pf-c-breadcrumb__item"
|
|
@@ -1108,6 +1114,7 @@ exports[`Page Check page to verify nav is created - PageNavigation syntax 1`] =
|
|
|
1108
1114
|
</button>
|
|
1109
1115
|
<ul
|
|
1110
1116
|
class="pf-c-nav__list"
|
|
1117
|
+
role="list"
|
|
1111
1118
|
>
|
|
1112
1119
|
<li
|
|
1113
1120
|
class="pf-c-nav__item"
|
|
@@ -1284,6 +1291,7 @@ exports[`Page Check page to verify skip to content points to main content region
|
|
|
1284
1291
|
>
|
|
1285
1292
|
<ol
|
|
1286
1293
|
class="pf-c-breadcrumb__list"
|
|
1294
|
+
role="list"
|
|
1287
1295
|
>
|
|
1288
1296
|
<li
|
|
1289
1297
|
class="pf-c-breadcrumb__item"
|
|
@@ -1512,6 +1520,7 @@ exports[`Page Sticky bottom breadcrumb on all height breakpoints - PageBreadcrum
|
|
|
1512
1520
|
>
|
|
1513
1521
|
<ol
|
|
1514
1522
|
class="pf-c-breadcrumb__list"
|
|
1523
|
+
role="list"
|
|
1515
1524
|
>
|
|
1516
1525
|
<li
|
|
1517
1526
|
class="pf-c-breadcrumb__item"
|
|
@@ -1679,6 +1688,7 @@ exports[`Page Verify sticky bottom breadcrumb on all height breakpoints 1`] = `
|
|
|
1679
1688
|
>
|
|
1680
1689
|
<ol
|
|
1681
1690
|
class="pf-c-breadcrumb__list"
|
|
1691
|
+
role="list"
|
|
1682
1692
|
>
|
|
1683
1693
|
<li
|
|
1684
1694
|
class="pf-c-breadcrumb__item"
|
|
@@ -1846,6 +1856,7 @@ exports[`Page Verify sticky top breadcrumb on all height breakpoints - PageBread
|
|
|
1846
1856
|
>
|
|
1847
1857
|
<ol
|
|
1848
1858
|
class="pf-c-breadcrumb__list"
|
|
1859
|
+
role="list"
|
|
1849
1860
|
>
|
|
1850
1861
|
<li
|
|
1851
1862
|
class="pf-c-breadcrumb__item"
|
|
@@ -2013,6 +2024,7 @@ exports[`Page Verify sticky top breadcrumb on all height breakpoints 1`] = `
|
|
|
2013
2024
|
>
|
|
2014
2025
|
<ol
|
|
2015
2026
|
class="pf-c-breadcrumb__list"
|
|
2027
|
+
role="list"
|
|
2016
2028
|
>
|
|
2017
2029
|
<li
|
|
2018
2030
|
class="pf-c-breadcrumb__item"
|
|
@@ -88,6 +88,8 @@ export const ProgressStep: React.FunctionComponent<ProgressStepProps> = ({
|
|
|
88
88
|
className
|
|
89
89
|
)}
|
|
90
90
|
aria-label={ariaLabel}
|
|
91
|
+
// CSS style `display: contents` gives this li a generic role, we need to override that
|
|
92
|
+
role="listitem"
|
|
91
93
|
{...(isCurrent && { 'aria-current': 'step' })}
|
|
92
94
|
{...props}
|
|
93
95
|
>
|
|
@@ -14,6 +14,8 @@ export interface ProgressStepperProps
|
|
|
14
14
|
isVertical?: boolean;
|
|
15
15
|
/** Flag indicating the progress stepper should be rendered compactly. */
|
|
16
16
|
isCompact?: boolean;
|
|
17
|
+
/** Adds an accessible label to the progress stepper. */
|
|
18
|
+
'aria-label'?: string;
|
|
17
19
|
}
|
|
18
20
|
|
|
19
21
|
export const ProgressStepper: React.FunctionComponent<ProgressStepperProps> = ({
|
|
@@ -22,6 +24,7 @@ export const ProgressStepper: React.FunctionComponent<ProgressStepperProps> = ({
|
|
|
22
24
|
isCenterAligned,
|
|
23
25
|
isVertical,
|
|
24
26
|
isCompact,
|
|
27
|
+
'aria-label': ariaLabel,
|
|
25
28
|
...props
|
|
26
29
|
}: ProgressStepperProps) => (
|
|
27
30
|
<ol
|
|
@@ -32,6 +35,8 @@ export const ProgressStepper: React.FunctionComponent<ProgressStepperProps> = ({
|
|
|
32
35
|
isCompact && styles.modifiers.compact,
|
|
33
36
|
className
|
|
34
37
|
)}
|
|
38
|
+
role="list"
|
|
39
|
+
aria-label={ariaLabel}
|
|
35
40
|
{...props}
|
|
36
41
|
>
|
|
37
42
|
{children}
|
|
@@ -2,7 +2,7 @@ import React from 'react';
|
|
|
2
2
|
import { ProgressStepper, ProgressStep } from '@patternfly/react-core';
|
|
3
3
|
|
|
4
4
|
export const ProgressStepperBasic: React.FunctionComponent = () => (
|
|
5
|
-
<ProgressStepper>
|
|
5
|
+
<ProgressStepper aria-label="Basic progress stepper">
|
|
6
6
|
<ProgressStep
|
|
7
7
|
variant="success"
|
|
8
8
|
id="basic-step1"
|
|
@@ -2,7 +2,7 @@ import React from 'react';
|
|
|
2
2
|
import { ProgressStepper, ProgressStep } from '@patternfly/react-core';
|
|
3
3
|
|
|
4
4
|
export const ProgressStepperBasicFailure: React.FunctionComponent = () => (
|
|
5
|
-
<ProgressStepper>
|
|
5
|
+
<ProgressStepper aria-label="Basic progress stepper with failure">
|
|
6
6
|
<ProgressStep
|
|
7
7
|
variant="success"
|
|
8
8
|
id="basic-with-failure-step1"
|
|
@@ -2,7 +2,7 @@ import React from 'react';
|
|
|
2
2
|
import { ProgressStepper, ProgressStep } from '@patternfly/react-core';
|
|
3
3
|
|
|
4
4
|
export const ProgressStepperBasicIssue: React.FunctionComponent = () => (
|
|
5
|
-
<ProgressStepper>
|
|
5
|
+
<ProgressStepper aria-label="Basic progress stepper with issue">
|
|
6
6
|
<ProgressStep
|
|
7
7
|
variant="success"
|
|
8
8
|
id="basic-with-issue-step1"
|
|
@@ -24,7 +24,11 @@ export const ProgressStepperBasicWithAlignment: React.FunctionComponent = () =>
|
|
|
24
24
|
name="toggle-center"
|
|
25
25
|
/>
|
|
26
26
|
<br />
|
|
27
|
-
<ProgressStepper
|
|
27
|
+
<ProgressStepper
|
|
28
|
+
isVertical={isVertical}
|
|
29
|
+
isCenterAligned={isCenterAligned}
|
|
30
|
+
aria-label="Basic progress stepper with alignment"
|
|
31
|
+
>
|
|
28
32
|
<ProgressStep
|
|
29
33
|
variant="success"
|
|
30
34
|
description="This is the first thing to happen"
|
|
@@ -2,7 +2,7 @@ import React from 'react';
|
|
|
2
2
|
import { ProgressStepper, ProgressStep } from '@patternfly/react-core';
|
|
3
3
|
|
|
4
4
|
export const ProgressStepperBasicWithDescription: React.FunctionComponent = () => (
|
|
5
|
-
<ProgressStepper>
|
|
5
|
+
<ProgressStepper aria-label="Basic progress stepper with description">
|
|
6
6
|
<ProgressStep
|
|
7
7
|
variant="success"
|
|
8
8
|
description="This is the first thing to happen"
|
|
@@ -6,7 +6,7 @@ export const ProgressStepperCompact: React.FunctionComponent = () => {
|
|
|
6
6
|
const [isCenterAligned, setIsCenterAligned] = React.useState(false);
|
|
7
7
|
|
|
8
8
|
return (
|
|
9
|
-
<React.Fragment>
|
|
9
|
+
<React.Fragment aria-label="Compact progress stepper">
|
|
10
10
|
<Checkbox
|
|
11
11
|
label="Vertical alignment"
|
|
12
12
|
isChecked={isVertical}
|
|
@@ -4,7 +4,7 @@ import InProgressIcon from '@patternfly/react-icons/dist/esm/icons/in-progress-i
|
|
|
4
4
|
import PendingIcon from '@patternfly/react-icons/dist/esm/icons/pending-icon';
|
|
5
5
|
|
|
6
6
|
export const ProgressStepperCustomIcons: React.FunctionComponent = () => (
|
|
7
|
-
<ProgressStepper>
|
|
7
|
+
<ProgressStepper aria-label="Progress stepper with custom icons">
|
|
8
8
|
<ProgressStep
|
|
9
9
|
variant="success"
|
|
10
10
|
id="custom-step1"
|