Skip to content
This repository was archived by the owner on Apr 21, 2023. It is now read-only.

Conversation

iphayao
Copy link
Contributor

@iphayao iphayao commented Mar 20, 2019

No description provided.

@iphayao iphayao closed this Mar 20, 2019
@iphayao iphayao reopened this Mar 20, 2019
@iphayao
Copy link
Contributor Author

iphayao commented Mar 20, 2019

@tesseralis please review my PR.

@tesseralis tesseralis requested a review from kiroskirin March 20, 2019 19:21
@tesseralis
Copy link
Contributor

@iphayao I don't know Thai so I don't think I can review the actual translation :) I assigned @kiroskirin to it.

The one thing I noticed is that the title on line 53 isn't translated.

Copy link

@kovr kovr left a comment

Choose a reason for hiding this comment

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

I have made a few quick suggestions on typo

@iphayao
Copy link
Contributor Author

iphayao commented Mar 26, 2019

@kovr Thank you very much, I did fixed all your review.

Copy link
Contributor

@kiroskirin kiroskirin left a comment

Choose a reason for hiding this comment

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

รูปประโยคส่วนใหญ่จะดูเป็นทางการและแปลตรง ๆ ตัวมาก
แต่ถ้ากลับมาลองไล่อ่าน แล้วรู้สึกว่าไม่เข้าใจประโยคที่ตัวเองสื่อ ก็ลองเปลี่ยนแนวดูครับ

ถ้าคำไหนทับศัพท์ได้ ก็ทับศัพท์ไปเลยครับ
เพราะส่วนใหญ่คนที่จะมาอ่าน
ย่อมต้องเข้าใจคำพื้นฐานเหล่านั้นอยู่แล้ว
เช่น user interface สามารถใช้คำว่า UI แทนก็น่าจะเข้าใจได้ง่ายกว่าครับ

@tesseralis tesseralis mentioned this pull request Mar 28, 2019
88 tasks
@iphayao iphayao merged commit 0037f8b into stale-reactjs-org-translations:master Mar 28, 2019
react-translations-bot pushed a commit that referenced this pull request Oct 10, 2022
* Added documentation for React.createFactory

* Update beta/src/content/apis/react/createFactory.md

Co-authored-by: Strek <[email protected]>

* minor editorial updates (#2)

* Update createFactory.md

* Update createFactory.md

* edits

Co-authored-by: Strek <[email protected]>
Co-authored-by: Holly Sweeney <[email protected]>
Co-authored-by: Dan Abramov <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants