Skip to content

Translate 'ReactDOM' page#35

Merged
cvqprs merged 6 commits into
reactjs:masterfrom
patpienkowski:master
Apr 8, 2019
Merged

Translate 'ReactDOM' page#35
cvqprs merged 6 commits into
reactjs:masterfrom
patpienkowski:master

Conversation

@patpienkowski

Copy link
Copy Markdown
Contributor

Translations for ReactDOM page

Translations for ReactDOM page
@netlify

netlify Bot commented Mar 1, 2019

Copy link
Copy Markdown

Deploy preview for pl-reactjs ready!

Built with commit e624fca

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

Comment thread content/docs/reference-react-dom.md Outdated
Comment thread content/docs/reference-react-dom.md Outdated
Comment thread content/docs/reference-react-dom.md Outdated
Comment thread content/docs/reference-react-dom.md Outdated
Comment thread content/docs/reference-react-dom.md Outdated
Comment thread content/docs/reference-react-dom.md Outdated
Comment thread content/docs/reference-react-dom.md Outdated
Comment thread content/docs/reference-react-dom.md Outdated
Comment thread content/docs/reference-react-dom.md Outdated
Comment thread content/docs/reference-react-dom.md Outdated
Comment thread content/docs/reference-react-dom.md Outdated
Comment thread content/docs/reference-react-dom.md Outdated
Comment thread content/docs/reference-react-dom.md Outdated
Comment thread content/docs/reference-react-dom.md Outdated
Comment thread content/docs/reference-react-dom.md Outdated
Comment thread content/docs/reference-react-dom.md Outdated
Comment thread content/docs/reference-react-dom.md Outdated
Comment thread content/docs/reference-react-dom.md Outdated
Comment thread content/docs/reference-react-dom.md Outdated
Comment thread content/docs/reference-react-dom.md Outdated
Comment thread content/docs/reference-react-dom.md Outdated

@jakubdrozdek jakubdrozdek left a comment

Copy link
Copy Markdown
Member

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 jakubdrozdek added needs author response Changes requested needing author's input needs review A pull request ready to be reviewed needs +1 Needs another approval before merging labels Mar 3, 2019
@jakubdrozdek jakubdrozdek added this to the Core Pages milestone Mar 3, 2019
@patpienkowski

Copy link
Copy Markdown
Contributor Author

@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.
Dziękuję za twoją uczciwość. Potrzebuje więcej czasu żeby te tłumaczenie doprowadzić do lepszego stanu. Przepraszam jeszcze raz.

@jakubdrozdek

Copy link
Copy Markdown
Member

@patpienkowski Dzięki za zrozumienie ;-) Nie ma za co przepraszać. Czekamy na poprawki :-)

@jakubdrozdek

Copy link
Copy Markdown
Member

@patpienkowski Nie trzeba, możesz wrzucać wszystko do tego PR-a 😊

Requested changes added to reference-react-dom.md
@patpienkowski patpienkowski requested a review from a team March 20, 2019 22:10
tylna furtka - wyjscie awaryjne...
@patpienkowski

Copy link
Copy Markdown
Contributor Author

@jakubdrozdek Czesc! Dodalem poprawki pare dni temu. Nieiwiem czy zauwarzyles ale sa dodane jak bys mial czas przegladnac!

Comment thread content/docs/reference-react-dom.md Outdated
Comment thread content/docs/reference-react-dom.md Outdated
Comment thread content/docs/reference-react-dom.md Outdated
Comment thread content/docs/reference-react-dom.md Outdated
Comment thread content/docs/reference-react-dom.md Outdated
Comment thread content/docs/reference-react-dom.md Outdated
Comment thread content/docs/reference-react-dom.md Outdated
Comment thread content/docs/reference-react-dom.md Outdated
Comment thread content/docs/reference-react-dom.md Outdated
Comment thread content/docs/reference-react-dom.md Outdated

@jakubdrozdek jakubdrozdek left a comment

Copy link
Copy Markdown
Member

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.

@cvqprs cvqprs removed the needs review A pull request ready to be reviewed label Mar 26, 2019
@cvqprs

cvqprs commented Mar 31, 2019

Copy link
Copy Markdown
Contributor

Hej @patpienkowski daj znac prosze, czy bedziesz jeszcze pracowal nad tym PR. Jezlei nie, to moge go przejac, dodac poprawki etc.

@patpienkowski

patpienkowski commented Apr 1, 2019 via email

Copy link
Copy Markdown
Contributor Author

Comment thread content/docs/reference-react-dom.md Outdated
Comment thread content/docs/reference-react-dom.md Outdated
Comment thread content/docs/reference-react-dom.md Outdated
Comment thread content/docs/reference-react-dom.md Outdated
Comment thread content/docs/reference-react-dom.md Outdated
Comment thread content/docs/reference-react-dom.md Outdated
@cvqprs

cvqprs commented Apr 2, 2019

Copy link
Copy Markdown
Contributor

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 :-)

Comment thread content/docs/reference-react-dom.md Outdated

@cvqprs cvqprs left a comment

Copy link
Copy Markdown
Contributor

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! :-)

@cvqprs cvqprs dismissed bartlomiejzuber’s stale review April 8, 2019 05:34

Hej @bartlomiejzuber, @patpienkowski dodal zmiany zasugerowane przez @jakubdrozdek wiec zamykam ten request.

@cvqprs cvqprs merged commit 9d65636 into reactjs:master Apr 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs author response Changes requested needing author's input

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants