Repository navigation
feat(struttura): unify the organisation chart and align /about with Figma - #202
Merged
BIA3IA merged 7 commits intoNov 2, 2026
Merged
Conversation
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
|
This was referenced Oct 27, 2026
viganogabriele
marked this pull request as ready for review
October 31, 2026 17:54
BIA3IA
force-pushed
the
feat/chi-siamo-visual-refinement
branch
from
November 2, 2026 13:53
c0076fd to
ac36921
Compare
BIA3IA
force-pushed
the
feat/struttura-visual-refinement
branch
3 times, most recently
from
November 2, 2026 14:32
bfde305 to
85a1cb3
Compare
BIA3IA
force-pushed
the
feat/chi-siamo-visual-refinement
branch
from
November 2, 2026 14:32
0c8a2ae to
24b822c
Compare
- Hero: Figma's centred XL title (fluid) with the two anchor actions; the rotated portrait collage and floating "200+" chip are gone (the board portraits already appear under "I nostri direttivi"). - "Come è nato tutto": the two 2016/2022 cards that repeated the timeline right below are removed; the text stands alone until the archive photo Figma reserves exists. - Timeline: nodes on a line (vertical below lg, horizontal from lg) instead of four hoverable glass cards; the year is the link to its chapter. - Chapters: numeral in the margin and one continuous vertical rule across all six; diagrams drawn without containers (icon chains, 5 → ~25 → 4, before → after columns, status list) instead of stacks of glass cards. - No hover on things that are not links (board portraits, the closing statement); "L'obiettivo è raggiunto" is a statement, not a card. - Projects note, thanks list, board summary and "La storia continua" lose their glass panels; the closing gradient panel and the filled "?" pill (which read as a button) become years on a line with an open slot. - One container for every section (Figma's 1272px column, 24/40/64px gutters) and left alignment at every width for this long reading page. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Each milestone carries its own anchor and icon instead of two arrays kept in step by position. The chapter numeral column is 3rem, wide enough for "02" at 36px. The closing statement is a div: a figure without a caption has no meaning here.
Improve the page where Figma v1 (implemented) and v2 agree, without switching to v2, which is still an open design decision. - Hero: drop the two CTAs (in neither version), add the "La nostra visione" label both versions use, same top spacing as /storia. - Il problema: questions become compact non-interactive chips (icon first) in a 5/7 rail, instead of three tall button-like cards. - I numeri: figures at headline size beside the icon (they were 45px and outranked the section titles). - Da dove veniamo / Come siamo organizzati and Sostieni: free columns on the background; no glass cards holding section titles, no nested panel, no 57px "5x1000" and 36px tax code. - One container, gutters and title/lead roles shared with /storia and the homepage; text left-aligned on mobile. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The history blocks use SECTION_TITLE like every other section on the page. The #cosa-facciamo anchor lost its only link in this PR, so it goes too.
…igma - One OrganizationDiagram shared by /about and /storia instead of two copies. Nodes follow Figma's "Nodo organigramma" (glass tile, icon above a 22px label, 1px connectors); no CardIcon className overrides. - Mobile chart follows Figma's vertical "Sezione organigramma": team leads hang off a rail, Membri Team balances the admin column. - Soci box gets Figma's "Ruolo trasversale" tag and shares the chart's column width. - Board portraits no longer lift on hover (they are not links); the mobile carousel bleeds to the edge so the next portrait peeks in. - "Come funziona" and "Trova il tuo posto": plain text-primary titles as in Figma, no icon on the definitions. - Same container as /storia and /chi-siamo; the closing title uses the grey gradient (style guide: decorated background or coloured text). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…ortrait card - Mobile chart: the stem under Team growth now turns left and meets the rail the team leads hang from; before it stopped in mid-air. - Node takes a `fill` flag instead of six call sites cancelling its min-width and padding. - The chart's accessible label lists the teams from teamLeads. - BoardMemberCard replaces the two copies of the Direttivo portrait in /about and /storia. - Comments that narrated Figma or the change are removed; the xl overlap on /storia gets one explaining why.
BIA3IA
force-pushed
the
feat/chi-siamo-visual-refinement
branch
from
November 2, 2026 14:35
24b822c to
bb21321
Compare
BIA3IA
force-pushed
the
feat/struttura-visual-refinement
branch
from
November 2, 2026 14:35
85a1cb3 to
b3fd927
Compare
Base automatically changed from
feat/chi-siamo-visual-refinement
to
feat/home-visual-refinement
November 2, 2026 14:45
BIA3IA
force-pushed
the
feat/home-visual-refinement
branch
from
November 2, 2026 14:56
57dda2d to
b10fcbf
Compare
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.
This PR covers
/about(Struttura): Figma5625:7108for desktop,6210:11694for mobile. It also unifies the organisation chart with/storia.Stacked PR: the base is #201 (
/chi-siamo), which sits on #200 (/storia) and #199 (home). Review the diff againstfeat/chi-siamo-visual-refinement.What changes
One organisation chart instead of two
/aboutand/storiaeach had their own copy of the chart, around 200 lines each. Both now renderOrganizationDiagramfromcomponents/structure/organization-chart.tsx./storiaPR (feat(storia): de-cardify the history page and align it with Figma #200) left the chart for this one on purpose, to avoid unifying it twice.Desktop: nodes as in Figma
5625:7108) the nodes are glass tiles, with a 40 % fill and the GLASS effect. The audit said they were free-standing, which was wrong. The tiles stay.CardIconwith[&_h3]:…overrides.xl, labels may wrap instead of overflowing the narrow admin column./storiachart at 1280 px squeezed "Capi admin" against the tile edges.Mobile: Figma's vertical "Sezione organigramma"
Soci
Il Direttivo in carica
Come funziona l'associazione
text-primarywith no icon, as in Figma. Before, they had a blue gradient title and an icon.Trova il tuo posto
text-primary, as in Figma. Icons and CTAs are unchanged.Closing "Tutto ciò che c'è da sapere"
159:200: decorated background or coloured text).Container
/storiaand/chi-siamo: max 1400 px, gutters 24/40/64 px. Before, it was 1320 px with a fixed 24 px gutter.Shared changes and their reach
OrganizationDiagram(incomponents/structure) now renders the chart in/storiatoo/storia"Come è organizzata PoliNetwork": same structure, Figma node sizes, new mobile layout. Checked at 320–1920 px.components/ui,card-icon,glassor the carouselScreenshots
Chart, 1440 px (before → after)

Chart, 390 px (before → after)

The same chart in /storia, 1280 px

Full page




Verification
pnpm check,pnpm build,pnpm typecheckandnode --test tests/*.test.mjs(2/2) all pass./aboutand/storiawere checked in the browser at 320, 390, 768, 1024, 1279, 1280, 1440, 1728 and 1920 px.Trade-offs
xl. At 1024–1279 px, and in/storia's narrower chart, "Capi admin" wraps to two lines. Shrinking the label to fit would undo Figma's hierarchy.storia/constants,chi-siamo/constantsandstructure/constants. Hoisting it into a shared layout primitive belongs with the container/rhythm foundation, not this page PR.Deliberately left out
/join. Removing one is a content and IA decision.ui/herotitle is shared with other pages and is unchanged.Housekeeping
The screenshots live on the
pr-assets/struttura-visual-refinementbranch, which can be deleted after review.🤖 Generated with Claude Code