Skip to content

Translation of the page "Conditional rendering" #110

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 9 commits into from
Feb 28, 2019

Conversation

deblasis
Copy link
Collaborator

Apart from the translation itself, notable changes:

  • Translated default link description for CodePen links
  • Imported CodePen examples for this page into the source code and also translated them (instead of being on Dan's account, which we can't translate obviously 😄)
  • Fixed the bug:
    image
    Caused by the fact that we had non-empty description for "local" CodePen examples

When using a non-empty link title (translated) we get undefined
Changing the default to the translated version and leaving them empty
fixes the issue
imported codepen examples into source code
I'll ask if it's the case of doing that upstream as well for better...
"translatability" 😂
@netlify
Copy link

netlify bot commented Feb 24, 2019

Deploy preview for it-react-org failed.

Built with commit 3f9d55f

https://app.netlify.com/sites/it-react-org/deploys/5c7772d5f17498000757d775

@deblasis deblasis changed the title Conditional rendering Translation of the page "Conditional rendering" Feb 24, 2019
@deblasis deblasis added needs +1 Needs an approval before merging (not from the author) 💪 long page - more than 100 lines Looooong page, expect delays! labels Feb 24, 2019
@netlify
Copy link

netlify bot commented Feb 24, 2019

Deploy preview for it-reactjs ready!

Built with commit 3f9d55f

https://deploy-preview-110--it-reactjs.netlify.com

@deblasis
Copy link
Collaborator Author

Closes #21

Copy link
Collaborator

@LucaBlackDragon LucaBlackDragon left a comment

Choose a reason for hiding this comment

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

Just a single little typo!

@deblasis deblasis merged commit 4035d87 into master Feb 28, 2019
@deblasis deblasis deleted the conditional-rendering branch February 28, 2019 05:38
@LucaBlackDragon LucaBlackDragon added ✔ translated and removed needs +1 Needs an approval before merging (not from the author) ✔ translated labels Mar 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants