Skip to content

Conversation

@ComputelessComputer
Copy link
Collaborator

No description provided.

@ComputelessComputer ComputelessComputer merged commit 99f3e18 into main Nov 10, 2025
2 checks passed
@ComputelessComputer ComputelessComputer deleted the c-branch-1 branch November 10, 2025 16:08
@coderabbitai
Copy link

coderabbitai bot commented Nov 10, 2025

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

Three view routes modified: hero text removed from enterprise page, UI styling and layout adjustments applied to AI notetaking page including text line breaks and icon colors, and mini-apps page refactored from monolithic structure to modular components with separate sections.

Changes

Cohort / File(s) Summary
Enterprise page cleanup
apps/web/src/routes/_view/enterprise.tsx
Removed paragraph text "We'll help you with your concerns" from hero section; hero copy now ends at "Meeting notes your team will love, with enterprise features when you need them."
AI notetaking styling updates
apps/web/src/routes/_view/product/ai-notetaking.tsx
Added explicit line breaks (<br> elements) in headings and paragraphs; changed Icon components from text-green-600 to text-stone-600; removed trailing periods in descriptive text; adjusted vertical spacing (space-y, pt/pb) and section wrappers across Search, Editor, Transcription, Summaries, Sharing, and Floating Panel sections; added responsive layout adjustments for mobile/desktop variations.
Mini-apps component refactor
apps/web/src/routes/_view/product/mini-apps.tsx
Refactored from monolithic in-file composition to modular React components: introduced HeroSection, ContactsSection, CalendarSection, DailyNotesSection, NoteshelfSection, AdvancedSearchSection, and CTASection; replaced miniApps array and getColorClasses logic; separated sections with SlashSeparator components; added responsive conditionals for mobile "Coming Soon" placeholders.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

  • mini-apps.tsx requires careful review of component decomposition logic and responsive conditionals to ensure layout and functionality remain intact across screen sizes
  • ai-notetaking.tsx involves multiple distributed styling changes across sections; verify color/spacing consistency and that responsive adjustments apply correctly
  • enterprise.tsx is straightforward text removal but confirm no unintended layout shifts

Possibly related PRs

  • fastrepl/hyprnote#1633: Modifies the same enterprise.tsx file with background utility token changes, indicating concurrent updates to the enterprise page.
  • fastrepl/hyprnote#1617: Adjusts SlashSeparator styling (border-y vs border-b), directly complementing this PR's use of SlashSeparator in the mini-apps refactor.

Suggested reviewers

  • yujonglee
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch c-branch-1

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 63bf3e6 and f93945f.

📒 Files selected for processing (3)
  • apps/web/src/routes/_view/enterprise.tsx (0 hunks)
  • apps/web/src/routes/_view/product/ai-notetaking.tsx (13 hunks)
  • apps/web/src/routes/_view/product/mini-apps.tsx (1 hunks)

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 and usage tips.

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