Skip to content
This repository has been archived by the owner on Apr 21, 2023. It is now read-only.

Translation for Main Concepts' Handling Events page. #23

Conversation

asheerrizvi
Copy link
Contributor

Please point out any grammatical or spelling correction(s) which needs to be made and I'll fix it. Would also appreciate suggestions on words which could be further translated or presented in a better way.

#22

@netlify
Copy link

netlify bot commented Jul 31, 2020

Deploy preview for ur-reactjs ready!

Built with commit bdf215a

https://deploy-preview-23--ur-reactjs.netlify.app

Copy link
Contributor

@jabranr jabranr left a comment

Choose a reason for hiding this comment

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

There are way too many changes required. This is not a good translation. There are typos, incorrect sentences. I have given some suggestions but I don't want to go through rest of the it unless you go through it first and re-think the translation.

content/docs/handling-events.md Outdated Show resolved Hide resolved
content/docs/handling-events.md Outdated Show resolved Hide resolved
content/docs/handling-events.md Outdated Show resolved Hide resolved
content/docs/handling-events.md Outdated Show resolved Hide resolved
content/docs/handling-events.md Outdated Show resolved Hide resolved
content/docs/handling-events.md Outdated Show resolved Hide resolved
content/docs/handling-events.md Outdated Show resolved Hide resolved
content/docs/handling-events.md Outdated Show resolved Hide resolved
content/docs/handling-events.md Outdated Show resolved Hide resolved
@jabranr jabranr linked an issue Aug 3, 2020 that may be closed by this pull request
@asheerrizvi
Copy link
Contributor Author

There are way too many changes required. This is not a good translation. There are typos, incorrect sentences. I have given some suggestions but I don't want to go through rest of the it unless you go through it first and re-think the translation.

@jabranr Thank you for the feedback, I was unsure about many things so I wanted to put something out first. I will improve on your suggestions and come up with something better.

Copy link
Contributor

@jabranr jabranr left a comment

Choose a reason for hiding this comment

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

Great work! Thank you. Just couple of changes for typos.

content/docs/handling-events.md Outdated Show resolved Hide resolved
content/docs/handling-events.md Outdated Show resolved Hide resolved
@asheerrizvi
Copy link
Contributor Author

@jabranr thanks! I have done the requested changes, you can check now.

@jabranr jabranr added the ready to merge Work approved code review and ready to release label Sep 28, 2020
@jabranr jabranr merged commit 27a7bed into stale-reactjs-org-translations:master Sep 28, 2020
@jabranr jabranr added released Work merged into master and removed ready to merge Work approved code review and ready to release labels Sep 28, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
released Work merged into master
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Translate Handling Events Page.
2 participants