Skip to content

Commit 96c82c0

Browse files
committed
fix(pf): bug in wizard navbar
The wizard navigation should only contain one WizardNav component. Wrapping the WizardNavItems in two WizardNav components is causing all steps to be treated as substeps and therefore don't have step numbers. The fix is to remove one of the WizardNav wrappers.
1 parent baaf600 commit 96c82c0

File tree

1 file changed

+4
-4
lines changed
  • packages/pf4-component-mapper/src/wizard/wizard-components

1 file changed

+4
-4
lines changed

packages/pf4-component-mapper/src/wizard/wizard-components/wizard-nav.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
1-
import React, { useState, useEffect } from 'react';
1+
import React, { Fragment, useState, useEffect } from 'react';
22

3-
import { WizardNavItem, WizardNav } from '@patternfly/react-core';
3+
import { WizardNav, WizardNavItem } from '@patternfly/react-core';
44

55
import isEqual from 'lodash/isEqual';
66
import get from 'lodash/get';
@@ -21,7 +21,7 @@ const memoValues = (initialValue) => {
2121

2222
const WizardNavigationInternal = React.memo(
2323
({ navSchema, activeStepIndex, maxStepIndex, jumpToStep, valid, validating }) => (
24-
<WizardNav>
24+
<Fragment>
2525
{navSchema
2626
.filter((field) => field.primary)
2727
.map((step) => {
@@ -63,7 +63,7 @@ const WizardNavigationInternal = React.memo(
6363
</WizardNavItem>
6464
);
6565
})}
66-
</WizardNav>
66+
</Fragment>
6767
),
6868
isEqual
6969
);

0 commit comments

Comments
 (0)