Skip to content

Replace PR mockup with app screenshot#2719

Open
notkainoa wants to merge 2 commits into
pingdotgg:mainfrom
notkainoa:t3code/e521d56d
Open

Replace PR mockup with app screenshot#2719
notkainoa wants to merge 2 commits into
pingdotgg:mainfrom
notkainoa:t3code/e521d56d

Conversation

@notkainoa
Copy link
Copy Markdown
Contributor

@notkainoa notkainoa commented May 15, 2026

What Changed

Replaced the mock PR/cards visual in the marketing site section with an app screenshot.

Why

The mocked card UI was more placeholder than product-representative. Using a real image makes that section clearer and more accurate, while the sizing cleanup keeps the layout balanced.

UI Changes

Before:

Screenshot 2026-05-15 at 9 45 16 AM

After:

Screenshot 2026-05-15 at 10 12 54 AM

Checklist

  • This PR is small and focused
  • I explained what changed and why
  • I included before/after screenshots for any UI changes
  • I included a video for animation/interaction changes

Note

Low Risk
Low risk marketing-only UI change that swaps a mocked component for a static image and tweaks layout CSS; main risk is minor visual/regression across breakpoints.

Overview
Updates the marketing homepage Git section to replace the inline PR/card mock UI with a static screenshot (/git-pr-workflow.png).

Simplifies and adjusts the related styling by removing the PR mockup CSS and centering/scaling the new image (.git-inner alignment, .git-image-shell, .git-image) to keep the layout balanced.

Reviewed by Cursor Bugbot for commit a4ecc7d. Bugbot is set up for automated code reviews on this repo. Configure here.

Note

Replace PR mockup with a static screenshot in the Git section of the marketing index page

Swaps the hand-coded PR card UI (with trail and buttons) in index.astro for a lazily loaded <img> referencing git-pr-workflow.png. Updates .git-visual to center content and .git-inner to use align-items: stretch, and removes all CSS for the deleted PR card classes.

Macroscope summarized a4ecc7d.

notkainoa added 2 commits May 15, 2026 10:11
- Swap the handcrafted git PR illustration for a real screenshot
- Add the workflow image to the marketing page public assets
- Stretch the grid rows so the visual column matches the text height
- Center the image wrapper and constrain the illustration with contain sizing
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 15, 2026

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 70a82b86-f095-41e2-bc34-4e1880fbc721

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@github-actions github-actions Bot added the vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. label May 15, 2026
@notkainoa notkainoa marked this pull request as ready for review May 15, 2026 17:18
@github-actions github-actions Bot added the size:L 100-499 changed lines (additions + deletions). label May 15, 2026
@macroscopeapp
Copy link
Copy Markdown
Contributor

macroscopeapp Bot commented May 15, 2026

Approvability

Verdict: Approved

Marketing-only change that replaces a hand-coded HTML/CSS mockup with a screenshot image. No application logic or runtime behavior affected.

You can customize Macroscope's approvability policy. Learn more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L 100-499 changed lines (additions + deletions). vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant