From d509ba82a4e7159f933333b5fea35d35e066fbc3 Mon Sep 17 00:00:00 2001
From: Ricardo Dahis <rdahis@gmail.com>
Date: Fri, 4 Sep 2026 12:04:53 +1000
Subject: [PATCH] feat(i18n): show spatial coverage in pt

The spatial coverage field was gated behind `locale !== 'pt'`, so it
rendered only on the en and es pages. The backend has always exposed
`spatialCoverageNamePt` on both DatasetNode and TableNode, and the
`/search/` endpoint returns pt names in both results and aggregations,
so the gates were hiding data that was already there.

- dataset page: drop the pt gate on the spatial coverage grid item
- search result card: drop the pt gate on the metadata row, and fix
  the card height at 165px (it was shortened to 138px only for pt)
- search sidebar: drop the pt gate on the spatial coverage filter
- table page: re-enable the block, which was commented out for every
  locale, and add `spatialCoverageName${locale}` to the table query,
  which never fetched it

pt/en/es translations for `spatialCoverage`, `table.spatialCoverage`
and `table.notProvided` already existed in all three locale files.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
---
 .../components/organisms/DatasetSearchCard.js | 10 ++--
 next/components/organisms/TablePage.js        | 48 +++++++++----------
 next/pages/api/tables/getTable.js             |  1 +
 next/pages/dataset/[dataset].js               | 38 +++++++--------
 next/pages/search.js                          | 28 +++++------
 5 files changed, 57 insertions(+), 68 deletions(-)

