Skip to content

Fix landing page hero to use same screenshot as README#200

Merged
timoncool merged 1 commit into
mainfrom
devin/1762164572-fix-landing-hero-screenshot
Nov 3, 2025
Merged

Fix landing page hero to use same screenshot as README#200
timoncool merged 1 commit into
mainfrom
devin/1762164572-fix-landing-hero-screenshot

Conversation

@devin-ai-integration
Copy link
Copy Markdown

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 (Replace export dialog with impressive Project Statistics screenshot #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

Co-Authored-By: Ilya Timonin <timoncool@gmail.com>
@devin-ai-integration
Copy link
Copy Markdown
Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@vercel
Copy link
Copy Markdown

vercel Bot commented Nov 3, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
videosos Ready Ready Preview Comment Nov 3, 2025 10:11am

@timoncool timoncool merged commit 33a2c0f into main Nov 3, 2025
8 checks passed
@timoncool timoncool deleted the devin/1762164572-fix-landing-hero-screenshot branch November 3, 2025 10:11
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