Skip to content

Overall Fixes #174

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 39 commits into from
Feb 21, 2019
Merged

Overall Fixes #174

merged 39 commits into from
Feb 21, 2019

Conversation

Darking360
Copy link
Collaborator

@Darking360 Darking360 commented Feb 18, 2019

Translation: Overall Fixes

Related: Original issue: Spanish Translation Final Tasks!

Overall checking on going, reading all core pages and submitting some changes and fixes.

@Darking360 Darking360 changed the title WIP: Overall Fixes Overall Fixes Feb 19, 2019
@Darking360
Copy link
Collaborator Author

Hey guys 💪 I finished it 🎊

So basically, I checked core pages, and some of the advanced ones The things that I encountered were these:

  • Untranslated parts, or translated parts with the english version next to them.
  • Code comments untranslated.
  • Code that was translated, so I reverted those back to original.
  • Typos and accents.
  • Formatting issues, for example when they tried to add a bold font with ** they left a space after, so the format wasn't correct.
  • Links broken in same pages, for example clicking a link to #callback-refs but that takes the user nowhere or top of the page.
  • Links to docs in english, most of those changed to Spanish versions in wikipedia or MDN.

I think that was all, I'll certainly skipped one or two but that's most of it. Please take a look (I know it's a lot) but with this I think that we fix various issues 🚀 Thank you guys! I'll be looking for your feedback 📓

CC: @alejandronanez @carburo @dmoralesm @tesseralis

@alejandronanez
Copy link
Member

Holy cow, this is a huge (and amazing) PR - I’ll take a look at it tonight!

You’re a hero 🎖

@carburo
Copy link
Member

carburo commented Feb 19, 2019

I'll take a look later, probably tonight. Great job! 💪

Copy link
Member

@carburo carburo left a comment

Choose a reason for hiding this comment

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

Nice! That was a lot of work. Here are some small things I spotted.

@netlify
Copy link

netlify bot commented Feb 20, 2019

Deploy preview for es-reactjs ready!

Built with commit 3b2fb0f

https://deploy-preview-174--es-reactjs.netlify.com

@Darking360
Copy link
Collaborator Author

Greetings guys! I just added @carburo feedback.

If you could take a last look before merging this would be awesome!

CC: @alejandronanez @tesseralis @dmoralesm

@netlify
Copy link

netlify bot commented Feb 20, 2019

Deploy preview for es-reactjs ready!

Built with commit f3c5288

https://deploy-preview-174--es-reactjs.netlify.com

@netlify
Copy link

netlify bot commented Feb 20, 2019

Deploy preview for es-reactjs ready!

Built with commit 06aff0c

https://deploy-preview-174--es-reactjs.netlify.com

Copy link
Member

@alejandronanez alejandronanez left a comment

Choose a reason for hiding this comment

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

Holy! Amazing work @Darking360 👏👏 👏 👏 👏

:shipit:

@@ -147,7 +147,7 @@ return (
);
```

Estos dos fragmentos de código son equivalentes. Mientras **JSX es [completamente opcional](/docs/react-without-jsx.html)**, muchas personas lo encuentran útil para escribir código relacionado con la IU -- ya sea usando React o con otras bibliotecas.
Estos dos fragmentos de código son equivalentes. Mientras **JSX es [completamente opcional](/docs/react-without-jsx.html)**, muchas personas lo encuentran útil para escribir código relacionado con la interfaz de usuario -- ya sea usando React o con otras bibliotecas.
Copy link
Member

Choose a reason for hiding this comment

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

Bravo!

@@ -33,7 +33,7 @@ El equipo de React principalmente recomienda las siguientes soluciones:
- Si estás **aprendiendo React** o **creando una nueva [aplicación de página única](/docs/glossary.html#single-page-application),** usa [Create React App](#create-react-app).
- Si estás construyendo un **sito web renderizado en servidor con Node.js,** prueba [Next.js](#nextjs).
- Si estás construyendo un **sitio web orientado a contenido estático,** prueba [Gatsby](#gatsby).
- Si estás construyendo una **biblioteca de componentes** o **integrando una base de código existente**, prueba [Cadenas de Herramientas más Flexibles](#cadenas-de-herramientas-ms-flexibles).
- Si estás construyendo una **biblioteca de componentes** o **integrando una base de código existente**, prueba [Cadenas de Herramientas más Flexibles](#more-flexible-toolchains).
Copy link
Member

Choose a reason for hiding this comment

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

Good catch!

@alejandronanez alejandronanez merged commit abff318 into reactjs:master Feb 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants