Skip to content

Explain how studies get added, and make the setup card's add button primary - #789

Merged
InfinityBowman merged 1 commit into
mainfrom
feat/studies-explainer-and-setup-cta
Sep 13, 2026
Merged

InfinityBowman merged 1 commit into
mainfrom
feat/studies-explainer-and-setup-cta

Conversation

@InfinityBowman

Copy link
Copy Markdown
Owner

What changed

Setup card (Overview)

  • The Add more button in the Add studies cell is now the primary blue variant, so the next action stands out from the count line.
  • Once all four steps are done, the ghost "Finish later" in the header becomes an outlined "Done". Same dismiss handler.

All studies

  • New StudiesExplainer card above the list, in the same chrome as the setup card. Three columns: drop PDFs anywhere on the page, import through the Add studies button (inline link opens the sheet), or attach a PDF to a study later by expanding it.
  • Renders only once studies exist, so it never competes with the empty-project add form.
  • Dismissal is stored in localStorage keyed by user id, wrapped in try/catch for Safari private mode. Server-side persistence so it follows the user across devices is Persist dismissed UI hints in user preferences so they follow the user across devices #788.

Verification

  • tsc --noEmit clean
  • pnpm lint clean
  • pnpm --filter web test: 44 files, 312 tests passed

https://claude.ai/code/session_0127k6bNMdQoMLtEfnAQdAyV

…rimary

The Add studies cell on the setup card now uses the primary button so the
next action reads at a glance, and once every step is done the header's
Finish later becomes an outlined Done.

All studies gets a one-time explainer card in the same chrome as the setup
card: drop PDFs anywhere on the page, import through Add studies, or attach
a PDF to a study later. Dismissal is stored per user in localStorage;
moving it to server-side preferences is tracked in #788.

Claude-Session: https://claude.ai/code/session_0127k6bNMdQoMLtEfnAQdAyV
@coderabbitai

coderabbitai Bot commented Sep 13, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 48 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 3642d0c5-283c-4b4b-9efd-053fbecb1df1

📥 Commits

Reviewing files that changed from the base of the PR and between 71ff50a and db82c38.

📒 Files selected for processing (3)
  • packages/web/src/components/project/all-studies-tab/AllStudiesTab.tsx
  • packages/web/src/components/project/all-studies-tab/StudiesExplainer.tsx
  • packages/web/src/components/project/setup/ProjectSetupCard.tsx

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@InfinityBowman
InfinityBowman merged commit be7fb40 into main Sep 13, 2026
10 checks passed
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