Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

translating HomeContent.js #514

Closed
wants to merge 1 commit into from
Closed

translating HomeContent.js #514

wants to merge 1 commit into from

Conversation

tmtm8976
Copy link
Contributor

@tmtm8976 tmtm8976 commented Aug 2, 2023

Translating HomeContent.js, also used rtl and ltr to adjust the text

@AhmedBaset
Copy link
Collaborator

Hey @tmtm8976,
Thank you for your contribution.
Currently, we are focusing on translating the docs located in src/content/learn and src/content/reference. We plan to address layouts and components once we've completed a significant portion of the documentation. Therefore, this pull request will temporarily be closed until we reach that stage.

Additionally, I'd like to emphasize that this forked repo is specifically for translating texts only. We won't refactor classes, attributes, or any code here to avoid potential conflicts when syncing with the main react.dev. For code refactoring, please submit your changes directly to the main branch.

By the way, there's already an open PR that handles all RTL styles, awaiting review. You can check it out here.

Thanks for your understanding. If you're interested in contributing to the documentation translation, please join us in tackling the issue. Your involvement is appreciated!

@AhmedBaset AhmedBaset closed this Aug 2, 2023
@tmtm8976
Copy link
Contributor Author

tmtm8976 commented Aug 3, 2023

Thank you for your valuable feedback. I would like to translate as many pages as I can, and I will also check out the RTL styles PR.

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