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

Translate ReactDOMServer page #43

Merged
merged 1 commit into from
Mar 11, 2019

Conversation

galtalmor
Copy link
Collaborator

No description provided.

@tesseralis tesseralis mentioned this pull request Mar 7, 2019
88 tasks

- [`renderToNodeStream()`](#rendertonodestream)
- [`renderToStaticNodeStream()`](#rendertostaticnodestream)

* * *

## Reference {#reference}
## סימוכין {#reference}
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think it'll be too complicated for Hebrew readers, but I understand it has a good meaning. I would like to see what other contributors think about it and by the way, we should also add it to the style guide #22

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

It doesn't fit in every context of the word, but in this context, as I have to translate it as a header, this is the most suitable translation I found. Wouldn't mind changing it later on if we find a better translation.

Copy link
Collaborator

Choose a reason for hiding this comment

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

I found some, but I don't think they are a good fit (e.g עיון, התייחסות). What about leaving it as it is?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I wouldn't want to keep this in English since it is not a technical term and I find this translation to be accurate. I understand it sounds a little legal-ish, but this is the language :)

Copy link
Collaborator

Choose a reason for hiding this comment

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

We have to remember that our job is to make the reader’s life easier and obviously, in this case, it is not. Let's leave it open for now until we find something else.

@@ -49,9 +49,9 @@ If you call [`ReactDOM.hydrate()`](/docs/react-dom.html#hydrate) on a node that
ReactDOMServer.renderToStaticMarkup(element)
```

Similar to [`renderToString`](#rendertostring), except this doesn't create extra DOM attributes that React uses internally, such as `data-reactroot`. This is useful if you want to use React as a simple static page generator, as stripping away the extra attributes can save some bytes.
דומה ל-[`renderToString`](#rendertostring), מלבד העובדה שאינה יוצרת מאפייני DOM נוספים ש-React משתמשת בהם באופן פנימי, כגון `data-reactroot`. אפשרות זו שימושית אם ברצונך להשתמש ב-React כמחולל של דפים סטטיים פשוטים, שכן הסרת המאפיינים הנוספים יכולה לחסוך כמה בתים.
Copy link
Collaborator

Choose a reason for hiding this comment

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

According to Morfix, there are two options for byte, בית or בייט. I know בית is the Hebrew word but בייט sounds more clear to me. Tell me what do you think about it.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

As far as I know, in most literature I read בית is usually used, this is why I used it as well, also I think it sounds better in plurals. I think it is better than the other option, but if there's a consensus I wouldn't mind changing.

Copy link
Collaborator

Choose a reason for hiding this comment

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

At first, it confused me, but after I read it twice it was understandable. It doesn't mind to me, but I know it'll be clearer to others.

@galtalmor
Copy link
Collaborator Author

Let's push this in and if we come with better decisions change it in the future. I think there's no point in holding the PR.

@itayyehezkel
Copy link
Collaborator

Sounds great to me!

@itayyehezkel itayyehezkel merged commit d097d42 into reactjs:master Mar 11, 2019
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