feat(parks): give the parks menu five researched, source-cited topics #10
No reviewers
Labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
PCivil/folletos-digitales!10
Loading…
Reference in a new issue
No description provided.
Delete branch "feat/parks-menu-style-options"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Summary
The palette question this PR opened is settled: both menus ship
tierra. What separates them now is content, not colour.The five park topics are risks that materialise during a visit, which is what makes them different from the site-wide temario. Earthquakes was a required inclusion.
ParkTopicis a sibling ofEmergencyTopic, not a variant. It drops the infographic and visual steps and adds the two fields that carry the point of this change:The rule the data file enforces: no figure is asserted without a
sourcesentry behind it. Nothing is inferred. Every claim is recorded against its source indocs/research/parks-topics-sources.md, and each page ends with a visibleFuentesblock so a reader can check a number without leaving the page.Sources used, in the priority order the research applied — Venezuelan state bodies first, then Venezuelan academia, then international agencies: MPPRIJP, FUNVISIS (via France 24), INAMEH (via Mongabay), ACFIMAN (via Noticiencias UCV), INPARQUES, OPS/OMS, NOAA/NWS, USGS, CDC/NIOSH, NASA (via Phys.org).
A separate
ParkTopicSectionrenders these instead of extendingEmergencyTopicSection, because that component is shared with the frozen risk-management routes and ADR-0002 keeps them untouched.Evidence
Before: 62 routes. The tierra menu listed the same six generic topics as the risk menu, in two different palettes.
After: 61 routes — the tierra menu has five children instead of six.
Each topic renders the variant its data declares — checked against the built HTML, not the source:
Every page carries its sources block:
Preview deploys automatically on every push to this branch and is live at https://pr-10.folletos.gaboggamer.online/.
Merge Danger
Door: Two-way.
Everything here is additive or a prop change.
parkTopics,ParkTopicSectionand the research doc are new files;ParksMenugained an optionaltopicSetprop that defaults to the existing behaviour, so omitting it changes nothing. Reverting restores the previous palette split without data loss.The one asymmetry: the
verde-contrasteslug is retained although the green palette is gone, so existing links keep resolving. Renaming it later would be the one-way step, and ADR-0002 governs that.Blast Radius: Contained.
EmergencyTopicSection,emergencyTopicsand/prevencion-y-gestion-de-riesgoare not modified, so the shared blue topic pages are unaffected.docs/routes.mdupdated to match.Worth a reviewer's attention
Follow-ups, not in scope
verde-contrastemenu page and its six children.astro checkreports 33 pre-existing type errors, 22 of them lucideclass/className. The CI typecheck step is non-blocking until that is cleared.🤖 Generated with Claude Code
ParksMenu.astro holds the whole menu and takes a single `theme` prop ("tierra" or "bosque") that picks the palette class. Both style options render this one component, so the only difference between them is colour — which is the point of the comparison. Two deliberate departures from the menu it is modelled on: - Mobile-first, per the design guide's new layout rule 1. The risk cards are one markup tree that reflows (row on the phone, card on `md:`) instead of the existing pages' `hidden md:grid` plus `grid md:hidden` pair, so there is one set of cards to edit rather than two. The emergency number is the first element in the DOM and a 48px tap target. - Stroke icons from lucide-react instead of emoji, matching the comps the palettes were chosen from. The risk cards still link to the existing /prevencion-y-gestion-de-riesgo topic pages; this change adds no child routes. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>Preview deployment
This PR is now served from the VPS, so the two palette options can be compared in a browser instead of in a mockup:
Built from
feat/parks-menu-style-options, served by the existing Caddy instance through itspreviews/*.confdrop-in directory. This upload was done by hand; the Forgejo Actions workflow that rebuilds on every push and tears the preview down on close is still being wired up.Known gap in this preview: the style switch on the 24
/[slug]-[style]-[mode]routes does not appear here. It is gated onprocess.env.VERCEL_ENV === 'preview'(BrochurePage.astro:33), which is never set outside Vercel. Making that gate host-neutral is part of the CI work, not this PR.🤖 Generated with Claude Code
Adds the two park routes to `menuOptions` in index.astro as "Menú 03" and "Menú 04", so both palettes are reachable from the home page instead of only by typing the URL. The grid is already `sm:grid-cols-2`, so four cards fall into two rows with no layout change. Both entries are labelled as options under evaluation, and a comment records that the losing one is deleted along with its page once the palette is chosen. This does not touch the ADR-0002 freeze: that ADR makes the *URL* of `/` immutable ("never be renamed, moved, or deleted"), not its contents. Also updates docs/routes.md, which called both routes orphaned — they are now linked from `/`, and the reachability paragraph said otherwise. Images are placeholders drawn from existing assets (a tree in the rain, a forest fire); the repo has no real park photography, so the press team will need to supply it before this ships. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>feat(parks): add the parks menu as two palette optionsto feat(parks): give the parks menu five researched, source-cited topicsView command line instructions
Checkout
From your project repository, check out a new branch and test the changes.Merge
Merge the changes and update on Forgejo.Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.