Skip to content

feat(OnboardingWizard): rewrite stories with interactive demo wrapper and fix broken logos#64

Merged
garrity-miepub merged 3 commits intomainfrom
feature/onboarding-wizard-updates
Feb 3, 2026
Merged

feat(OnboardingWizard): rewrite stories with interactive demo wrapper and fix broken logos#64
garrity-miepub merged 3 commits intomainfrom
feature/onboarding-wizard-updates

Conversation

@garrity-miepub
Copy link
Copy Markdown
Contributor

@garrity-miepub garrity-miepub commented Feb 3, 2026

Stories Improvements:

  • Replace static stories with OnboardingWizardDemo wrapper for proper state management
  • Add demoMode selector: interactive, loading, error, customBranding, noHeader, incomplete
  • Implement conditional argTypes that show/hide controls based on selected demo mode
  • Add currentStep range slider (0-4) for easy step navigation
  • Change layout from fullscreen to padded with 700px container
  • Fix container styling to properly position footer at bottom

Logo Fixes:

  • Update default logoUrl from broken local path to MIE logo URL
  • Update customBranding story to use Enterprise Health HubSpot-hosted logo

UI Enhancements:

  • Improve payment step with actual form fields and proper label accessibility
  • Polish import step buttons with better spacing and hover states
  • Shorten button labels for cleaner appearance
onboarding.mov

… and fix broken logos

Stories Improvements:
- Replace static stories with OnboardingWizardDemo wrapper for proper state management
- Add demoMode selector: interactive, loading, error, customBranding, noHeader, incomplete
- Implement conditional argTypes that show/hide controls based on selected demo mode
- Add currentStep range slider (0-4) for easy step navigation
- Change layout from fullscreen to padded with 700px container
- Fix container styling to properly position footer at bottom

Logo Fixes:
- Update default logoUrl from broken local path to MIE logo URL
- Update customBranding story to use Enterprise Health HubSpot-hosted logo

UI Enhancements:
- Improve payment step with actual form fields and proper label accessibility
- Polish import step buttons with better spacing and hover states
- Shorten button labels for cleaner appearance
Copilot AI review requested due to automatic review settings February 3, 2026 21:35
…e support

Component Integration:
- Replace raw HTML buttons with Button component
- Replace custom spinner with Spinner component
- Replace custom error div with Alert component
- Replace custom progress bar with Progress component
- Use Input component in stories for form fields

Dark Mode / Theming:
- Use bg-background for theme-aware backgrounds
- Use text-foreground for proper text colors
- Use border-border for theme-aware borders
- Use text-muted-foreground for secondary text

UX Improvements:
- Back button always visible but disabled on first step
- Progress bar position stays consistent across steps
- All interactive elements use consistent Button variants
@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages bot commented Feb 3, 2026

Deploying ui with  Cloudflare Pages  Cloudflare Pages

Latest commit: c7f4f26
Status: ✅  Deploy successful!
Preview URL: https://560c3f0c.ui-6d0.pages.dev
Branch Preview URL: https://feature-onboarding-wizard-up.ui-6d0.pages.dev

View logs

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR modernizes the OnboardingWizard Storybook stories with an interactive demo wrapper and fixes branding/logo issues while polishing some step UIs.

Changes:

  • Update OnboardingWizard’s default logoUrl to a hosted MIE logo instead of a broken local asset.
  • Replace static stories with an OnboardingWizardDemo wrapper that supports multiple demo modes (interactive, loading, error, custom branding, no header, incomplete) with conditional controls and a step slider.
  • Refine Import and Payment steps in the sample data with improved button layout, hover states, and a more realistic, accessible payment form UI.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
src/components/OnboardingWizard/OnboardingWizard.tsx Switches the default logoUrl to a hosted MIE image to fix broken logo references.
src/components/OnboardingWizard/OnboardingWizard.stories.tsx Rewrites stories to use an interactive OnboardingWizardDemo wrapper, adds demo modes and conditional argTypes, and improves the sample Import/Payment step UIs and branding examples.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@garrity-miepub garrity-miepub force-pushed the feature/onboarding-wizard-updates branch from 24bdc21 to 434da31 Compare February 3, 2026 21:51
Copilot AI review requested due to automatic review settings February 3, 2026 21:51
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@garrity-miepub garrity-miepub marked this pull request as ready for review February 3, 2026 21:58
- Remove hard-coded currentStep:4 in incomplete mode to allow navigation
- Use 0-based indices for incompleteSteps (1='Import Employees', 2='Payment')
- Set effectiveInitialStep to 4 for incomplete mode when starting fresh
- Navigation from OnboardingCompletion's 'Go to step' buttons now works correctly
@garrity-miepub garrity-miepub force-pushed the feature/onboarding-wizard-updates branch from 434da31 to c7f4f26 Compare February 3, 2026 22:00
@garrity-miepub garrity-miepub merged commit 5851752 into main Feb 3, 2026
10 checks passed
@garrity-miepub garrity-miepub deleted the feature/onboarding-wizard-updates branch February 3, 2026 22:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants