Remove onboarding progress bar - #97590
Conversation
Co-authored-by: Rushat Gabhane <rushatgabhane@users.noreply.github.com>
|
The only failing check is It fails for one reason: Good news on the part the PR body flagged as untested: all the automated checks that couldn't be run locally have now passed in CI — Next step: fill out the PR Author Checklist (and the |
|
@MelvinBot merge main and show web screenshot please |
|
Looked into the failing checks — every real CI check is green (tests, typecheck, ESLint, Storybook, translations, perf, builds, etc.). The one red check is PR Author Checklist / checklist, which isn't a code or CI failure I can push a fix for. That check fails only because the checkbox items in the PR Author Checklist section of the description haven't been ticked yet (and the QA Steps still have the To make it pass:
Once the checklist is filled out, this check will re-run and pass. Nothing in the code needs changing. |
…ngProgressBar Co-authored-by: Rushat Gabhane <rushatgabhane@users.noreply.github.com>
|
✅ Merged Web screenshot: The onboarding flow isn't reachable in this test session — CI signs in with Home screen after signing in — header has no progress bar (onboarding is skipped for this CI test account) |

Explanation of Change
Per the design decision in the linked issue, we are removing the onboarding progress bar entirely (rather than trying to make the color-only bar pass WCAG 1.4.1). This removes the visual bar and all of its dedicated plumbing:
HeaderWithBackButton: removed theprogressBarPercentageprop, the progress-bar branch inmiddleContent, the associatedrole="progressbar"markup, and thepl0container style that only applied when the bar was shown.src/styles/index.ts: removed the now-unusedheaderProgressBarContainer/headerProgressBar/headerProgressBarFillstyles.getOnboardingStepCounter: droppedprogressBarPercentagefromOnboardingStepResultand the two return sites (plus the now-unusedmaxSuffixLength/maxPrivateSuffixLengthhelpers used only for that calculation). The step calculator still returnsstepCounter, which onboarding back-navigation andisEmployeesFirstStepcontinue to rely on.progressBarPercentage/stepCounterto the header so no progress indicator is shown, and removed theuseOnboardingStepCounterhook where it became unused (kept inBaseOnboardingEmployees, which still needs it forisEmployeesFirstStep).common.progressBarLabeltranslation key from all locales and theProgressBarStorybook story.Scope note for reviewers / @Expensify/design: I interpreted "remove the progress bar entirely" as showing no progress indicator in onboarding (neither the bar nor a "Step X of X" subtitle). If you'd instead prefer to surface the accessible "Step X of X" text in place of the bar, that's a small follow-up — let me know. The generic
stepCounterprop onHeaderWithBackButtonis untouched and still used by the Two-Factor Auth flow.Fixed Issues
$ #77334
PROPOSAL:
Tests
Offline tests
Same as tests.
QA Steps
// TODO: These must be filled out, or the issue title must include "[No QA]."
PR Author Checklist
### Fixed Issuessection aboveTestssectionOffline stepssectionQA stepssectionAvatar, I verified the components usingAvatarare working as expected)StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))npm run compress-svg)Avataris modified, I verified thatAvataris working as expected in all cases)Designlabel and/or tagged@Expensify/designso the design team can review the changes.mainbranch was merged into this PR after a review, I tested again and verified the outcome was still expected according to theTeststeps.Screenshots/Videos
Android: Native
Android: mWeb Chrome
iOS: Native
iOS: mWeb Safari
MacOS: Chrome / Safari