Repository navigation
fix(a11y): fit the 404 page on phones and declare the page language - #216
Merged
BIA3IA merged 5 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
|
viganogabriele
force-pushed
the
feat/not-found-visual-refinement
branch
from
October 31, 2026 11:25
16dc339 to
bb7541d
Compare
18 tasks
viganogabriele
marked this pull request as ready for review
October 31, 2026 17:54
BIA3IA
force-pushed
the
feat/not-found-visual-refinement
branch
from
November 2, 2026 13:53
bb7541d to
fcecb70
Compare
BIA3IA
force-pushed
the
feat/footer-visual-refinement
branch
from
November 2, 2026 14:06
246204d to
38f08a4
Compare
BIA3IA
force-pushed
the
feat/not-found-visual-refinement
branch
from
November 2, 2026 14:06
fcecb70 to
43f9e76
Compare
BIA3IA
force-pushed
the
feat/footer-visual-refinement
branch
from
November 2, 2026 14:20
38f08a4 to
a68245c
Compare
BIA3IA
force-pushed
the
feat/not-found-visual-refinement
branch
from
November 2, 2026 14:20
43f9e76 to
3995c1d
Compare
BIA3IA
force-pushed
the
feat/footer-visual-refinement
branch
from
November 2, 2026 14:32
a68245c to
c15b5d6
Compare
BIA3IA
force-pushed
the
feat/not-found-visual-refinement
branch
from
November 2, 2026 14:32
3995c1d to
152a419
Compare
BIA3IA
force-pushed
the
feat/footer-visual-refinement
branch
from
November 2, 2026 14:35
c15b5d6 to
cafc9f3
Compare
BIA3IA
force-pushed
the
feat/not-found-visual-refinement
branch
3 times, most recently
from
November 2, 2026 15:00
a0f06e7 to
f175d2c
Compare
BIA3IA
force-pushed
the
feat/footer-visual-refinement
branch
2 times, most recently
from
November 2, 2026 15:02
8f144eb to
eb0d4c3
Compare
BIA3IA
force-pushed
the
feat/not-found-visual-refinement
branch
2 times, most recently
from
November 2, 2026 15:14
7c880d8 to
09acb0a
Compare
BIA3IA
force-pushed
the
feat/footer-visual-refinement
branch
2 times, most recently
from
November 2, 2026 15:18
2119d69 to
ad920d0
Compare
BIA3IA
force-pushed
the
feat/not-found-visual-refinement
branch
from
November 2, 2026 15:19
09acb0a to
069e3e6
Compare
BIA3IA
force-pushed
the
feat/footer-visual-refinement
branch
from
November 2, 2026 15:26
ad920d0 to
e3005d4
Compare
BIA3IA
force-pushed
the
feat/not-found-visual-refinement
branch
from
November 2, 2026 15:26
069e3e6 to
e676a11
Compare
BIA3IA
force-pushed
the
feat/footer-visual-refinement
branch
from
November 2, 2026 15:28
e3005d4 to
93afac2
Compare
BIA3IA
force-pushed
the
feat/not-found-visual-refinement
branch
from
November 2, 2026 15:28
e676a11 to
a39582e
Compare
BIA3IA
force-pushed
the
feat/footer-visual-refinement
branch
from
November 2, 2026 16:03
93afac2 to
d660306
Compare
BIA3IA
force-pushed
the
feat/not-found-visual-refinement
branch
from
November 2, 2026 16:03
a39582e to
c5eb343
Compare
BIA3IA
force-pushed
the
feat/footer-visual-refinement
branch
from
November 2, 2026 16:05
d660306 to
e0df443
Compare
BIA3IA
force-pushed
the
feat/not-found-visual-refinement
branch
from
November 2, 2026 16:05
c5eb343 to
92f4d31
Compare
The footer was the official component's bare layout: a 57px blue gradient title, 24px icon targets, two link columns ~320px apart and three "#" policy links. It now follows "footer-francesca" (593:2123), the version used across the Figma pages: "Keep in touch!" with the social pill, a site map with the association's documents and the 5x1000 card, the contact addresses, the join and report-a-bug calls to action, and a hairline legal row. Only existing content is used: no TikTok, no language selector, and the Statuto and Regolamento interno in place of policy pages that don't exist. The Su di noi support section gets an anchor for the 5x1000 card. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The footer content now sits in one rounded glass panel inside the page gutters, closing the page as a single object like the header pill opens it. The 5x1000 card becomes a lighter tile instead of glass on glass and stacks where its column is narrow; the social targets go to 36px below `sm` so the row stays whole at 320px. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…t exists - The document links can wrap again: their nowrap set a 276px minimum that pushed the grid and the 5x1000 card out of the panel up to 356px. - The social pill is a light surface instead of glass on the glass panel, and CardMultipleIcons is styled for its only use rather than overridden from outside with [&>div] selectors. - The two calls to action align at the bottom when one heading wraps. - The title uses SECTION_TITLE; the statute, regulations and materials URLs come from their existing constants. - English labels carry lang="en"; the copyright years use an en dash. - Comments that narrated Figma or the change are removed.
The 404 message was 32px with no side padding, so it ran edge to edge at 320px: it gets the page gutters, a smaller size on phones and balanced lines. `<html lang>` was "en" on an Italian site, so screen readers and hyphenation used English rules; it is now "it". Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
py-40 made the page 764px tall at 320x568 and pushed the button below the fold. The top padding still clears the fixed header.
BIA3IA
force-pushed
the
feat/footer-visual-refinement
branch
from
November 2, 2026 16:19
d857698 to
1a4fb4b
Compare
BIA3IA
force-pushed
the
feat/not-found-visual-refinement
branch
from
November 2, 2026 16:19
92f4d31 to
17968c0
Compare
Base automatically changed from
feat/footer-visual-refinement
to
feat/home-visual-refinement
November 2, 2026 16:20
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.
Stacked on #215 (footer):
feat/footer-visual-refinement. This is the last part of the shared site chrome.Changes
headline-large(32 px) with no side padding, so at 320 px it ran edge to edge. It now has the 24 px page gutter,headline-extrasmallon phones (headline-largefromsm) andtext-balance, with vertical padding so it doesn't sit under the header on short screens. The content and the "Home" button are unchanged, and the page stays faithful to the Figma "error page" (1374:1907).<html lang="en">→"it". The site is in Italian, so screen readers were reading it with English pronunciation, and hyphenation andtext-balanceused English rules.Screenshots (before | after)
Verification
lang="it"and exactly oneh1on all 17 checked routes. No overflow on the 404 from 320 to 1920 px.pnpm check,pnpm build,pnpm typecheckand the tests (3/3) all pass.Merge order
#214 → #215 → this. The
pr-assets/not-found-visual-refinementbranch only holds the screenshots.🤖 Generated with Claude Code