Skip to content

merge: dev → main — OG banner refinement (full-bleed + github contribution widget)#14

Merged
vkkatariya merged 4 commits into
mainfrom
dev
Jun 30, 2026
Merged

merge: dev → main — OG banner refinement (full-bleed + github contribution widget)#14
vkkatariya merged 4 commits into
mainfrom
dev

Conversation

@vkkatariya

Copy link
Copy Markdown
Owner

Summary

Refinement of the OG sharing banner deployed earlier today (PR #10). Replaces prototypes/assets/og-image.png with a redesigned version that:

  • Full-bleed within 1200×630 canvas — removed dark margin around the OG card
  • V-centered body — content is now centered vertically rather than top-anchored
  • Scaled fonts — logo/name/pills/contact fonts enlarged to fill the larger card
  • New GitHub contribution sub-widget — 26×5 heatmap, lighter glass treatment, placed next to the "open to work" pill; replicates the site's real .cg/.cc widget pattern
  • Fixed name overflow — tuned font-size so the calligraphic "Vishal Katariya" doesn't bleed into the email pill

The OG meta tags in the 7 HTML files don't need to change — the file path is the same.

What's in this PR (4 commits)

96d34e6 Merge pull request #13 from vkkatariya/fix/og-banner-margin
c1b4c26 docs(devlog): add fix/og-banner-margin session entry
0f9867a fix(portfolio): full-bleed OG banner, scale content, add github contribution widget
7688421 docs(workflow): final cleanup — remove stale lineage refs from session startup block

Note: 7688421 was already on main from the v3 dual-session cleanup, but is included in this PR's diff because the fix branch was created from before that commit.

Files changed

  • prototypes/assets/og-image.png — replaced (96KB → 148KB, +55%)
  • tasks/DEVLOG.md — session entry for the OG refinement work
  • CLAUDE.md — minor (already on main as part of v3 cleanup, included in diff because the fix branch was off dev)
  • tasks/lessons.md — minor (L-070 added, already on main, included for same reason)

Visual review

  • 6 rounds of visual review with Vishal before commit
  • L-068 compliance: visual verification done before merge
  • Agent's commit message: "full-bleed OG banner, scale content, add github contribution widget"

Vercel deployment

Auto-deploys on push to main. The OG image will be live at https://vishal-katariya.com/og-image.png (and used as preview for any link shared to the site).

Audit notes (Hermes, 2026-06-30)

✅ Only 1 binary file modified (no breaking code changes)
✅ 1 devlog entry (mandatory per AGENTS.md)
✅ Both file types (96KB → 148KB) are reasonable for a 1200×630 banner
✅ Pixel diff: 51% of pixels changed — consistent with "redesign + new widget + scale fonts", not a small tweak
✅ All work was on fix/og-banner-margin sub-branch (per branch discipline)
✅ PR #13 was the standard dev target, not main (so this is a normal dev→main integration)

vkkatariya and others added 4 commits June 30, 2026 23:09
…n startup block

Leftover stale references to the old v2 dual-session model:
- Workflow references line still mentioned "lineage branches" as content
- Cross-session handoff section described the old "cloud-session-start"
  marker pattern that no longer exists in the new v3 model

These are documentation cleanup only — no architectural change.
…ibution widget

Removed dark margin around the OG card (full-bleed within 1200x630 canvas,
v-centered body). Scaled up logo/name/pills/contact fonts to fill the
larger card. Added a scaled-down github contribution sub-widget
(26x5 heatmap, lighter glass) next to the "open to work" pill, replicating
the site's real .cg/.cc widget. Fixed name overflow into the email pill
by tuning font-size. 6 rounds of visual review with Vishal before commit.
fix(portfolio): full-bleed OG banner + github contribution widget
@vercel

vercel Bot commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

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

Project Deployment Actions Updated (UTC)
portfolio-website Ready Ready Preview, Comment Jun 30, 2026 9:41pm

@vkkatariya vkkatariya merged commit e281c9f into main Jun 30, 2026
2 checks passed
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