Translate higher order components page #95

rgrnwd · 2019-07-21T11:57:05Z

Translation for Advanced - Higher order components page

…o feature/higher-order-components

netlify · 2019-07-21T12:02:27Z

Deploy preview for he-reactjs ready!

Built with commit 50e0d4a

https://deploy-preview-95--he-reactjs.netlify.com

content/docs/higher-order-components.md

galnir · 2019-08-05T17:39:05Z

content/docs/higher-order-components.md

-      // ... and renders the wrapped component with the fresh data!
-      // Notice that we pass through any additional props
+      // ... ומרנדרת את הקומפוננטה העטופה עם המידע החדש
+      // הלאה props-שימו לב שאנחנו מעבירים את כל ה


Suggested change

// הלאה props-שימו לב שאנחנו מעבירים את כל ה

// שימו לב שאנחנו מעבירים את כל ה-props הלאה

This correction breaks the sentence when viewed in the browser, for some reason in comments the sentence order from right-to-left is different.

Can you take the suggested sentence and paste it on a program like Word or even in the browser search bar and see if it shows correctly? In my experience GitHub breaks these sentences but they appear fine in the docs

I don't have word (mac) - but in the browser bar it still shows as broken

In the version I committed it looks like this (can you confirm it's the same in your browser?)

Yes you're right I checked here and it displays correctly 👍

galnir · 2019-08-05T17:45:04Z

content/docs/higher-order-components.md


 ```js
 render() {
-  // Filter out extra props that are specific to this HOC and shouldn't be
-  // passed through
+  // שלא אמורים לעבור הלאה לקומפוננטה העטופה HOC-הקשורים ל props-קודם כל נמצא את ה


Suggested change

// שלא אמורים לעבור הלאה לקומפוננטה העטופה HOC-הקשורים ל props-קודם כל נמצא את ה

// קודם כל נמצא את ה-props הקשורים ל-HOC ושלא אמורים לעבור הלאה לקומפוננטה העטופה

Same here @galnir

galnir · 2019-08-05T17:46:15Z

content/docs/higher-order-components.md

  const injectedProp = someStateOrInstanceMethod;

-  // Pass props to wrapped component
+  // לקומפוננטה העטופה props-ועכשיו נוכל להעביר את ה


Suggested change

// לקומפוננטה העטופה props-ועכשיו נוכל להעביר את ה

// ועכשיו נוכל להעביר את ה-props לקומפוננטה העטופה

content/docs/higher-order-components.md

galnir · 2019-08-05T17:50:02Z

content/docs/higher-order-components.md

 const enhance = connect(commentListSelector, commentListActions);
-// The returned function is a HOC, which returns a component that is connected
-// to the Redux store
+// Redux store-שמחזירה קומפוננטה שמחוברת ל ,HOC הפונקציה המוחזרת היא


Suggested change

// Redux store-שמחזירה קומפוננטה שמחוברת ל ,HOC הפונקציה המוחזרת היא

// הפונקציה המוחזרת היא HOC, שמחזירה קומפוננטה שמחוברת ל- Redux store

content/docs/higher-order-components.md

galnir · 2019-08-05T17:55:09Z

content/docs/higher-order-components.md


 ```js
 render() {
-  // A new version of EnhancedComponent is created on every render
+  // נוצרות בכל רינדור EnhancedComponent גרסה חדשה של


Suggested change

// נוצרות בכל רינדור EnhancedComponent גרסה חדשה של

// גרסה חדשה של EnhancedComponent נוצרת בכל רינדור

content/docs/higher-order-components.md

galnir · 2019-08-05T17:57:49Z

content/docs/higher-order-components.md

 WrappedComponent.staticMethod = function() {/*...*/}
-// Now apply a HOC
+// על הקומפוננטה HOC-עכשיו נשתמש ב


Suggested change

// על הקומפוננטה HOC-עכשיו נשתמש ב

// עכשיו נשתמש ב- HOC על הקומפוננטה

content/docs/higher-order-components.md

galnir · 2019-08-05T18:00:58Z

content/docs/higher-order-components.md


-While the convention for higher-order components is to pass through all props to the wrapped component, this does not work for refs. That's because `ref` is not really a prop — like `key`, it's handled specially by React. If you add a ref to an element whose component is the result of a HOC, the ref refers to an instance of the outermost container component, not the wrapped component.
+למרות שהמוסכמה לקומפוננטות מסדר גבוה יותר היא להעביר את כל ה- props הלאה לקומפוננטה העטופה, זה לא עובד עבור רפרנסים (refs). הסיבה לכך היא שה- `ref` הוא לא בדיוק prop - כמו `key`, React מטפלת בו באופן מיוחד. אם תוסיפו רפרנס לאלמנט שהקומפוננטה שלו נוצרה על ידי HOC, הרפרנס מתייחס למופע הקומפוננטה המכילה החיצונית ביותר, ולא לקומפוננטה העטופה.


Suggested change

למרות שהמוסכמה לקומפוננטות מסדר גבוה יותר היא להעביר את כל ה- props הלאה לקומפוננטה העטופה, זה לא עובד עבור רפרנסים (refs). הסיבה לכך היא שה- `ref` הוא לא בדיוק prop - כמו `key`, React מטפלת בו באופן מיוחד. אם תוסיפו רפרנס לאלמנט שהקומפוננטה שלו נוצרה על ידי HOC, הרפרנס מתייחס למופע הקומפוננטה המכילה החיצונית ביותר, ולא לקומפוננטה העטופה.

למרות שהמוסכמה לקומפוננטות מסדר גבוה יותר היא להעביר את כל ה- props הלאה לקומפוננטה העטופה, זה לא עובד עבור רפרנסים (refs). הסיבה לכך היא שה- `ref` הוא לא בדיוק prop - כמו React, 'key' מטפלת בו באופן מיוחד. אם תוסיפו רפרנס לאלמנט שהקומפוננטה שלו נוצרה על ידי HOC, הרפרנס מתייחס למופע הקומפוננטה המכילה החיצונית ביותר, ולא לקומפוננטה העטופה.

This also seems to break the sentence order in the browser

galnir

Some minor changes

rgrnwd added 10 commits July 14, 2019 08:44

translate hoc introduction section

72816ee

translate hoc cross-cutting concerns section

cf5e4a3

Merge branch 'master' of https://github.com/rgrnwd/he.reactjs.org int…

5c88e01

…o feature/higher-order-components

translate hoc page cross-cutting concerns section

b0fd21d

translate composition over mutation section in hoc page

232bd4a

translate unrelated props convention section in hoc page

e639dce

translate maximising composition convention section in hoc page

1f81b6d

translate last convention section

2d0da2d

translate caveats section

9a17634

update page title on nav bar

577455b

tesseralis mentioned this pull request Jul 21, 2019

Hebrew Translation Progress #1

Closed

88 tasks

galnir reviewed Aug 5, 2019

View reviewed changes

content/docs/higher-order-components.md Outdated Show resolved Hide resolved

galnir reviewed Aug 5, 2019

View reviewed changes

content/docs/higher-order-components.md Outdated Show resolved Hide resolved

galnir reviewed Aug 5, 2019

View reviewed changes

content/docs/higher-order-components.md Outdated Show resolved Hide resolved

galnir reviewed Aug 5, 2019

View reviewed changes

content/docs/higher-order-components.md Outdated Show resolved Hide resolved

galnir reviewed Aug 5, 2019

View reviewed changes

content/docs/higher-order-components.md Outdated Show resolved Hide resolved

galnir reviewed Aug 5, 2019

View reviewed changes

content/docs/higher-order-components.md Outdated Show resolved Hide resolved

galnir reviewed Aug 5, 2019

View reviewed changes

content/docs/higher-order-components.md Outdated Show resolved Hide resolved

galnir reviewed Aug 5, 2019

View reviewed changes

content/docs/higher-order-components.md Outdated Show resolved Hide resolved

galnir reviewed Aug 5, 2019

View reviewed changes

content/docs/higher-order-components.md Outdated Show resolved Hide resolved

galnir reviewed Aug 5, 2019

View reviewed changes

content/docs/higher-order-components.md Outdated Show resolved Hide resolved

galnir reviewed Aug 5, 2019

View reviewed changes

content/docs/higher-order-components.md Outdated Show resolved Hide resolved

galnir reviewed Aug 5, 2019

View reviewed changes

galnir requested changes Aug 5, 2019

View reviewed changes

rgrnwd added 2 commits August 12, 2019 21:17

fix some translations based on PR feedback

afad029

fix one typo and add a space

50e0d4a

galnir approved these changes Aug 15, 2019

View reviewed changes

galnir merged commit 1c1d0b0 into reactjs:master Aug 16, 2019

	// הלאה props-שימו לב שאנחנו מעבירים את כל ה
	// שימו לב שאנחנו מעבירים את כל ה-props הלאה

	// שלא אמורים לעבור הלאה לקומפוננטה העטופה HOC-הקשורים ל props-קודם כל נמצא את ה
	// קודם כל נמצא את ה-props הקשורים ל-HOC ושלא אמורים לעבור הלאה לקומפוננטה העטופה

	// לקומפוננטה העטופה props-ועכשיו נוכל להעביר את ה
	// ועכשיו נוכל להעביר את ה-props לקומפוננטה העטופה

	// Redux store-שמחזירה קומפוננטה שמחוברת ל ,HOC הפונקציה המוחזרת היא
	// הפונקציה המוחזרת היא HOC, שמחזירה קומפוננטה שמחוברת ל- Redux store

	// נוצרות בכל רינדור EnhancedComponent גרסה חדשה של
	// גרסה חדשה של EnhancedComponent נוצרת בכל רינדור

	// על הקומפוננטה HOC-עכשיו נשתמש ב
	// עכשיו נשתמש ב- HOC על הקומפוננטה


		While the convention for higher-order components is to pass through all props to the wrapped component, this does not work for refs. That's because `ref` is not really a prop — like `key`, it's handled specially by React. If you add a ref to an element whose component is the result of a HOC, the ref refers to an instance of the outermost container component, not the wrapped component.
		למרות שהמוסכמה לקומפוננטות מסדר גבוה יותר היא להעביר את כל ה- props הלאה לקומפוננטה העטופה, זה לא עובד עבור רפרנסים (refs). הסיבה לכך היא שה- `ref` הוא לא בדיוק prop - כמו `key`, React מטפלת בו באופן מיוחד. אם תוסיפו רפרנס לאלמנט שהקומפוננטה שלו נוצרה על ידי HOC, הרפרנס מתייחס למופע הקומפוננטה המכילה החיצונית ביותר, ולא לקומפוננטה העטופה.

Translate higher order components page #95

Translate higher order components page #95

Uh oh!

Conversation

rgrnwd commented Jul 21, 2019

Uh oh!

netlify bot commented Jul 21, 2019 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

galnir Aug 5, 2019 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

galnir left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

netlify bot commented Jul 21, 2019 •

edited

Loading

galnir Aug 5, 2019 •

edited

Loading