feat(case-study): add Espai de Dades — PortalJS Cloud portal for Catalonia's third sector#1606
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
7 Skipped Deployments
|
|
📝 WalkthroughWalkthroughA new case study markdown file is added for "Espai de Dades / Open Data Portal for Catalonia's Third Sector." It contains frontmatter metadata, keystats, problem/solution/results sections, a features list, a testimonial quote, a portal link, four FAQ entries, and a multi-paragraph narrative. ChangesEspai de Dades Case Study
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@site/layouts/casestudy.tsx`:
- Around line 374-376: Replace the span element that contains "The story" text
with an h2 heading element while preserving all the existing className
attributes (font-mono text-xs font-semibold uppercase tracking-[0.18em]
text-blue-600 dark:text-blue-400). This improves accessibility for assistive
technologies by using a semantic heading element instead of a generic span for
the section title.
- Around line 743-745: The inline arrow SVG element within the CTA link is
decorative and should be hidden from assistive technologies to prevent
unnecessary screen reader announcements. Add the aria-hidden="true" attribute to
the SVG element that contains the path with the arrow icon (the one with
viewBox="0 0 16 16") to mark it as purely decorative and exclude it from
accessibility announcements.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 0650b7a7-5ccf-43f5-b085-ac5759370621
📒 Files selected for processing (7)
site/content/case-studies/espai-de-dades-third-sector-data-portal.mdsite/layouts/casestudy.tsxsite/public/images/casestudies/espai-de-dades-dataset.webpsite/public/images/casestudies/espai-de-dades-featured.webpsite/public/images/casestudies/espai-de-dades-homepage.webpsite/public/images/casestudies/espai-de-dades-search.webpsite/public/images/casestudies/onetandem-logo.webp
| <span className="font-mono text-xs font-semibold uppercase tracking-[0.18em] text-blue-600 dark:text-blue-400"> | ||
| The story | ||
| </span> |
There was a problem hiding this comment.
🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win
Use a semantic heading element for the section title.
Line 374 renders the section title with a <span>, which weakens heading navigation for assistive tech. Use an h2 (or the correct heading level) with the same classes.
Suggested fix
- <span className="font-mono text-xs font-semibold uppercase tracking-[0.18em] text-blue-600 dark:text-blue-400">
+ <h2 className="font-mono text-xs font-semibold uppercase tracking-[0.18em] text-blue-600 dark:text-blue-400">
The story
- </span>
+ </h2>📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| <span className="font-mono text-xs font-semibold uppercase tracking-[0.18em] text-blue-600 dark:text-blue-400"> | |
| The story | |
| </span> | |
| <h2 className="font-mono text-xs font-semibold uppercase tracking-[0.18em] text-blue-600 dark:text-blue-400"> | |
| The story | |
| </h2> |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@site/layouts/casestudy.tsx` around lines 374 - 376, Replace the span element
that contains "The story" text with an h2 heading element while preserving all
the existing className attributes (font-mono text-xs font-semibold uppercase
tracking-[0.18em] text-blue-600 dark:text-blue-400). This improves accessibility
for assistive technologies by using a semantic heading element instead of a
generic span for the section title.
| <svg viewBox="0 0 16 16" fill="none" stroke="currentColor" strokeWidth="2" strokeLinecap="round" strokeLinejoin="round" className="w-4 h-4"> | ||
| <path d="M3 8h10M9 4l4 4-4 4" /> | ||
| </svg> |
There was a problem hiding this comment.
🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win
Hide decorative CTA SVG from assistive technologies.
The inline arrow is decorative; mark it aria-hidden and non-focusable to avoid noisy link announcements.
Suggested fix
- <svg viewBox="0 0 16 16" fill="none" stroke="currentColor" strokeWidth="2" strokeLinecap="round" strokeLinejoin="round" className="w-4 h-4">
+ <svg
+ viewBox="0 0 16 16"
+ fill="none"
+ stroke="currentColor"
+ strokeWidth="2"
+ strokeLinecap="round"
+ strokeLinejoin="round"
+ className="w-4 h-4"
+ aria-hidden="true"
+ focusable="false"
+ >
<path d="M3 8h10M9 4l4 4-4 4" />
</svg>📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| <svg viewBox="0 0 16 16" fill="none" stroke="currentColor" strokeWidth="2" strokeLinecap="round" strokeLinejoin="round" className="w-4 h-4"> | |
| <path d="M3 8h10M9 4l4 4-4 4" /> | |
| </svg> | |
| <svg | |
| viewBox="0 0 16 16" | |
| fill="none" | |
| stroke="currentColor" | |
| strokeWidth="2" | |
| strokeLinecap="round" | |
| strokeLinejoin="round" | |
| className="w-4 h-4" | |
| aria-hidden="true" | |
| focusable="false" | |
| > | |
| <path d="M3 8h10M9 4l4 4-4 4" /> | |
| </svg> |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@site/layouts/casestudy.tsx` around lines 743 - 745, The inline arrow SVG
element within the CTA link is decorative and should be hidden from assistive
technologies to prevent unnecessary screen reader announcements. Add the
aria-hidden="true" attribute to the SVG element that contains the path with the
arrow icon (the one with viewBox="0 0 16 16") to mark it as purely decorative
and exclude it from accessibility announcements.
…lonia's third sector New case study for Espai de Dades, built by OneTandem for Taula d'entitats del Tercer Sector de Catalunya. Includes hero image, 3 portal screenshots, and OneTandem logo. Content covers context narrative, challenge/solution/impact, 6 features, testimonial, FAQ, and hyperlinks to client and partner sites. Depends on: feat/casestudy-journey-redesign Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
popovayoana
force-pushed
the
feat/espai-de-dades-case-study
branch
from
9336d22 to
9a1570f
Compare
popovayoana
changed the title
2 participants
Summary
New case study for Espai de Dades — a public-facing open data portal built by OneTandem for Taula d'entitats del Tercer Sector de Catalunya using PortalJS Cloud.
Depends on #1608 — merge the Journey layout redesign first.
What's included
content/case-studies/espai-de-dades-third-sector-data-portal.md— full case study with:Test plan
/case-studies/espai-de-dades-third-sector-data-portal— hero, story intro, Journey columns, features, testimonial, portal screenshots, FAQs, CTA all render🤖 Generated with Claude Code