Skip to content

fix(l10n): drop the printer vendor name from UI strings - #12

Merged
glenntosco merged 1 commit into
mainfrom
claude/project-thread-4ng9io
Oct 24, 2026
Merged

glenntosco merged 1 commit into
mainfrom
claude/project-thread-4ng9io

Conversation

@glenntosco

Copy link
Copy Markdown
Collaborator

Requested by Glenn · project thread

What changed and why

Three English and Spanish strings still named one printer brand:

Key New English New Spanish
Print.Error.ConnectionRefused "...Check the port (most label printers use 9100)..." "...(la mayoría de las impresoras de etiquetas usan el 9100)..."
Print.Field.Port "...(most label printers use 9100)." "...(la mayoría de las impresoras de etiquetas usan el 9100)."
Start.Subtitle "...print it on your ZPL-compatible printer." "...imprímala en su impresora compatible con ZPL."

pt-BR and fr-CA already used this neutral wording, so all four languages now say the same thing. Two XML doc comments in AppSettings and PrinterSettingsViewModel got the same treatment. Test names that cite the ZPL programming guide's sample symbols are left alone, since they reference the spec.

How it was tested

Text-only change; no keys added or removed, so resx parity tests are unaffected. CI build and tests.

Checklist

  • dotnet build Rotula.sln -c Release and dotnet test Rotula.sln -c Release pass (CI)
  • New behavior has tests; Rotula.Core stays at or above 80% line coverage (no code change)
  • Public members have XML docs
  • New user-visible text exists in all four languages
  • No network calls or telemetry added; light theme only
  • New dependencies: none
  • CHANGELOG.md: not needed for a wording fix
  • bash scripts/check-hygiene.sh origin/main..HEAD passes

Generated by Claude Code

…strings

The English and Spanish connection-refused, port-validation and start-page
subtitle strings named one printer brand. They now say "most label printers"
and "ZPL-compatible printer", matching the pt-BR and fr-CA wording.
@glenntosco glenntosco self-assigned this Oct 24, 2026
@glenntosco
glenntosco merged commit 853f87d into main Oct 24, 2026
6 checks passed
glenntosco added a commit that referenced this pull request Oct 27, 2026
fix(l10n): drop the printer vendor name from UI strings
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