Skip to content

Conversation

EmptyWork
Copy link
Contributor

@EmptyWork EmptyWork commented Nov 27, 2021

Proses Penerjemahan

Status : masih menerjemahkan.

Translate events dan semua keturunannya

Tree file yang sedang di terjemahkan

2-ui
├───2-events
│   ├───index.md ✅
│   ├───01-introduction-browser-events
│   │   ├───article.md [===50%  ]
│   │   ├───01-hide-other ✅
│   │   ├───02-hide-self-onclick ✅
│   │   ├───03-which-handlers-run ✅
│   │   ├───04-move-ball-field [3/4]
│   │   │   ├───solution.md

@CLAassistant
Copy link

CLAassistant commented Nov 27, 2021

CLA assistant check
All committers have signed the CLA.

@EmptyWork EmptyWork marked this pull request as draft November 27, 2021 02:19
Copy link
Contributor Author

@EmptyWork EmptyWork left a comment

Choose a reason for hiding this comment

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

typo: bukan hide-other-onclick tapi hide-self-onclick

Copy link
Contributor

@yoga1234 yoga1234 left a comment

Choose a reason for hiding this comment

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

LGTM! Ada beberapa typo

@EmptyWork
Copy link
Contributor Author

@yoga1234 lebih baik untuk handler itu di terjemahkan sebagai pengendali atau penangan?

@yoga1234
Copy link
Contributor

@yoga1234 lebih baik untuk handler itu di terjemahkan sebagai pengendali atau penangan?

Sepertinya penangan, karena belum ditentukan bakal pake apa jjuga sih..

@yoga1234
Copy link
Contributor

@EmptyWork kalo ada masukan boleh PR kesini bro - glossary

Copy link
Contributor

@yoga1234 yoga1234 left a comment

Choose a reason for hiding this comment

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

LGTM!

@yoga1234 yoga1234 marked this pull request as ready for review November 29, 2021 05:48
@yoga1234 yoga1234 merged commit 0a7a0ad into javascript-tutorial:master Nov 29, 2021
@EmptyWork EmptyWork mentioned this pull request Dec 8, 2021
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.

4 participants