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

Docs forms section fa translation - first draft #33

Merged
merged 21 commits into from
Mar 12, 2019

Conversation

mjraadi
Copy link
Contributor

@mjraadi mjraadi commented Mar 7, 2019

Please take a look at my translation and let me know if it needs any changes.

@sorousht sorousht self-assigned this Mar 7, 2019
Copy link
Collaborator

@sorousht sorousht left a comment

Choose a reason for hiding this comment

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

سلام @mjraadi عزیز

شما اولین مشارکت کننده‌ای هستید که نتیجه کارش رو ارسال کرد! خوشحالم و تشکر می‌کنم.

چند مورد کلی هست که پیشنهاد می‌کنم برطرف بشه، پیش از اینکه به موارد جزئی بپردازیم:

۱. رعایت نیم‌فاصله: شاید برای ما هنوز عادت نشده باشه، اما به کیفیت کارمون خیلی کمک می‌کنه.
۲. تا اینجای کار، تصمیم تیم بر این شد که برای ترجمه واژه "element" از کلمه "المنت" استفاده کنیم. چون بهتر مفهوم رو منتقل می‌کنه (البته هنوز به واژه‌نامه ما اضافه نشده‌!)
۳. تا حد امکان چیزی از متن کم و به اون اضافه نشه

اینجا درباره نیم‌فاصله نکات جالبی هست.

تشکر

@mjraadi
Copy link
Contributor Author

mjraadi commented Mar 7, 2019

سلام @sorousht عزیز:

امروز تایم آزاد داشتم نشستم پاش و خوشحالم که میتونم مشارکتی توی یادگیری این کتابخونه داشته باشم.

مورد ۱ و ۲ رو بر طرف می کنم.

واسه مورد ۳ من سعی کردم چیزی رو حذف نکنم اما واسه اینکه ترجمه کاملا تحت اللفظی نباشه و مفهوم رو بهتر برسونه یه چند کلمه ای بعضی جاها واسه بیان بهتر اضافه کردم. نظرتون چی هست؟

سپاس

@sorousht
Copy link
Collaborator

sorousht commented Mar 7, 2019

@mjraadi جان،

ممنون، برای مورد سوم، مثلا عنوان صفحه "فرم‌ها" باشه درست‌تر هست. یا هرجا که برای فهم بهتر صلاح می‌دونید کلمه‌ای اضافه بشه، اون رو توی براکت [*] قرار می‌دهیم که خواننده متوجه بشه مترجم اون کلمه رو اضافه کرده.

اگر موردی بود، در خدمتم :)

@sorousht sorousht self-requested a review March 9, 2019 18:17
Copy link
Collaborator

@sorousht sorousht left a comment

Choose a reason for hiding this comment

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

@mjraadi عزیز، مواردی که نوشتم نظرات شخصی است و اگر فکر می کنید که هرکدوم از اون‌ها مناسب نیست، انتخاب با شماست و می‌تونیم دربارش صحبت کنیم.

content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
sJJdGG and others added 11 commits March 10, 2019 16:34
Co-Authored-By: mjraadi <raadi@bitsnbytes.ir>
Co-Authored-By: mjraadi <raadi@bitsnbytes.ir>
Co-Authored-By: mjraadi <raadi@bitsnbytes.ir>
Co-Authored-By: mjraadi <raadi@bitsnbytes.ir>
Co-Authored-By: mjraadi <raadi@bitsnbytes.ir>
Co-Authored-By: mjraadi <raadi@bitsnbytes.ir>
Co-Authored-By: mjraadi <raadi@bitsnbytes.ir>
Co-Authored-By: mjraadi <raadi@bitsnbytes.ir>
Co-Authored-By: mjraadi <raadi@bitsnbytes.ir>
@mjraadi
Copy link
Contributor Author

mjraadi commented Mar 10, 2019

Dear @sorousht and @sJJdGG see if your requested changes have been correctly addressed.
Thanks.

@sJJdGG sJJdGG self-assigned this Mar 10, 2019
@sJJdGG sJJdGG self-requested a review March 12, 2019 09:51
Copy link
Member

@sJJdGG sJJdGG left a comment

Choose a reason for hiding this comment

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

LGTM

@sJJdGG sJJdGG merged commit 696d138 into reactjs:fa Mar 12, 2019
@mjraadi mjraadi deleted the docs-forms-fa-translation branch March 12, 2019 10:20
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.

8 participants