feat(press): inline asset previews + layout pass on /press - #2836
Open
0xkkonrad wants to merge 1 commit into
Open
feat(press): inline asset previews + layout pass on /press#28360xkkonrad wants to merge 1 commit into
0xkkonrad wants to merge 1 commit into
Conversation
Previews derived from frontmatter hrefs by extension (SVG marks unoptimized, five mascots in one well, KNERD specimen on pink), format pills with download, items-start grids, text-h4 headings, jump pills, dedup of the flagship headline, sizes + 2-col mobile team photos, two-column tail with a pink contact card, hero base size 6xl so the KNERD title no longer clips at 390px, and i18n for the boilerplate labels and photo alt text across all four locales.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
This was referenced Aug 26, 2026
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
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.
What
Improves
/press(the pagepeanut.me/presskitredirects to) — inline previews of the actual assets, and a layout pass. Stacks on #2811 (feat/press-brand-subpage); base is that branch so this PR shows only its own delta. If #2811 merges first, GitHub retargets this todev.Previews
next/imageunoptimized, nodangerouslyAllowSVG); Mascot shows all five.webpside by side in one well; the Font group renders a KNERD specimen viaTitleon pink (the three-layer offset the guidelines mandate); Brand Guidelines (PDF only) has no well and spans the row.SVG · EPS · AI) when unambiguous within the group, keep filenames otherwise, and carrydownloadon same-origin files (the GitHub link keepstarget=_blank).src/components/Marketing/pressAssets.tswith a 6-case jest test.Layout
#assets,#contact) at the top, zero new strings.items-starton the card grids (kills the stretched-card whitespace),text-h4section headings on both press pages,shadow-4only on the asset cards + contact card.sizeson thefillimages (browser now fetchesw=384instead ofw=3840), 2 columns on mobile, note moved above the grid.MarketingHero:text-6xl sm:text-7xl md:text-9xl— at 390px "PRESS"/"BRAND" were clipped by the hero'soverflow-hidden. Only the base breakpoint changes.Short / Medium / Press / Partner, the photo alt text, the one-liner label and a one-line assets note are now keys in all four locales.Verified
tsc --noEmit(0), jest 6/6.next dev --webpackDOM audit at 1000px and 390px on/en/press,/pt-br/press,/en/press/brand: no horizontal overflow, hero letterforms inside the viewport, all 16 previewed images load, 28 download links ≥ 44px tall, no English labels on pt-BR.Not in this PR (content-side, needs an owner)
content/team/en.md: second founder isKonradwith no surname → JSON-LDPerson"Konrad"; bios are the TODO placeholders.media_contactissupport@peanut.me.