feat: update home prompt input and dynamic tagline#182
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
Important
Looks good to me! 👍
Reviewed everything up to 8f95c44 in 7 seconds. Click for details.
- Reviewed
716lines of code in2files - Skipped
0files when reviewing. - Skipped posting
0draft comments. View those below. - Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
Workflow ID: wflow_DyS0GQeg1awJs4Fl
You can customize by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.
|
Caution Review failedThe pull request is closed. 📝 WalkthroughWalkthroughThe PR updates the home page's main tagline text and styling, and substantially restructures the prompt input component to include an intro typing sequence, improved PDF/URL upload handling with drag-drop support, and simplified placeholder logic with conditional prefix typing. Changes
Estimated code review effort🎯 4 (Complex) | ⏱️ ~50 minutes Possibly related PRs
Poem
✨ Finishing touches
🧪 Generate unit tests (beta)
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. Comment |
|
Greptile OverviewGreptile SummaryUpdates the home hero experience by:
The main integration remains the same: PDFs are uploaded via Confidence Score: 4/5
Important Files Changed
Sequence DiagramsequenceDiagram
participant U as User
participant H as HomePromptInput
participant DZ as react-dropzone
participant UP as usePdfUpload
participant API as useCreateWorkspaceFromPrompt
participant R as Next Router
U->>H: Type prompt / press Enter
H->>API: mutate(prompt, {template, initialState})
API-->>H: onSuccess(workspace)
H->>UP: clearFiles()
H->>R: push(/workspace/:slug?params)
U->>DZ: Drag & drop PDFs
DZ-->>H: onDrop(acceptedFiles)
H->>UP: uploadFiles(acceptedFiles)
UP-->>H: uploadedFiles updated
H-->>U: toast success / error
|
| import { usePdfUpload } from "@/hooks/workspace/use-pdf-upload"; | ||
| import { ArrowUp, FileText, Loader2, Plus, Upload, X, Link as LinkIcon } from "lucide-react"; | ||
| // import { useImageUpload } from "@/hooks/workspace/use-image-upload"; | ||
| import { ArrowUp, FileText, Loader2, Upload, X, Link as LinkIcon } from "lucide-react"; |
There was a problem hiding this comment.
Unused imports cause lint fail
resolvedTheme (line 59) and baseText (line 51) are unused, and Button is imported but never used as a JSX component (only <button> is used). If this repo runs eslint/tsc with noUnusedLocals/noUnusedVars, this will fail CI.
| import { ArrowUp, FileText, Loader2, Upload, X, Link as LinkIcon } from "lucide-react"; | |
| import { useState, useRef, useMemo, useEffect } from "react"; | |
| import { useRouter } from "next/navigation"; | |
| import { toast } from "sonner"; | |
| import { useCreateWorkspaceFromPrompt } from "@/hooks/workspace/use-create-workspace"; | |
| import { usePdfUpload } from "@/hooks/workspace/use-pdf-upload"; | |
| // import { useImageUpload } from "@/hooks/workspace/use-image-upload"; | |
| import { ArrowUp, FileText, Loader2, Upload, X, Link as LinkIcon } from "lucide-react"; | |
| // import { ImageIcon } from "lucide-react"; | |
| import { cn } from "@/lib/utils"; | |
| import { Input } from "@/components/ui/input"; | |
| import TypingText from "@/components/ui/typing-text"; | |
| import { useDropzone } from "react-dropzone"; |



Contributors: @ibraheemshaikh5
Important
Updates
DynamicTaglinetext and styling, and enhancesHomePromptInputwith new placeholders, PDF handling, and dynamic typing.font-lightand addstracking-tight.This description was created by
for 8f95c44. You can customize this summary. It will automatically update as commits are pushed.
Summary by CodeRabbit
Release Notes
New Features
Updates