Skip to content

Translate 'Introducing Hooks' page #28

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

Merged
merged 13 commits into from
Mar 5, 2019
Merged

Conversation

g12i
Copy link
Contributor

@g12i g12i commented Feb 24, 2019

@netlify
Copy link

netlify bot commented Feb 24, 2019

Deploy preview for pl-reactjs ready!

Built with commit 58d85cb

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

@g12i g12i changed the title Translate 'Introducing Hooks' page Translate 'Introducing Hooks' page #WIP Feb 24, 2019
@g12i g12i added the work in progress Author opened the pull-request but it's still being worked on label Feb 24, 2019
@g12i g12i added needs review A pull request ready to be reviewed and removed work in progress Author opened the pull-request but it's still being worked on labels Feb 25, 2019
@jakubdrozdek
Copy link
Member

Nadal jest "WIP"? Jeśli nie, to zmień też tytuł PR-a :)

@g12i g12i changed the title Translate 'Introducing Hooks' page #WIP Translate 'Introducing Hooks' page Feb 26, 2019
@@ -25,86 +25,86 @@ function Example() {
}
```

This new function `useState` is the first "Hook" we'll learn about, but this example is just a teaser. Don't worry if it doesn't make sense yet!
Funkcja `useState` jest pierwszym "Hookiem", o którym będziemy się uczyć. Przykład ten jest jednak zaledwie zwiastunem. Nie przejmuj się, jeżeli nie ma to jeszcze większego sensu!
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Funkcja `useState` jest pierwszym "Hookiem", o którym będziemy się uczyć. Przykład ten jest jednak zaledwie zwiastunem. Nie przejmuj się, jeżeli nie ma to jeszcze większego sensu!
Funkcja `useState` jest pierwszym "hookiem", o którym będziemy się uczyć. Przykład ten jest jednak zaledwie zwiastunem. Nie przejmuj się, jeżeli nie ma to jeszcze większego sensu!

Copy link
Member

Choose a reason for hiding this comment

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

Moim zdaniem lepiej zaczynać "hooki" małą literą.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Chciałem być wierny oryginałowi w tej kwestii - z jakiegoś powodu wszędzie użyli wielkiej litery.

Copy link
Member

@jakubdrozdek jakubdrozdek Feb 27, 2019

Choose a reason for hiding this comment

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

Może podeszli do tego jak do hasła reklamowego, nazwy własnej ;-) W języku polskim nadal, imho, lepiej pasuje pisać małą literą. Nie ma w tej dokumentacji zbyt wiele tego typu nazw własnych, a sporo jest nazw przeróżnych mechanizmów.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sprawdziłem Hiszpański, też jest wielką literą:
https://es.reactjs.org/docs/hooks-intro.html
Może skonsultować się z twórcami?

Copy link
Member

Choose a reason for hiding this comment

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

Skonsultowałem się z "twórcami" :P i padła odpowiedź, że możemy pisać jak chcemy. Oni piszą wielką literą, bo chcieli nadać tej nazwie pewien stopień "specjalności".

Copy link
Member

Choose a reason for hiding this comment

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

Dla mnie dlatego pisanie od małej litery jest lepsze, bo to nie jest do końca ich wymyślona, własna nazwa. Hooki istnieją w programowaniu od lat i nie są niczym wyjątkowym - ot, kolejny mechanizm. Gdyby iść za ich schematem, to trzeba by pisać wielką literą "Props", "Lifecycle", "Components" - a tego tak nie piszemy :-)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok, masz rację - zmienię na małą. Jak ktoś jest przeciwko, teraz jest dobry czas na sprzeciw ;).

Copy link
Contributor Author

@g12i g12i Mar 1, 2019

Choose a reason for hiding this comment

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

Wciąż nie mogę się zdecydować. @cvqprs @iamarek @TeoTN @bartlomiejzuber @marcinostrouch @jakubdrozdek - może głosowanie? 🚀 - wielką, 👀 - małą

@jakubdrozdek
Copy link
Member

Bardzo dobre tłumaczenie 🎉 , przyjemnie się czyta :-) Dodałem "parę" drobnych sugestii (jak zwykle ;-) ).

@jakubdrozdek jakubdrozdek added the needs author response Changes requested needing author's input label Feb 26, 2019
@g12i g12i removed the needs author response Changes requested needing author's input label Feb 26, 2019
@g12i g12i added the question Further information is requested label Feb 27, 2019
Copy link
Member

@cvqprs cvqprs left a comment

Choose a reason for hiding this comment

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

2 kosmetyczne zmiany - reszta +100

@g12i
Copy link
Contributor Author

g12i commented Mar 4, 2019

@cvqprs @jakubdrozdek - dodałem wszystkie poprawki. Na Slacku urządzę ankietę na temat pisowni Hooków (jest szansa, że wypowie się więcej osób). I jeżeli zapadnie taki wyrok, to poprawię to w osobnym PR. Tymczasem proszę o +1 (i zapraszam innych do review)

@g12i g12i added needs +1 Needs another approval before merging and removed question Further information is requested labels Mar 4, 2019
@jakubdrozdek jakubdrozdek requested a review from cvqprs March 4, 2019 22:58
@cvqprs cvqprs merged commit d5e8967 into reactjs:master Mar 5, 2019
@cvqprs cvqprs added 🌐 translated Translation is completed for this page and removed needs +1 Needs another approval before merging needs review A pull request ready to be reviewed labels Mar 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🌐 translated Translation is completed for this page
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants