Skip to content

feat(OrderSidebar): improve stories with proper preview and controls#68

Merged
garrity-miepub merged 1 commit into
mainfrom
feature/order-sidebar-updates
Feb 3, 2026
Merged

feat(OrderSidebar): improve stories with proper preview and controls#68
garrity-miepub merged 1 commit into
mainfrom
feature/order-sidebar-updates

Conversation

@garrity-miepub

Copy link
Copy Markdown
Collaborator
  • Add decorator with 700px height container for full sidebar visibility
  • Disable controls for complex types that break Storybook:
    • createdAt, scheduledDate (Date objects cause serialization issues)
    • actions (complex array with callbacks)
    • children (ReactNode)
  • Keeps working controls: open, orderId, status, patientName, employerName, serviceName, priority, notes
order-sidebar.mov

- Add decorator with 700px height container for full sidebar visibility
- Disable controls for complex types that break Storybook:
  - createdAt, scheduledDate (Date objects cause serialization issues)
  - actions (complex array with callbacks)
  - children (ReactNode)
- Keeps working controls: open, orderId, status, patientName,
  employerName, serviceName, priority, notes
Copilot AI review requested due to automatic review settings February 3, 2026 23:49
@garrity-miepub garrity-miepub marked this pull request as ready for review February 3, 2026 23:50

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 updates the OrderSidebar Storybook stories to improve visual preview and avoid control serialization issues with complex props.

Changes:

  • Wraps all OrderSidebar stories in a 700px-tall container via a Storybook decorator to ensure full sidebar visibility in layout: 'fullscreen'.
  • Configures argTypes to disable Storybook controls for complex/unsupported props (createdAt, scheduledDate, actions, children) while keeping simple, useful controls enabled.

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

@garrity-miepub garrity-miepub merged commit e1977a0 into main Feb 3, 2026
16 checks passed
@garrity-miepub garrity-miepub deleted the feature/order-sidebar-updates branch February 3, 2026 23:56
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