diff --git a/next/components/organisms/DatasetSearchCard.js b/next/components/organisms/DatasetSearchCard.js
index 754b423b..847f3d75 100644
--- a/next/components/organisms/DatasetSearchCard.js
+++ b/next/components/organisms/DatasetSearchCard.js
@@ -183,7 +183,7 @@ const DatasetSearchCard= ({
             borderRadius="16px"
             overflow="hidden"
             width="222px"
-            height={locale !== 'pt' ? "165px" : "138px"}
+            height="165px"
             position="relative"
             _hover={{ opacity: 0.9 }}
           >
@@ -258,11 +258,9 @@ const DatasetSearchCard= ({
                 {temporalCoverageText || t('notProvided')}
               </MetadataRow>
 
-              {locale !== 'pt' && (
-                <MetadataRow label={t('spatialCoverage')}>
-                  {spatialCoverage || t('notProvided')}
-                </MetadataRow>
-              )}
+              <MetadataRow label={t('spatialCoverage')}>
+                {spatialCoverage || t('notProvided')}
+              </MetadataRow>
 
               <MetadataRow label={t('resources')}>
                 {resourcesText}
diff --git a/next/components/organisms/TablePage.js b/next/components/organisms/TablePage.js
index 71556a4a..3de02599 100644
--- a/next/components/organisms/TablePage.js
+++ b/next/components/organisms/TablePage.js
@@ -577,34 +577,30 @@ export default function TablePage({ id, isBDSudo, changeTab, datasetName }) {
         </StackSkeleton>
       </Stack>
 
-      {/* {locale !== 'pt' ?
-        <Stack spacing="8px"  marginBottom="40px !important">
-          <StackSkeleton width="300px" height="28px" isLoading={isLoading}>
-            <TitleText typography="small">
-              {t('table.spatialCoverage')}
-            </TitleText>
-          </StackSkeleton>
+      <Stack spacing="8px"  marginBottom="40px !important">
+        <StackSkeleton width="300px" height="28px" isLoading={isLoading}>
+          <TitleText typography="small">
+            {t('table.spatialCoverage')}
+          </TitleText>
+        </StackSkeleton>
 
-          <StackSkeleton
-            height="20px"
-            width={resource?.[`spatialCoverageName${capitalize(locale)}`] ? "100%" : "200px"}
-            isLoading={isLoading}
+        <StackSkeleton
+          height="20px"
+          width={resource?.[`spatialCoverageName${capitalize(locale)}`] ? "100%" : "200px"}
+          isLoading={isLoading}
+        >
+          <BodyText
+            typography="small"
+            color="#464A51"
           >
-            <BodyText
-              typography="small"
-              color="#464A51"
-            >
-              {resource?.[`spatialCoverageName${capitalize(locale)}`]
-                ? Object.values(resource[`spatialCoverageName${capitalize(locale)}`])
-                    .sort((a, b) => a.localeCompare(b, locale))
-                    .join(', ')
-                : t('table.notProvided')}
-            </BodyText>
-          </StackSkeleton>
-        </Stack>
-        :
-        <></>
-      } */}
+            {resource?.[`spatialCoverageName${capitalize(locale)}`]
+              ? Object.values(resource[`spatialCoverageName${capitalize(locale)}`])
+                  .sort((a, b) => a.localeCompare(b, locale))
+                  .join(', ')
+              : t('table.notProvided')}
+          </BodyText>
+        </StackSkeleton>
+      </Stack>
 
       <Stack
         spacing="8px"
diff --git a/next/pages/api/tables/getTable.js b/next/pages/api/tables/getTable.js
index 8678a17e..7a5d9218 100644
--- a/next/pages/api/tables/getTable.js
+++ b/next/pages/api/tables/getTable.js
@@ -54,6 +54,7 @@ async function getTable(id, locale='pt') {
                 isDeprecated
                 temporalCoverage
                 fullTemporalCoverage
+                spatialCoverageName${capitalize(locale)}
                 rawDataSource {
                   edges {
                     node {
diff --git a/next/pages/dataset/[dataset].js b/next/pages/dataset/[dataset].js
index 32ce98ec..5c6bdc3a 100644
--- a/next/pages/dataset/[dataset].js
+++ b/next/pages/dataset/[dataset].js
@@ -483,26 +483,24 @@ export default function DatasetPage ({ dataset, userGuide, hiddenDataset }) {
                 </BodyText>
               </GridItem>
 
-              {locale !== 'pt' &&
-                <GridItem colSpan={{ base: 5, lg: 4 }} marginBottom="8px">
-                  <LabelText
-                    typography="large"
-                    marginBottom="8px"
-                  >
-                    {t('spatialCoverage')}
-                  </LabelText>
-                  <BodyText
-                    typography="small"
-                    color="#464A51"
-                  >
-                    {dataset?.[`spatialCoverageName${capitalize(locale)}`]
-                      ? Object.values(dataset[`spatialCoverageName${capitalize(locale)}`])
-                          .sort((a, b) => a.localeCompare(b, locale))
-                          .join(', ')
-                      : t('notProvided')}
-                  </BodyText>
-                </GridItem>
-              }
+              <GridItem colSpan={{ base: 5, lg: 4 }} marginBottom="8px">
+                <LabelText
+                  typography="large"
+                  marginBottom="8px"
+                >
+                  {t('spatialCoverage')}
+                </LabelText>
+                <BodyText
+                  typography="small"
+                  color="#464A51"
+                >
+                  {dataset?.[`spatialCoverageName${capitalize(locale)}`]
+                    ? Object.values(dataset[`spatialCoverageName${capitalize(locale)}`])
+                        .sort((a, b) => a.localeCompare(b, locale))
+                        .join(', ')
+                    : t('notProvided')}
+                </BodyText>
+              </GridItem>
             </Grid>
           </GridItem>
         </Grid>
diff --git a/next/pages/search.js b/next/pages/search.js
index 54c9d543..a701ca30 100644
--- a/next/pages/search.js
+++ b/next/pages/search.js
@@ -704,23 +704,19 @@ export default function SearchDatasetPage() {
 
           <Divider marginY="16px !important" borderColor="#DEDFE0"/>
 
-          {locale !== 'pt' && (
-            <>
-              <CheckboxFilterAccordion
-                isActive={validateActiveFilterAccordin("spatial_coverage")}
-                choices={memoizedFilters.spatialCoverages}
-                valueField="key"
-                displayField="name"
-                fieldName={t('spatialCoverage')}
-                valuesChecked={valuesCheckedFilter("spatial_coverage")}
-                onChange={(value) => handleSelectFilter(["spatial_coverage",`${value}`])}
-                isLoading={!isLoading}
-                facet="spatial_coverage"
-              />
+          <CheckboxFilterAccordion
+            isActive={validateActiveFilterAccordin("spatial_coverage")}
+            choices={memoizedFilters.spatialCoverages}
+            valueField="key"
+            displayField="name"
+            fieldName={t('spatialCoverage')}
+            valuesChecked={valuesCheckedFilter("spatial_coverage")}
+            onChange={(value) => handleSelectFilter(["spatial_coverage",`${value}`])}
+            isLoading={!isLoading}
+            facet="spatial_coverage"
+          />
 
-              <Divider marginY="16px !important" borderColor="#DEDFE0"/>
-            </>
-          )}
+          <Divider marginY="16px !important" borderColor="#DEDFE0"/>
 
           <CheckboxFilterAccordion
             isActive={validateActiveFilterAccordin("organization")}