Conversation
…nd refresh hero image Co-Authored-By: Ilya Timonin <timoncool@gmail.com>
Author
🤖 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:
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
3 tasks
timoncool
added a commit
that referenced
this pull request
Nov 3, 2025
# Fix landing page hero to use same screenshot as README ## Summary This PR addresses user feedback to align the landing page hero image with the README. The landing page now uses the same screenshot (`screenshot-app.png`) that appears in the README as "Video Editor — Main Interface" instead of using a separate hero-specific image. **Changes:** - Updated `landing-hero.tsx` to reference `/screenshot-app.png` instead of `/hero-app-v2.png` - Removed unused `hero-app-v2.png` file This ensures visual consistency between the landing page and README documentation. ## Review & Testing Checklist for Human **Risk Level: 🟡 Yellow** - Visual/UX change requiring human verification - [ ] **Verify landing page appearance**: Visit the Vercel preview and confirm the landing page hero displays correctly with `screenshot-app.png` and looks visually appealing/professional - [ ] **Check for cache issues**: After deployment, verify the new screenshot shows up (may need hard refresh Ctrl+Shift+R if browser cached the old image) - [ ] **Visual consistency check**: Confirm using the same screenshot for both landing hero and README "Main Interface" makes sense from a user experience perspective ### Test Plan 1. Open the Vercel preview URL (check CI comments for link) 2. Verify the landing page hero shows the full application interface screenshot 3. Compare to the first screenshot in README.md to confirm they're identical 4. After merge to production, do a hard refresh and verify the hero image updated ### Notes - This change was made in response to strong user feedback: "why didn't you put the first screenshot like in README on the landing page?" - The previous PR (#199) created a separate `hero-app-v2.png` file, but the user wanted consistency with the README instead - The `screenshot-app.png` file (1.1MB) shows the full VideoSOS interface with media gallery, timeline, and generate panel - No functionality changes - purely visual alignment --- **Link to Devin run:** https://app.devin.ai/sessions/a2ec44394e724a3ba98c151755b619c4 **Requested by:** Ilya Timonin (timoncool@gmail.com) / @timoncool
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Replace export dialog with impressive Project Statistics screenshot
Summary
This PR addresses user feedback about screenshot choices in the README files. The previous PR (#198) included an "Export Dialog" screenshot that the user found unimpressive. This PR replaces it with a more visually striking "Project Statistics" screenshot that showcases the app's analytics and cost tracking capabilities.
Additionally, the landing page hero image filename was changed from
hero-app.pngtohero-app-v2.pngto force cache invalidation, addressing the user's concern that "the landing page wasn't updated."Changes:
Review & Testing Checklist for Human
Risk Level: 🟡 Yellow - Visual/marketing changes requiring human verification
Test Plan
Notes
screenshot-export.pngfile remains in the repo but is no longer referenced anywhere. Consider deleting it in a future cleanup if it won't be used.Link to Devin run: https://app.devin.ai/sessions/a2ec44394e724a3ba98c151755b619c4
Requested by: Ilya Timonin (timoncool@gmail.com) / @timoncool