Skip to content

chore(storybook): replace fa locale with ar for RTL testing#17170

Merged
wackerow merged 1 commit into
devfrom
chore/replace-fa-locale-with-ar-in-storybook
Jan 26, 2026
Merged

chore(storybook): replace fa locale with ar for RTL testing#17170
wackerow merged 1 commit into
devfrom
chore/replace-fa-locale-with-ar-in-storybook

Conversation

@pettinarip
Copy link
Copy Markdown
Member

@pettinarip pettinarip commented Jan 26, 2026

Summary

  • Replace the removed fa (Farsi) locale with ar (Arabic) in Storybook configuration
  • Arabic is still actively supported and maintains RTL layout testing capabilities

Test plan

  • Verify Storybook builds successfully
  • Confirm RTL stories render correctly with Arabic locale

The fa (Farsi) locale has been removed from the project. Replace it with
ar (Arabic) which is still actively supported and also provides RTL
layout testing capabilities.
@netlify
Copy link
Copy Markdown

netlify Bot commented Jan 26, 2026

Deploy Preview for ethereumorg ready!

Name Link
🔨 Latest commit d7304a5
🔍 Latest deploy log https://app.netlify.com/projects/ethereumorg/deploys/69775e3a757a530008a1e897
😎 Deploy Preview https://deploy-preview-17170.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: 55 (🟢 up 2 from production)
Accessibility: 94 (no change from production)
Best Practices: 100 (no change from production)
SEO: 100 (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 the tooling 🔧 Changes related to tooling of the project label Jan 26, 2026
Copy link
Copy Markdown
Member

@wackerow wackerow left a comment

Choose a reason for hiding this comment

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

Lgtm.. rendering RTL appropriately, though we're just missing a bunch of strings for the stories in the ar locale.. this should be remedied with #17105

@wackerow wackerow merged commit 07a5e56 into dev Jan 26, 2026
7 checks passed
@wackerow wackerow deleted the chore/replace-fa-locale-with-ar-in-storybook branch January 26, 2026 22:04
@pettinarip pettinarip mentioned this pull request Jan 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tooling 🔧 Changes related to tooling of the project

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants