Repository navigation
docs: add Protected Areas 2021/2024 nomenclature and mapping guideline - #79
Merged
Merged
Conversation
Add the Priority Area Monitoring - Protected Areas 2021 and 2024 Nomenclature and Mapping Guideline, converted from the source PDF. Includes the full LC/LU class nomenclature and its 377 figures. `validate_qmd_files.py`: 38/38 valid; all media references resolve.
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 1e0c785f80
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| category: products | ||
| author: European Environment Agency (EEA) | ||
| --- | ||
| I. Introduction |
There was a problem hiding this comment.
I. Introduction and the subsequent Roman-numeral section labels are plain paragraphs, and the file contains no level-1 or level-2 headings at all before its level-3 class headings. With the repository's numbered, depth-3 TOC, the main sections are therefore absent from navigation and every class heading is orphaned at level 3; mark the main sections and class groups with the appropriate #/## hierarchy.
Useful? React with 👍 / 👎.
Comment on lines
+1687
to
+1690
|  | ||
|
|
||
| Appearance: | ||
|  |
There was a problem hiding this comment.
This coniferous-plantation figure and the following appearance text are placed inside Highly artificial broadleaved plantations, then repeated verbatim under the correct Highly artificial coniferous plantations section at lines 1754–1764. As rendered, readers get irrelevant conifer examples in the broadleaved class and see the same material twice; remove this earlier duplicate.
Useful? React with 👍 / 👎.
Comment on lines
+3841
to
+3845
|  | ||
|
|
||
|  | ||
|
|
||
|  |
There was a problem hiding this comment.
These Eucalyptus figures, followed by two dry-grassland figures, are appended at end of file without a new heading, so the renderer presents them as part of Coastal waters. Their captions instead match the Highly artificial broadleaved plantations and Semi-natural grassland sections around lines 1662 and 2189; relocate them there so the published guideline does not associate unrelated land-cover examples with coastal waters.
Useful? React with 👍 / 👎.
- Restore the LC/LU nomenclature legend colours in Table 2 by sampling each code cell's fill from the source PDF (pages 10-11) and injecting it as an inline background-color across the three code-level columns (88 cells). - Close the tbl-caption divs properly (blank line before the closing fence) so the caption markers no longer leak into the rendered table text.
mckeea
self-requested a review
September 20, 2026 11:35
mckeea
approved these changes
Sep 20, 2026
mckeea
added a commit
that referenced
this pull request
Oct 2, 2026
* Update LLM cache and version history [skip ci] * ci: prepare workflows for branch protection rulesets * ci: pin ruff rule set so upstream default changes don't break CI * docs: correct required YAML fields and document type in editor guide * chore: drop editor guide cache entry to force regeneration * chore: drop one image cache entry to force a cache write * Update LLM cache and version history [skip ci] * docs: unify CLMS filenaming doc names and title pattern Rename both docs to the shared clms-filenaming_<topic>_v1.qmd pattern, align both titles to 'CLMS Product Filenaming - <Topic>', and replace em dashes with plain hyphens throughout. * Update LLM cache and version history [skip ci] * feat: add parsEO dashboard (filename builder + schema tools) * fix: use relative API URL, add fetch error handling, debounce, XSS escaping * fix: empty API base for same-origin (SWAG proxy) * fix: remove API URL field, auto-connect on same-origin * feat: clickable chip-tree for filename assembly * feat: add parsEO dashboard to DOCS/tools (Quarto site) * feat: add parsEO dashboard to DOCS/tools with index.qmd landing page * feat: parsEO dashboard as OJS Quarto dashboard (replaces standalone HTML) * corrected type SFW * fix: remove bar labels and default to weekly view in CDSE migration dashboard Remove the +N number labels from the onboarding history bar chart (values are still available via tooltip on hover). Switch the default aggregation mode from daily to weekly for a cleaner overview. * fix: plot negative onboarding deltas below zero and key weeks by ISO week-year * fix: make the parsEO dashboard actually run * Update LLM cache and version history [skip ci] * Update non-browsable doc map [skip ci] * docs: use NET-LINE for EU Hydro examples and drop it from the tree (#78) * docs: correct product descriptions in CLMS filenaming guidelines (#77) Apply Lorenzo's feedback: - Attribute IW burst to Sentinel-1 swath, not EGMS InSAR - Rephrase EGMS L3 grid and L2 orbit descriptions with their official labels * chore(ci): merge DOCS/clcplus-core from publish-clcplus-core-20260910-094246 Triggered by: CI Workflow Branch: publish-clcplus-core-20260910-094246 Commit: f658ece Repo: f658ece * Update LLM cache and version history [skip ci] * chore(ci): merge DOCS/clcplus-core from publish-clcplus-core-20260910-095249 Triggered by: CI Workflow Branch: publish-clcplus-core-20260910-095249 Commit: 382cc06 Repo: 382cc06 * chore(ci): merge DOCS/clcplus-core from publish-clcplus-core-20260910-102206 Triggered by: CI Workflow Branch: publish-clcplus-core-20260910-102206 Commit: cd1b8ca Repo: cd1b8ca * Update LLM cache and version history [skip ci] * fix(build): realign drifted grid tables and guard dangling links in Typst PDFs * chore(ci): merge DOCS/clcplus-core from publish-clcplus-core-20260914-100407 Triggered by: CI Workflow Branch: publish-clcplus-core-20260914-100407 Commit: 1215deb Repo: 1215deb * fix(docs): unblock Typst build for CLC+ Core User Manual [skip ci] * fix(docs): realign drifted grid tables in CLC+ Core User Manual [skip ci] * fix(preview): apply dangling-link guard to local Typst renders [skip ci] * feat(build): add BUILD_ONLY filter to build-docs.sh [skip ci] * docs(agents): require suggestions before implementation [skip ci] * docs: add Protected Areas 2021/2024 nomenclature and mapping guideline (#79) * docs: add Protected Areas 2021/2024 nomenclature and mapping guideline Add the Priority Area Monitoring - Protected Areas 2021 and 2024 Nomenclature and Mapping Guideline, converted from the source PDF. Includes the full LC/LU class nomenclature and its 377 figures. `validate_qmd_files.py`: 38/38 valid; all media references resolve. * docs: restore Table 2 colors and fix fenced-div caption closures - Restore the LC/LU nomenclature legend colours in Table 2 by sampling each code cell's fill from the source PDF (pages 10-11) and injecting it as an inline background-color across the three code-level columns (88 cells). - Close the tbl-caption divs properly (blank line before the closing fence) so the caption markers no longer leak into the rendered table text. * Update LLM cache and version history [skip ci] * feat(build): compress media at build time, keeping originals in git * fix(build): compress only document media and repoint references safely * fix(build): detect transparency portably and flatten onto white * fix(build): use %[opaque] for transparency, portable across ImageMagick 6 and 7 * chore: sync published version state from main * chore: hold back unpublished docs (products batch, CDSE and parsEO dashboards) * chore: remove orphan media and cache of withdrawn HRL SLF PUM draft * chore: take MRVPP cache entries from main --------- Co-authored-by: GitHub Actions <actions@github.com> Co-authored-by: Matteo Mattiuzzi <matteo@mattiuzzi.com> Co-authored-by: ci_docker_builder <ci_docker_builder@users.noreply.github.com>
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
2 participants
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.
Add the PA21/PA24 Protected Areas Nomenclature and Mapping Guideline to a new
Protected_Areasproduct folder.DOCS/Protected_Areas(will also host the future PUM and ATBD)category: products, date 2026-09-18validate_qmd_files.py38/38 validNote: document converted from the source PDF by a Gemini pipeline. The conversion's own report flags known content items worth a look before final publication: