Skip to content

fix(w3c/headers): translate some header text to Dutch - #5326

Merged
marcoscaceres merged 3 commits into
speced:mainfrom
Regionaal-Archief-Rivierenland:headerl10n
Aug 12, 2026
Merged

fix(w3c/headers): translate some header text to Dutch#5326
marcoscaceres merged 3 commits into
speced:mainfrom
Regionaal-Archief-Rivierenland:headerl10n

Conversation

@rien333

@rien333 rien333 commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

Modeled after the German and English text.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds missing Dutch (nl) localization strings for W3C header metadata labels, modeled after the existing English and German variants.

Changes:

  • Added several missing Dutch header labels (e.g., archives, commit history, feedback, history, implementation report, recommendation).
  • Added Dutch “more details” summary text and a Dutch multiple_alternates(plural) formatter.
  • Added Dutch publication history label.
Comments suppressed due to low confidence (1)

src/w3c/templates/headers.js:127

  • multiple_alternates(plural) uses the plural flag backwards (plural true currently selects the singular "dit … formaat" and vice versa), so Dutch output will be grammatically incorrect.
    multiple_alternates(plural) {
      return `Dit document is ook beschikbaar in ${
        plural
          ? "dit niet-normatieve formaat"
          : "deze niet-normatieve formaten"
      }:`;

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/w3c/templates/headers.js Outdated
@rien333 rien333 changed the title l10n: translate some header text to Dutch fix(w3c/headers): translate some header text to Dutch Jul 31, 2026
@marcoscaceres
marcoscaceres enabled auto-merge (squash) August 11, 2026 11:48
rien333 and others added 3 commits August 12, 2026 15:10
Modeled after the German and English text.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
auto-merge was automatically disabled August 12, 2026 13:10

Head branch was pushed to by a user without write access

@rien333

rien333 commented Aug 12, 2026

Copy link
Copy Markdown
Contributor Author

Think I pressed the wrong button here, my bad.

@marcoscaceres
marcoscaceres enabled auto-merge (squash) August 12, 2026 14:51
@marcoscaceres
marcoscaceres disabled auto-merge August 12, 2026 21:23
@marcoscaceres
marcoscaceres merged commit 0b746cd into speced:main Aug 12, 2026
8 checks passed
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.

3 participants