Skip to content

[FEATURE] [Chromatic] Expand Storybook coverage — 18 components currently have no story #1789

Closed as not planned

Description

@magic-peach

Feature description
Chromatic visual regression only covers components with a .stories.tsx file. Cross-referencing src/components/*.tsx against existing *.stories.tsx files, these components currently have no story: ColorPicker, ComparisonPreview, DraggableTextOverlays, ErrorBoundary, FontSelector, Footer, ImageOverlay, LottiePlayer, NativeShareButton, OnboardingTour, ScrollToTop, TextControls, ThemeToggle, ThumbnailStrip, TipCarousel, VideoPreview, WaveformCanvas (ThemeProvider and DatadogInit are context/logic-only and don't need one).

Problem this solves
Visual regressions in any of these 17 components currently go completely undetected by the Chromatic CI check that's already sponsoring and running on every PR.

Proposed solution
Good first-issue-sized in small batches (e.g., 3-4 components per PR) rather than one giant PR — add a .stories.tsx for each following the pattern already established in existing stories (e.g. TrimControl.stories.tsx, FramingControl.stories.tsx).

Alternatives considered
One mega-PR covering all 17 — worse for review and for GSSoC point distribution across multiple contributors; split it up.

Activity

  1. added
    gssoc'26GirlScript Summer of Code 2026
    featureNew feature request
    testingTests needed or test infrastructure
    size: mediumMedium issue - a few files
    on Sep 15, 2026
  2. github-actions commented on Sep 15, 2026

    @github-actions
    Contributor

    👋 Thanks for opening an issue, @magic-peach!

    Our maintainers will review this shortly 🚀

    Before you start working

    • 📖 Read the CONTRIBUTING.md guide
    • 💬 Wait for a maintainer to assign you the issue before starting work
    • 🔍 Search for existing issues to avoid duplicates

    For GSSoC'26 contributors

    If you're contributing under GirlScript Summer of Code 2026, please mention it in your comment when requesting assignment:

    "I'd like to work on this — contributing under GSSoC'26"

    Architecture notes

    Reframe is a static Next.js app with output: "export". Key points:

    • All video processing uses FFmpeg.wasm (client-side, no server)
    • headers() in next.config.ts won't work — use vercel.json instead
    • not-found.tsx requires public/404.html + vercel.json routes config

    Happy contributing! ⭐ Star the repo if you find it useful!

  3. github-actions commented on Sep 15, 2026

    @github-actions
    Contributor

    Claude encountered an error —— View job


    I'll analyze this and get back to you.

  4. github-actions commented on Oct 16, 2026

    @github-actions
    Contributor

    👋 This issue has been inactive for 30 days. It will be automatically closed in 14 days if there's no further activity.

    If you're still interested in working on this, please leave a comment!
    For GSSoC'26 contributors: make sure to keep your assigned issues active.

    ⭐ Star Reframe if you find it useful!

  5. github-actions commented on Oct 30, 2026

    @github-actions
    Contributor

    🔒 Closing this issue due to inactivity. Feel free to reopen if you'd like to work on it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    featureNew feature requestgssoc'26GirlScript Summer of Code 2026level:intermediateIntermediate level - 35 ptsneeds investigationNeeds more investigation before work beginssize: mediumMedium issue - a few filestestingTests needed or test infrastructuretype:featureNew featuretype:testingTesting

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions