-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Add dropdown menu to recent projects cards #3041
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Add dropdown menu to recent projects cards #3041
Conversation
- Added dropdown menu with rename, clone, convert to template, and delete options to SquareProjectCardPresentation component - Updated SelectProjectPresentation to pass dropdown callbacks to recent projects cards - Matches functionality from ProjectCardPresentation in the Projects section Co-Authored-By: Satya Patel <satyapatel111@gmail.com>
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
|
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the Note Other AI code review bot(s) detectedCodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review. Comment |
| <div className="absolute top-3 right-3 z-30 opacity-0 transition-opacity duration-200 group-hover:opacity-100"> | ||
| <DropdownMenu> | ||
| <DropdownMenuTrigger asChild> | ||
| <Button |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Consider adding an aria-label (e.g. aria-label="Project actions") to the dropdown trigger button for better accessibility.
- Replaced crypto.randomUUID() with uuidv4() from uuid package - Fixes Chromatic CI failure due to crypto not being available in Node environment Co-Authored-By: Satya Patel <satyapatel111@gmail.com>
Description
Added a dropdown menu to the Recent projects cards (in the carousel) that matches the functionality already present in the Projects section cards. The dropdown provides quick access to project management actions:
The implementation mirrors the existing
ProjectCardPresentationcomponent to ensure UI consistency.Related Issues
Related to Storybook story:
pages-projectspage--creating-project(Pages / ProjectsPage / Creating Project)Type of Change
Changes Made
Modified Files:
square-project-card-presentation.tsxonRename,onClone,onToggleTemplate,onDelete,isTemplateselect-presentation.tsxSquareProjectCardPresentationTesting
Recommended testing steps:
bun run storybookinapps/web/clientHuman Review Checklist
Please verify:
Additional Notes
ProjectCardPresentationto ensure UI consistencyisTemplateprop is determined by checking for 'template' inproject.metadata.tagsLink to Devin run: https://app.devin.ai/sessions/202b2eaabf054572938b39101cf916e3
Requested by: Satya Patel (@saddlepaddle)
Important
Adds a dropdown menu to recent projects cards in
SquareProjectCardPresentationfor project management actions, ensuring UI consistency with existing project cards.SquareProjectCardPresentationfor recent projects cards with options: Rename, Clone, Convert/Unmark as template, Delete.ProjectCardPresentationstyling for UI consistency.square-project-card-presentation.tsx: Implements dropdown menu with callbacksonRename,onClone,onToggleTemplate,onDelete, andisTemplateprop.select-presentation.tsx: Passes dropdown callbacks toSquareProjectCardPresentationand connects handlers for project actions.This description was created by
for 8d004cf. You can customize this summary. It will automatically update as commits are pushed.