Skip to content

Conversation

cvqprs
Copy link
Member

@cvqprs cvqprs commented Mar 4, 2019

Panowie, postanowiłem podzielić PR od @TeoTN na 2 części.

Ten PR jest dla Introducing JSX.

Pozwoli to na zachowanie porzadku (1 rozdział = 1PR) i szybszy approve.

Poprzednie tłumaczenie i zmiany dodane.

@cvqprs cvqprs changed the title New PR for introducing-jsx (previous translation included) Translate 'Introducing JSX' page. Mar 4, 2019
@cvqprs cvqprs added needs review A pull request ready to be reviewed needs +1 Needs another approval before merging labels Mar 4, 2019
@cvqprs
Copy link
Member Author

cvqprs commented Mar 4, 2019

@jakubdrozdek @iamarek wszystkie wasze sugestie sa w PR - także szybki rzut okiem, +1 i robimy merge :-) Pozostałych oczywiście też zachęcam do spojrzenia.

@netlify
Copy link

netlify bot commented Mar 4, 2019

Deploy preview for pl-reactjs ready!

Built with commit aba2197

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

Copy link
Contributor

@g12i g12i left a comment

Choose a reason for hiding this comment

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

image

Strasznie źle się czyta diff przez te nowe linie - przywróć proszę oryginalny układ.

@jakubdrozdek jakubdrozdek added this to the Core Pages milestone Mar 6, 2019
@cvqprs
Copy link
Member Author

cvqprs commented Mar 7, 2019

@g12i poprawilem w c674e7c - musze przestac korzystac z online edytorow do sprawdzania ogonkow :-)

Copy link
Member

@jakubdrozdek jakubdrozdek left a comment

Choose a reason for hiding this comment

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

Dodałem trochę poprawek.

@cvqprs cvqprs requested review from jakubdrozdek and g12i March 9, 2019 08:55
@cvqprs cvqprs removed the needs review A pull request ready to be reviewed label Mar 9, 2019
@cvqprs
Copy link
Member Author

cvqprs commented Mar 9, 2019

@jakubdrozdek @g12i poprawilem formatowanie i dodalem sugerowane zmiany. Prosze o review i +1 jak wszystko jest OK. Fajnie by bylo w ten weekend wyczyscic troche PR.


You may use quotes to specify string literals as attributes:
Możesz przekazać literał znakowy jako atrybut, używając poniższej składni:
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Możesz przekazać literał znakowy jako atrybut, używając poniższej składni:
Możesz przekazać literał ciągu znaków jako atrybut, używając poniższej składni:

Copy link
Member Author

Choose a reason for hiding this comment

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

W tym wypadku trzymał bym sie przyjętego nazewnictwa i pozostał przy literał znakowy - jest to nazwa zrozumiała i używana przez wszystkich więc nie ma co jej zbytnio komplikowac.

https://pl.wikipedia.org/wiki/Litera%C5%82_znakowy

@cvqprs cvqprs requested a review from g12i March 9, 2019 12:54
@cvqprs
Copy link
Member Author

cvqprs commented Mar 9, 2019

@jakubdrozdek i @g12i - dzieki za wszystkie uwagi i sugestie! :-) Robie merga i closed

@cvqprs cvqprs merged commit 6050594 into master Mar 9, 2019
@cvqprs cvqprs added 🌐 translated Translation is completed for this page and removed needs +1 Needs another approval before merging 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