Skip to content

Commit 0dc21d0

Browse files
committed
🐛 Fix wizard back button in first step
Resolves: https://issues.redhat.com/browse/MTA-6037 In the first step of a wizard, if there is only one step, the back button should be hidden. If there is more than one step, the back button on the first step should be disabled. Signed-off-by: Scott J Dickerson <[email protected]>
1 parent 7f7f27c commit 0dc21d0

File tree

3 files changed

+11
-5
lines changed

3 files changed

+11
-5
lines changed

client/src/app/pages/applications/generate-assets-wizard/generate-assets-wizard.tsx

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -145,6 +145,8 @@ const GenerateAssetsWizardInner: React.FC<IGenerateAssetsWizard> = ({
145145
id="select-target-profile"
146146
name={t("generateAssetsWizard.selectTargetProfile.stepTitle")}
147147
footer={{
148+
nextButtonText: t("actions.next"),
149+
backButtonText: t("actions.back"),
148150
isNextDisabled: !state.profile,
149151
}}
150152
>
@@ -159,6 +161,8 @@ const GenerateAssetsWizardInner: React.FC<IGenerateAssetsWizard> = ({
159161
id="capture-parameters"
160162
name={t("generateAssetsWizard.captureParameters.stepTitle")}
161163
footer={{
164+
nextButtonText: t("actions.next"),
165+
backButtonText: t("actions.back"),
162166
isNextDisabled: !state.parameters.isValid,
163167
}}
164168
>
@@ -176,10 +180,11 @@ const GenerateAssetsWizardInner: React.FC<IGenerateAssetsWizard> = ({
176180
nextButtonText: results
177181
? t("actions.close")
178182
: t("actions.generateAssets"),
179-
onNext: results ? handleCancel : submitTasksAndSaveResults,
183+
backButtonText: t("actions.back"),
180184
isNextDisabled: !state.isReady && !results,
181185
isBackDisabled: !!results,
182186
isCancelHidden: !!results,
187+
onNext: results ? handleCancel : submitTasksAndSaveResults,
183188
}}
184189
>
185190
{!results ? (

client/src/app/pages/applications/retrieve-config-wizard/retrieve-config-wizard.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -164,9 +164,9 @@ const RetrieveConfigWizardInner: React.FC<IRetrieveConfigWizard> = ({
164164
nextButtonText: showResults
165165
? t("actions.close")
166166
: t("actions.retrieve"),
167-
onNext: showResults ? handleCancel : handleSubmit(onSubmit),
168-
isBackDisabled: showResults,
167+
isBackHidden: true, // one step wizard, no back button needed
169168
isCancelHidden: showResults,
169+
onNext: showResults ? handleCancel : handleSubmit(onSubmit),
170170
}}
171171
>
172172
{!showResults ? (

client/src/app/pages/source-platforms/discover-import-wizard/discover-import-wizard.tsx

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -133,6 +133,7 @@ const DiscoverImportWizardInner: React.FC<IDiscoverImportWizard> = ({
133133
name={t("platformDiscoverWizard.filterInput.stepTitle")}
134134
footer={{
135135
nextButtonText: t("actions.next"),
136+
backButtonText: t("actions.back"),
136137
isNextDisabled: !filters.isValid,
137138
}}
138139
>
@@ -150,11 +151,11 @@ const DiscoverImportWizardInner: React.FC<IDiscoverImportWizard> = ({
150151
nextButtonText: results
151152
? t("actions.close")
152153
: t("actions.discoverApplications"),
153-
onNext: results ? handleCancel : onSubmitTask,
154-
isNextDisabled: !state.isReady && !results,
155154
backButtonText: t("actions.back"),
155+
isNextDisabled: !state.isReady && !results,
156156
isBackDisabled: !!results,
157157
isCancelHidden: !!results,
158+
onNext: results ? handleCancel : onSubmitTask,
158159
}}
159160
>
160161
{!results ? (

0 commit comments

Comments
 (0)