"Add site" Stepper: Introduce click handler to allow clicks on the previous step that is already complete

[ivan-ottinger]

Related issues

• Resolves STU-852

Proposed Changes

• introduce handleStepClick to handle clicks on the previous step that is already complete
• ensure the selected import file is remembered when navigating between steps
• adjust styling of the previous step label

Screen.Capture.on.2025-10-15.at.17-24-50.mp4

Testing Instructions

1. Check out the PR branch and build the app with npm install && npm start.
2. Click the Add site button in the app sidebar and try to navigate through the steps of Start from Blueprint and Import from backup flows.
3. When you complete the first step and get to the second one by clicking the Continue button, it should be possible to get back to the first step by clicking the step label (not just the Back button).
4. When you are in the Import from backup flow and navigating back to the step where the import file is selected, it should be remembered.
5. When you review how the step labels are animated during transition between steps, there should be no odd movement / shift in those labels.

Pre-merge Checklist

• Have you checked for TypeScript, React or other console errors?

[ivan-ottinger]

I am curios to hear what you think about this change.

Before (you can see slight shift / movement in labels):

Screen.Capture.on.2025-10-15.at.17-17-26.mov

After (there's no shift in the labels; instead the unselected label has thinner font size):

Screen.Capture.on.2025-10-15.at.17-16-26.mp4

[bcotrim]

Nice catch!

[github-actions]

📊 Performance Test Results

Comparing a4bb4c0 vs trunk

site-editor

Metric	trunk	a4bb4c0	Diff	Change
load	16639.00 ms	14375.00 ms	-2264.00 ms	🟢 -13.6%

site-startup

Metric	trunk	a4bb4c0	Diff	Change
siteCreation	23444.00 ms	23218.00 ms	-226.00 ms	🟢 -1.0%
siteStartup	9051.00 ms	8104.00 ms	-947.00 ms	🟢 -10.5%

---

Results are median values from multiple test runs.

Legend: 🟢 Improvement (faster) | 🔴 Regression (slower) | ⚪ No change

[bcotrim]

LGTM 👍
Added some suggestions, but feel free to merge as is.

[bcotrim]

I would also move this logic to the useStepper hook.

[ivan-ottinger]

Thanks for the suggestions, Bernardo. I agree, it make sense to move these. I have moved the click handler as well: ad724a7.

[gcsecsey]

The changes LGTM and work well. I can navigate back using the labels, and the file selection in earlier step is also preserved. 👍

[ivan-ottinger]

Thank you for your reviews, Bernardo and Gergely!