Skip to content

[SITES-40889] [Core Components] Improve VCF placeholder copy and route literals through i18n()#3035

Merged
alexandru-stancioiu merged 1 commit into
mainfrom
SITES-40889-vcf-placeholder-i18n
May 15, 2026
Merged

[SITES-40889] [Core Components] Improve VCF placeholder copy and route literals through i18n()#3035
alexandru-stancioiu merged 1 commit into
mainfrom
SITES-40889-vcf-placeholder-i18n

Conversation

@alexandru-stancioiu
Copy link
Copy Markdown
Contributor

@alexandru-stancioiu alexandru-stancioiu commented May 15, 2026

  • Author "no preview" placeholder: reword title from "Visual Content Fragment preview unavailable" to "Visual Content Fragment unavailable" and drop the editor-specific detail suffix.
  • Author "load failed" detail no longer references "preview services" (always available on publish) or "fragment configuration" (opaque to authors); align with the publish-side wording "The Visual Content Fragment could not be displayed."
  • Move the literal strings inside i18n("...") calls at the placeholder-builder site so the literals are visible to translation tooling, rather than being passed positionally and translated inside the HTML builder.
  • Update vcfRendererTest.js assertion to match the new title.
Q                       A
Fixed Issues? follow up for https://jira.corp.adobe.com/browse/SITES-40889
Patch: Bug Fix?
Minor: New Feature?
Major: Breaking Change?
Tests Added + Pass? Yes
Documentation Provided Yes (code comments and or markdown)
Any Dependency Changes?
License Apache License, Version 2.0

…e literals through i18n()

- Author "no preview" placeholder: reword title from "Visual Content Fragment preview unavailable" to
  "Visual Content Fragment unavailable" and drop the editor-specific detail suffix.
- Author "load failed" detail no longer references "preview services" (always available on publish)
  or "fragment configuration" (opaque to authors); align with the publish-side wording
  "The Visual Content Fragment could not be displayed."
- Move the literal strings inside i18n("...") calls at the placeholder-builder site so the
  literals are visible to translation tooling, rather than being passed positionally and
  translated inside the HTML builder.
- Update vcfRendererTest.js assertion to match the new title.

Co-authored-by: Cursor <cursoragent@cursor.com>
@sonarqubecloud
Copy link
Copy Markdown

@codecov
Copy link
Copy Markdown

codecov Bot commented May 15, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@YahorC YahorC added this to the 2.31.0 milestone May 15, 2026
@alexandru-stancioiu alexandru-stancioiu merged commit 58a232c into main May 15, 2026
13 checks passed
@alexandru-stancioiu alexandru-stancioiu deleted the SITES-40889-vcf-placeholder-i18n branch May 15, 2026 10:09
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