Skip to content

feat(ui): Add quick access items to empty state actions#2282

Merged
ComputelessComputer merged 1 commit intomainfrom
feat/quick-access-empty-state
Dec 14, 2025
Merged

feat(ui): Add quick access items to empty state actions#2282
ComputelessComputer merged 1 commit intomainfrom
feat/quick-access-empty-state

Conversation

@ComputelessComputer
Copy link
Collaborator

No description provided.

@netlify
Copy link

netlify bot commented Dec 13, 2025

Deploy Preview for hyprnote-storybook ready!

Name Link
🔨 Latest commit 64fe123
🔍 Latest deploy log https://app.netlify.com/projects/hyprnote-storybook/deploys/693d9d140c112c0008255758
😎 Deploy Preview https://deploy-preview-2282--hyprnote-storybook.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 13, 2025

📝 Walkthrough

Walkthrough

The change reorganizes a menu structure by promoting Calendar and Contacts actions from a dropdown to top-level menu items, while restructuring the "Others" dropdown to include Templates, Shortcuts, and Prompts instead. Existing click handlers are preserved during the reorganization.

Changes

Cohort / File(s) Summary
Menu reorganization
apps/desktop/src/components/main/body/empty/index.tsx
Moved Calendar and Contacts actions to top-level menu; restructured "Others" dropdown by removing Calendar/Contacts entries and adding Templates, Shortcuts, and Prompts; preserved all existing onClick handlers

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

  • Verify that all handler references (openCalendar, openContacts, and handlers for Templates/Shortcuts/Prompts) are correctly assigned after reorganization
  • Confirm that no handlers were duplicated, misaligned, or accidentally removed during the restructuring
  • Validate the visual layout hierarchy of top-level vs. dropdown menu items

Pre-merge checks and finishing touches

❌ Failed checks (1 warning, 1 inconclusive)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
Description check ❓ Inconclusive No pull request description was provided by the author, making it impossible to assess whether it relates to the changeset. Add a brief description explaining what quick access items were added, why they were moved from the Others dropdown, and any relevant context for reviewers.
✅ Passed checks (1 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: adding quick access items (Calendar and Contacts) to the empty state actions, which is the primary focus of the changeset.
✨ 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 feat/quick-access-empty-state

📜 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 2545475 and 64fe123.

📒 Files selected for processing (1)
  • apps/desktop/src/components/main/body/empty/index.tsx (2 hunks)
🧰 Additional context used
📓 Path-based instructions (1)
**/*.{ts,tsx}

📄 CodeRabbit inference engine (AGENTS.md)

**/*.{ts,tsx}: Avoid creating a bunch of types/interfaces if they are not shared. Especially for function props, just inline them instead.
Never do manual state management for form/mutation. Use useForm (from tanstack-form) and useQuery/useMutation (from tanstack-query) instead for 99% of cases. Avoid patterns like setError.
If there are many classNames with conditional logic, use cn (import from @hypr/utils). It is similar to clsx. Always pass an array and split by logical grouping.
Use motion/react instead of framer-motion.

Files:

  • apps/desktop/src/components/main/body/empty/index.tsx
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (7)
  • GitHub Check: Redirect rules - hyprnote
  • GitHub Check: Header rules - hyprnote
  • GitHub Check: Pages changed - hyprnote
  • GitHub Check: fmt
  • GitHub Check: desktop_ci (macos, depot-macos-14)
  • GitHub Check: desktop_ci (linux, depot-ubuntu-22.04-8)
  • GitHub Check: desktop_ci (linux, depot-ubuntu-24.04-8)
🔇 Additional comments (3)
apps/desktop/src/components/main/body/empty/index.tsx (3)

75-76: LGTM! Calendar and Contacts actions added successfully.

The new top-level actions are properly wired to their respective callbacks and follow the same pattern as the existing "New Note" action.


91-93: LGTM! Dropdown actions updated correctly.

The "Others" dropdown now contains Templates, Shortcuts, and Prompts with proper click handlers. The reorganization makes the quick access items more prominent.


50-69: The code correctly uses all valid tab types defined in the schema. All five callbacks (openCalendar, openContacts, openTemplates, openShortcuts, openPrompts) pass valid tab types to openCurrent:

  • "extension" with extensionId: "calendar"
  • "contacts"
  • "templates"
  • "chat_shortcuts"
  • "prompts"

No changes needed.


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.

@netlify
Copy link

netlify bot commented Dec 13, 2025

Deploy Preview for hyprnote ready!

Name Link
🔨 Latest commit 64fe123
🔍 Latest deploy log https://app.netlify.com/projects/hyprnote/deploys/693d9d14b49c26000830e114
😎 Deploy Preview https://deploy-preview-2282--hyprnote.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@ComputelessComputer ComputelessComputer merged commit b37b9ab into main Dec 14, 2025
16 checks passed
@ComputelessComputer ComputelessComputer deleted the feat/quick-access-empty-state branch December 14, 2025 01:36
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