-
Notifications
You must be signed in to change notification settings - Fork 54
Translate 'ReactDOM' page #35
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
Conversation
Translations for ReactDOM page
Deploy preview for pl-reactjs ready! Built with commit e624fca |
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.
@patpienkowski Cześć, witamy na pokładzie.
Przykro mi to mówić, ale niestety Twoje tłumaczenie nie jest najlepsze. Wygląda, jakbyś większość tekstu przepuścił przez translator i nawet go nie przeczytał. Dodałem sporo sugestii poprawek. Przeczytaj je proszę i dokończ tłumaczenie (nie poprawiłem jedynie kilku ostatnich akapitów).
Jak już skończysz, to dla pewności przeczytaj cały tekst raz jeszcze, żeby upewnić się, że wszystko jest spójne logicznie.
Dokumentacja jest jak książka i powinno się ją tak czytać. Jeśli pomiędzy zdaniami nie będzie logicznego połączenia, to czytelnik wyjdzie po chwili i nigdy nie wróci. A tego chcemy uniknąć :-)
@jakubdrozdek Masz racje. Przepraszam za takie tłumaczenie. W Polsce się nie urodziłem więc według mnie takie tłumaczenie brzmi hehe. Nie spodziewałem się ze tak zle będzie. |
@patpienkowski Dzięki za zrozumienie ;-) Nie ma za co przepraszać. Czekamy na poprawki :-) |
@patpienkowski Nie trzeba, możesz wrzucać wszystko do tego PR-a 😊 |
Requested changes added to reference-react-dom.md
tylna furtka - wyjscie awaryjne...
@jakubdrozdek Czesc! Dodalem poprawki pare dni temu. Nieiwiem czy zauwarzyles ale sa dodane jak bys mial czas przegladnac! |
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.
Pojawiły się nowe błędy.
Hej @patpienkowski daj znac prosze, czy bedziesz jeszcze pracowal nad tym PR. Jezlei nie, to moge go przejac, dodac poprawki etc. |
Tak dodam poprawki
…On Sun, Mar 31, 2019 at 1:32 AM Piotr Szymanski ***@***.***> wrote:
Hej @patpienkowski <https://github.com/patpienkowski> daj znac prosze,
czy bedziesz jeszcze pracowal nad tym PR. Jezlei nie, to moge go przejac,
dodac poprawki etc.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#35 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AsePZfLEVGYNxV5L0Y9PpASo8YDQvAddks5vcFZvgaJpZM4bZ-3F>
.
|
Changes after re-review
hej @patpienkowski prawie wszystko ok :-) Czyta sie dobrze, wiec tutaj zero uwagi, jedynie formatowanie sie troche rozjechalo. Porownaj sobie https://deploy-preview-35--pl-reactjs.netlify.com/docs/react-dom.html vs https://reactjs.org/docs/react-dom.html sekcje: Uwaga i findDOMNode Trzeba naniesc drobne zmiany zeby poprawic wyglad wtedy dajemy +1 i merge, nie wiecej niz 10 minut pracy :-) |
formatting
Changes
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.
Super! Wszystko ok! Gratuluje pierwszego tlumaczenia! :-)
Hej @bartlomiejzuber, @patpienkowski dodal zmiany zasugerowane przez @jakubdrozdek wiec zamykam ten request.
Translations for ReactDOM page