Skip to content

fix: restore "show-more" translation key removed by #17662#17774

Merged
pettinarip merged 1 commit into
stagingfrom
fix/restore-show-more-key
Mar 13, 2026
Merged

fix: restore "show-more" translation key removed by #17662#17774
pettinarip merged 1 commit into
stagingfrom
fix/restore-show-more-key

Conversation

@pettinarip
Copy link
Copy Markdown
Member

Summary

  • Restores the "show-more" translation key to common.json across all 25 locales — it was incorrectly removed by perf: remove unused keys from common.json namespace #17662
  • Replaces dynamic key construction in TruncatedText.tsx with explicit key references so static analysis tools can detect usage

What happened

PR #17662 used automated scanning to find unused keys in common.json. It missed that TruncatedText.tsx uses "show-more" because the key was constructed dynamically:

// Before (invisible to static analysis)
t(`show-${isExpanded ? "less" : "more"}`)

// After (detectable by static analysis)
isExpanded ? t("show-less") : t("show-more")

Test plan

  • Verify TruncatedText "Show more" / "Show less" toggle works correctly
  • Spot-check a few non-English locales to confirm translations render

PR #17662 incorrectly identified "show-more" as unused because
TruncatedText.tsx constructed the key dynamically via template literal.
This restores the key across all 25 locales and switches to explicit
key references so static analysis tools can detect usage.
@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 13, 2026

Deploy Preview for ethereumorg ready!

Name Link
🔨 Latest commit 4cbd221
🔍 Latest deploy log https://app.netlify.com/projects/ethereumorg/deploys/69b3f354eb094e0008313089
😎 Deploy Preview https://deploy-preview-17774.ethereum.it
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
7 paths audited
Performance: 58 (🟢 up 5 from production)
Accessibility: 94 (🟢 up 1 from production)
Best Practices: 100 (no change from production)
SEO: 99 (no change from production)
PWA: 59 (no change from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions github-actions Bot added content 🖋️ This involves copy additions or edits translation 🌍 This is related to our Translation Program labels Mar 13, 2026
@pettinarip pettinarip merged commit 90b8f4b into staging Mar 13, 2026
10 of 11 checks passed
@pettinarip pettinarip deleted the fix/restore-show-more-key branch March 13, 2026 11:48
@pettinarip pettinarip mentioned this pull request Mar 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

content 🖋️ This involves copy additions or edits translation 🌍 This is related to our Translation Program

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant