Skip to content

feat(contact): align Contattaci with the shared layout and fix the card grid - #206

Merged
BIA3IA merged 4 commits into
feat/home-visual-refinementfrom
feat/contact-visual-refinement
Nov 2, 2026
Merged

BIA3IA merged 4 commits into
feat/home-visual-refinementfrom
feat/contact-visual-refinement

Conversation

@viganogabriele

Copy link
Copy Markdown
Contributor

Stacked on #205 (feat/collabora-visual-refinement); part of the page-by-page visual refinement started in #199. Merge the stack in order.

/contact (Figma desktop 5062:15713, mobile 5075:16345) was close to Figma at 1728 px but built on its own layout: a min-[1616px] breakpoint, gap-60/gap-75 between sections, an invalid max-w-[959], and centered text everywhere below 1616 px. This PR moves it onto the shared layout of the redesigned pages and fixes the problems the audit found.

What changes

Layout and rhythm

  • The page uses CONTAINER, SECTION_TITLE, SECTION_LEAD, PAGE_MAIN and PAGE_HERO from components/layout/page.ts. It now has the same column, title styles and section spacing as Storia, Su di noi, Struttura, Team, Unisciti and Collabora.
  • Text is left-aligned at every width, consistent with the other pages. Before, it was centered until 1616 px and left-aligned above.
  • At 1440 px the page goes from 4092 to 3134 px tall, with the same content. The extra height was oversized gaps (240–300 px), not content.

Email cards (the audit's "3 + 1 instead of 4 in a row")

  • The layout is a grid: four columns from xl as in Figma, 2×2 from sm, one column on phones. No card is ever alone on a row; before, flex-wrap left "Collaborazioni" orphaned at most widths.
  • Cards size to their content instead of the fixed 312×264 box, and rows stay even.
  • From xl the card titles step down from 24 to 20 px, so "Informazioni Generali" fits on one line and the descriptions line up across the row.
  • Only the email pills (mailto:) are interactive; the cards themselves have no hover.

"Hai mille dubbi sul Poli?"

  • "Trova il tuo gruppo" was a <button> with no action. It now links to /groups, the same destination as the inline link in the paragraph above it.
  • Text and photo sit side by side from xl, as in Figma. Below xl they stack, with the photo capped at 768 px. At 1024 px the two-column version squeezed the title onto four lines.
  • The photo keeps Figma's 2.1:1 crop at every width.

"Vuoi sviluppare un progetto insieme a noi?"

  • The text column is 768 px wide, keeping lines under ~75 characters.
  • The highlighted phrase uses text-blue-secondary (AA on the background) instead of text-blue-primary.

Shared changes and their reach

Change Other pages affected
ContactCard: no fixed size, title 20 px from xl none (only used on /contact)

Screenshots

All captures are full pages, before and after.

Desktop 1440
desktop 1440

Desktop 1279 (last width with the stacked layout)
desktop 1279

Tablet 1024
tablet 1024

Mobile 390
mobile 390

Verification

  • pnpm check, pnpm build, pnpm typecheck and node --test tests/*.test.mjs (2/2) all pass.
  • I checked the page in the browser at 320, 390, 768, 1024, 1180, 1279, 1280, 1366, 1440, 1728 and 1920 px. No horizontal overflow at any width.

Trade-offs

  • Phones are left-aligned. Figma's mobile frame centers the text; I kept left alignment for consistency with the other redesigned pages.
  • The layout switches at 1279/1280 px, where the photo moves beside the text (the page shortens by about 490 px). It is the same xl switch used by the other pages.

Deliberately left out

  • Copy is unchanged. In "Candidature" the link reads "Struttura" but points to /join; whether to change the label or the destination is a content decision.
  • The footer stays as it is, because it is an open design decision.

Housekeeping

The screenshots live on the pr-assets/contact-visual-refinement branch, which can be deleted after review.

🤖 Generated with Claude Code

@coderabbitai

coderabbitai Bot commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

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
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 316f8eb9-df62-4ddd-abba-1bec7487dce8

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
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@viganogabriele
viganogabriele marked this pull request as ready for review October 31, 2026 17:54
@BIA3IA
BIA3IA force-pushed the feat/collabora-visual-refinement branch from e50dd5e to 525492f Compare November 2, 2026 13:53
@BIA3IA
BIA3IA force-pushed the feat/contact-visual-refinement branch 2 times, most recently from a216c84 to c7fb4d8 Compare November 2, 2026 14:20
@BIA3IA
BIA3IA force-pushed the feat/collabora-visual-refinement branch from 16d3485 to e444f4d Compare November 2, 2026 14:32
@BIA3IA
BIA3IA force-pushed the feat/contact-visual-refinement branch from c7fb4d8 to 342273c Compare November 2, 2026 14:32
@BIA3IA
BIA3IA force-pushed the feat/collabora-visual-refinement branch from e444f4d to 58cb833 Compare November 2, 2026 14:35
@BIA3IA
BIA3IA force-pushed the feat/contact-visual-refinement branch from 342273c to 30bd5ec Compare November 2, 2026 14:35
@BIA3IA
BIA3IA force-pushed the feat/collabora-visual-refinement branch from 58cb833 to b8bf329 Compare November 2, 2026 14:56
@BIA3IA
BIA3IA force-pushed the feat/contact-visual-refinement branch 2 times, most recently from 014b639 to 3c64c05 Compare November 2, 2026 15:00
@BIA3IA
BIA3IA force-pushed the feat/collabora-visual-refinement branch 2 times, most recently from 586398d to 70bf9ce Compare November 2, 2026 15:02
@BIA3IA
BIA3IA force-pushed the feat/contact-visual-refinement branch 2 times, most recently from 4937b0e to d6de145 Compare November 2, 2026 15:14
@BIA3IA
BIA3IA force-pushed the feat/collabora-visual-refinement branch from 16a2f75 to b885e7b Compare November 2, 2026 15:18
@BIA3IA
BIA3IA force-pushed the feat/contact-visual-refinement branch from d6de145 to 54900b4 Compare November 2, 2026 15:19
Base automatically changed from feat/collabora-visual-refinement to feat/home-visual-refinement November 2, 2026 15:19
viganogabriele and others added 3 commits November 2, 2026 15:25
…rd grid

- Page uses the shared container, title/lead styles and vertical rhythm of
  the other redesigned pages instead of ad-hoc paddings and a 1616px
  breakpoint.
- Email cards: four in a row from xl as in Figma, 2×2 below, one column on
  phones, so a card is never left alone on a row; cards size to content
  instead of the fixed 264×312 box.
- "Trova il tuo gruppo" had no action: it now links to /groups, like the
  inline link in the same paragraph.
- Group photo keeps Figma's crop next to the text from xl and is capped
  when stacked; text is left-aligned at every width.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- The group photo is 512x246: it was declared 721x342 and stretched up to
  768px wide. It now shows at its real size at most.
- "Candidature" linked to /join with the label "Struttura"; the label now
  matches the page it opens.
- The inline links use INLINE_LINK, with a visible focus ring.
@BIA3IA
BIA3IA force-pushed the feat/contact-visual-refinement branch from cab415e to 7dccd60 Compare November 2, 2026 15:26
@BIA3IA
BIA3IA merged commit 63a8c11 into feat/home-visual-refinement Nov 2, 2026
2 checks passed
@BIA3IA
BIA3IA deleted the feat/contact-visual-refinement branch November 2, 2026 15:30
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.

2 participants