Skip to content

Conversation

@wackerow
Copy link
Member

@wackerow wackerow commented Dec 10, 2021

Description

  • Updates the Arabic (ar) homepage to latest translations from Crowdin.
  • Updates logic about when to show the legacy homepage. This is a temporary measure and can be phased out after all languages have been updated to the newer homepage (prioritized in Crowdin)

Temporary, will be phased out after new homepage translated for all languages (prioritized)
@github-actions github-actions bot added content 🖋️ This involves copy additions or edits review needed 👀 translation 🌍 This is related to our Translation Program labels Dec 10, 2021
@gatsby-cloud
Copy link

gatsby-cloud bot commented Dec 10, 2021

Gatsby Cloud Build Report

ethereum-org-website-dev2

🎉 Your build was successful! See the Deploy preview here.

Build Details

View the build logs here.

🕐 Build time: 6m

Performance

Lighthouse report

Metric Score
Performance 🔶 19
Accessibility 💚 97
Best Practices 💚 93
SEO 🔶 79

🔗 View full report

@gatsby-cloud
Copy link

gatsby-cloud bot commented Dec 10, 2021

Gatsby Cloud Build Report

ethereum-org-website-dev

🎉 Your build was successful! See the Deploy preview here.

Build Details

View the build logs here.

🕐 Build time: 6m

Performance

Lighthouse report

Metric Score
Performance 🔶 21
Accessibility 💚 97
Best Practices 💚 93
SEO 🔶 79

🔗 View full report

@minimalsm
Copy link
Contributor

LGTM.

Could definitely do a clean-up to support rtl better. But seems like it could get messy real quick... (see nav screenshot)

Nav

Screenshot 2021-12-13 at 14 37 51

Mobile nav

This should be flipped

Screenshot 2021-12-13 at 14 37 19

Screenshot 2021-12-13 at 14 28 13

### Footer

Should be rtl

Screenshot 2021-12-13 at 14 40 49

@minimalsm minimalsm merged commit 25c17f2 into dev Dec 13, 2021
@minimalsm minimalsm deleted the ar-homepage branch December 13, 2021 14:41
@wackerow wackerow mentioned this pull request Dec 16, 2021
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.

3 participants