Skip to content

i18n: two es/tl wording items raised in #720 review, for the native-speaker pass #738

Description

@mforce

Two translation-quality items raised by automated reviewers during #720 (PR #734). Both were deliberately NOT changed — see the reasoning — and are filed here for the Phase 1.5 native-speaker es/tl review.

1. tl — possible malformed verb in glossaryDiscountDef

A reviewer claims web/src/i18n/tl.ts glossaryDiscountDef contains ilinalagay where the correct form is inilalagay (i- + -in- infix + reduplication of lagay).

Marked SUSPECTED by the reviewer, which is honest: it is not a native speaker, and neither is the driver. AGENTS.md #688 records that mechanical Tagalog affixation checks were built for exactly this class and rejected on evidence — Tagalog affixation defeats substring and stem matching. This is also the #662 class: a lower-case i18n typo that shipped in tl because catalogParity compares key sets only.

Needs a native speaker to settle. Do not fix on a tool's say-so.

2. es — sobre vs por encima de for "above list"

CodeRabbit initially raised this as an internal inconsistency. It was refuted with evidence and the reviewer withdrew it: all five keys agree with each other — aboveList, glossaryAboveListTerm, listPriceHintAbove, listPriceHintAboveNoPct, and the help prose all use "Sobre el precio de lista". #688 is about a locale disagreeing with itself, which does not apply.

What remains is a genuine quality question: sobre el precio de lista can read as "about/on the list price" as well as "above" it, where por encima de is unambiguous. A judgement for a native speaker, not a defect.

Why neither was changed in #720

Both locales are machine-drafted (#182). Substituting one non-native guess for another is not an improvement, and #720 had no native speaker available. Recording the specific claims here means the native-speaker pass gets concrete strings to check rather than a general instruction to re-read three catalogs.

Strings to check

Locale Key Current
tl glossaryDiscountDef contains ilinalagay
es aboveList, glossaryAboveListTerm "Sobre el precio de lista"
es listPriceHintAbove, listPriceHintAboveNoPct "… sobre el precio de lista"

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    documentationImprovements or additions to documentationepic-1.5severity:p4Defect: tooling, CI noise or test gap; no user impact

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions