Skip to content

Fix: Table title when multiple contents#1119

Merged
MikaelNordberg merged 32 commits intomainfrom
fix/PXWEB2-487-table-multiple-contents-caption-fix
Feb 12, 2026
Merged

Fix: Table title when multiple contents#1119
MikaelNordberg merged 32 commits intomainfrom
fix/PXWEB2-487-table-multiple-contents-caption-fix

Conversation

@MikaelNordberg
Copy link
Copy Markdown
Contributor

@MikaelNordberg MikaelNordberg commented Jan 28, 2026

Updated logic for generating the table title displayed on screen.
Now possible to configure if table title shall use dynamic texts for contents or content text at table level.
Content variable label is omitted in the enumeration of variable labels.

Other things also included in this PR:

  • Removed norwegian from configuration file
  • Proper types in the jsonstat2 mapper

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jan 28, 2026

Branch Deployment to Cloudflare Pages

🚛 Preview URL: https://27711294.pxweb2.pages.dev
🚧 Branch preview URL: https://fix-pxweb2-487-table-multipl.pxweb2.pages.dev
🚚 Alternative Preview URL: https://349d8730.pxweb2.pages.dev
👷 Alternative Branch preview URL: https://alternative-fix-pxweb2-487-t.pxweb2.pages.dev

Comment thread packages/pxweb2/public/config/config.js Outdated
@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown
Contributor

@SjurSutterudSagen SjurSutterudSagen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good job, nice work on the title. And great that we finally get the types fixed in that mapper

@MikaelNordberg MikaelNordberg merged commit af16466 into main Feb 12, 2026
15 of 17 checks passed
@MikaelNordberg MikaelNordberg deleted the fix/PXWEB2-487-table-multiple-contents-caption-fix branch February 12, 2026 12:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants