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
translate legacy factories page #85
translate legacy factories page #85
Conversation
Deploy preview for tr-reactjs ready! Built with commit 6cbb27d |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you so much. Just a few things. Please check them!
content/warnings/legacy-factories.md
Outdated
@@ -54,6 +54,6 @@ function render(MyComponent) { | |||
} | |||
``` | |||
|
|||
## In Depth {#in-depth} | |||
## Daha derin detay {#in-depth} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This might be more natural:
Daha Fazla Detay
@@ -27,9 +27,9 @@ function render() { | |||
} | |||
``` | |||
|
|||
## Without JSX {#without-jsx} | |||
## JSX Olmadan {#without-jsx} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can keep this. This describes a scenario. But below, (Dynamic components without JSX
) it implies a name or an adjective. There is a tiny difference between them.
content/warnings/legacy-factories.md
Outdated
|
||
## Dynamic components without JSX {#dynamic-components-without-jsx} | ||
## JSX Olmadan dinamik bileşenler {#dynamic-components-without-jsx} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
JSX'siz dinamik bileşenler
I believe we (others) used this pattern in other translations. So it is better to stick to it.
Hi I've implemented the improvements |
No description provided.