Skip to content

Translate page "Rendering Elements" - #28

Merged
sJJdGG merged 25 commits into
reactjs:masterfrom
sorousht:rendering-elements
Mar 7, 2019
Merged

sJJdGG merged 25 commits into
reactjs:masterfrom
sorousht:rendering-elements

Conversation

@sorousht

@sorousht sorousht commented Mar 1, 2019

Copy link
Copy Markdown
Contributor

No description provided.

@sorousht sorousht added the translation A new page has been translated label Mar 1, 2019
@sorousht
sorousht requested review from sJJdGG and zahrajoulaei March 1, 2019 14:13
@netlify

netlify Bot commented Mar 1, 2019 •

Copy link
Copy Markdown

Deploy preview for fa-reactjs ready!

Built with commit b06a454

https://deploy-preview-28--fa-reactjs.netlify.com

Comment thread content/docs/rendering-elements.md Outdated
Comment thread content/docs/rendering-elements.md Outdated
Comment thread content/docs/rendering-elements.md Outdated
Comment thread content/docs/rendering-elements.md Outdated
Comment thread content/docs/rendering-elements.md Outdated
sJJdGG and others added 5 commits March 3, 2019 14:29
Co-Authored-By: sorousht <soroush.torkzadeh@gmail.com>
Use a batter word for **Note**

Co-Authored-By: sorousht <soroush.torkzadeh@gmail.com>
Co-Authored-By: sorousht <soroush.torkzadeh@gmail.com>
Co-Authored-By: sorousht <soroush.torkzadeh@gmail.com>
Co-Authored-By: sorousht <soroush.torkzadeh@gmail.com>
@sorousht
sorousht requested a review from sJJdGG March 3, 2019 11:12
zahrajoulaei
zahrajoulaei previously approved these changes Mar 4, 2019
Comment thread content/docs/rendering-elements.md Outdated

@sJJdGG sJJdGG left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@sorousht If you find some changes unnecessary let me know

Comment thread content/docs/rendering-elements.md Outdated
Comment thread content/docs/rendering-elements.md Outdated
Comment thread content/docs/rendering-elements.md Outdated
Comment thread content/docs/rendering-elements.md Outdated
Comment thread content/docs/rendering-elements.md Outdated
Comment thread content/docs/rendering-elements.md Outdated
Comment thread content/docs/rendering-elements.md Outdated
Comment thread content/docs/rendering-elements.md Outdated
Comment thread content/docs/rendering-elements.md Outdated
Comment thread content/docs/rendering-elements.md Outdated
Comment thread content/docs/rendering-elements.md Outdated
Thanks to @sJJdGG

Co-Authored-By: sorousht <soroush.torkzadeh@gmail.com>
sJJdGG and others added 5 commits March 4, 2019 14:21
Co-Authored-By: sorousht <soroush.torkzadeh@gmail.com>
Thanks to @sJJdGG

Co-Authored-By: sorousht <soroush.torkzadeh@gmail.com>
Co-Authored-By: sorousht <soroush.torkzadeh@gmail.com>
Co-Authored-By: sorousht <soroush.torkzadeh@gmail.com>
Co-Authored-By: sorousht <soroush.torkzadeh@gmail.com>
Comment thread content/docs/rendering-elements.md Outdated
sJJdGG and others added 7 commits March 4, 2019 19:15
Co-Authored-By: sorousht <soroush.torkzadeh@gmail.com>
Co-Authored-By: sorousht <soroush.torkzadeh@gmail.com>
Co-Authored-By: sorousht <soroush.torkzadeh@gmail.com>
Co-Authored-By: sorousht <soroush.torkzadeh@gmail.com>
…ring-elements.md

Co-Authored-By: sorousht <soroush.torkzadeh@gmail.com>
Co-Authored-By: sorousht <soroush.torkzadeh@gmail.com>
Co-Authored-By: sorousht <soroush.torkzadeh@gmail.com>
@sorousht
sorousht requested a review from sJJdGG March 5, 2019 13:18

@sJJdGG sJJdGG left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Render is more of a verb that does something and is a synonym of verb "make" and it'd better be translated to "رندر کردن" instead of "رندر شدن". "being rendered" is the one that should be translated to "رندر شدن". Or at least that's what I think.

@sorousht @zahrajoulaei What do you think?

Comment thread content/docs/rendering-elements.md Outdated
Comment thread content/docs/rendering-elements.md Outdated

@sJJdGG sJJdGG left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@sorousht Some final minor enhancements I could think of. Let me know what you think.

Comment thread content/docs/rendering-elements.md Outdated
Comment thread content/docs/rendering-elements.md Outdated
sJJdGG and others added 4 commits March 6, 2019 12:04
Co-Authored-By: sorousht <soroush.torkzadeh@gmail.com>
Co-Authored-By: sorousht <soroush.torkzadeh@gmail.com>
Co-Authored-By: sorousht <soroush.torkzadeh@gmail.com>
Co-Authored-By: sorousht <soroush.torkzadeh@gmail.com>
@sorousht

sorousht commented Mar 6, 2019

Copy link
Copy Markdown
Contributor Author

Render is more of a verb that does something and is a synonym of verb "make" and it'd better be translated to "رندر کردن" instead of "رندر شدن". "being rendered" is the one that should be translated to "رندر شدن". Or at least that's what I think.

@sorousht @zahrajoulaei What do you think?

Yeah, your right.
"رندر شدن" means "being rendered" and,
"رندر کردن" means "rendering".

So I fix that.

@sorousht
sorousht requested a review from sJJdGG March 6, 2019 08:45

@sJJdGG sJJdGG left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Seems completely ok except this minor word placement

Comment thread content/docs/rendering-elements.md Outdated
Co-Authored-By: sorousht <soroush.torkzadeh@gmail.com>
@sJJdGG
sJJdGG self-requested a review March 7, 2019 10:51

@sJJdGG sJJdGG left a comment

Copy link
Copy Markdown
Contributor

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 9daceec into reactjs:master Mar 7, 2019
@tesseralis tesseralis mentioned this pull request Mar 30, 2019
44 of 87 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

translation A new page has been translated

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants