Skip to content

Conversation

@kylecarbs
Copy link
Member

Replaces dynamic timestamps with Apple's iconic demo time (January 24, 2024, 9:41 AM PST) and mocks VERSION in Storybook to ensure consistent visual snapshots in Chromatic.

Changes

  • Stable timestamps in stories: All story files now use a consistent timestamp instead of Date.now()
  • Mocked VERSION for Storybook: Created .storybook/mocks/version.ts to provide stable version info
  • TitleBar story: Added TitleBar.stories.tsx to test version display

Why

Chromatic visual regression tests were failing due to:

  1. Timestamps changing on every render
  2. VERSION changing based on git state during build

Now all visual tests use stable, predictable values.

Generated with cmux

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

- Replace Date.now() with stable timestamp in all story files
- Mock VERSION in Storybook with stable values for Chromatic
- Use Apple's iconic demo time: January 24, 2024, 9:41 AM PST
- Ensures consistent visual snapshots across Chromatic runs
@kylecarbs kylecarbs force-pushed the tailwind-story-ignore branch from f5c9931 to eea24f8 Compare October 23, 2025 02:55
Add fixedTipIndex prop to TipsCarousel to override automatic rotation
in Storybook stories, ensuring consistent visual snapshots.
@kylecarbs kylecarbs merged commit 4850101 into main Oct 23, 2025
12 of 13 checks passed
@kylecarbs kylecarbs deleted the tailwind-story-ignore branch October 23, 2025 03: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.

1 participant