Skip to content

Ready for Review#149

Merged
sahinyanlik merged 13 commits into
javascript-tutorial:masterfrom
fatmabadri:master
Jan 15, 2021
Merged

Ready for Review#149
sahinyanlik merged 13 commits into
javascript-tutorial:masterfrom
fatmabadri:master

Conversation

@fatmabadri

Copy link
Copy Markdown
Contributor

No description provided.

@CLAassistant

CLAassistant commented Jan 9, 2021

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

: `parentElem`in alt ogeleri arasinda `node` ile birlikte `oldChild`i yenisiyle yer degistirir.

All these methods return the inserted node. In other words, `parentElem.appendChild(node)` returns `node`. But usually the returned value is not used, we just run the method.
Tüm bu yöntemler eklenen düğümü geri döndürür. Diger anlatimla,`parentElem.appendChild(node)` `node`i geri döndürür.Ama genellikle geri döndürülen değer kullanılmaz, sadece yöntemi çalıştırırız.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Noktadan sonra boşluk olmalı.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Evet, haklısınız.

```

...Would lead to:
...Suna yonlendirir:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Şuna*

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

:) Klavyemde Türkçe karakterler yok. Bu repo için görsel klavye kullanacağım



## Removal methods
## Removal methodsğ

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Sanırım burada bir yanlışlık olmuş.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Evet, haklısınız

## Removal methodsğ

To remove nodes, there are the following methods:
Düğümleri kaldırmak için, aşaıdaki yöntemler vardır:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

aşağıdaki*

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yanlışlarımı düzelttiğiniz için teşekkür ederim.

```

The call to `document.write(html)` writes the `html` into page "right here and now". The `html` string can be dynamically generated, so it's kind of flexible. We can use JavaScript to create a full-fledged webpage and write it.
`document.write(html)` çağrısı "burada ve şimdi" sayfasına 'html' yazar. `html` string dinamik olarak oluşturulmus olabilir, bu nedenle esnektir. Tam teşekküllü bir web sayfası oluşturmak ve yazmak için JavaScript kullanabiliriz.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Tam teşekküllü, çok güzel olmuş :))

@sahinyanlik

Copy link
Copy Markdown
Contributor

Fatma hanım bir de lisans'ı kabul edebilir misiniz?

@fatmabadri

Copy link
Copy Markdown
Contributor Author

@sahinyanlik
Lisansi kabul ettim ama pull requesti kapatmaktan emin degilim

@sahinyanlik

Copy link
Copy Markdown
Contributor

Tamamdır, siz tamamlayınca tekrar bakalım bir de en son yaptığınız değişikliklere baktığımda sanki bazı yerleri geri almışsınız gibi gördüm. 48d0c72 burada örneğin

@sahinyanlik

Copy link
Copy Markdown
Contributor

Tabi tabi siz ne zaman isterseniz o zaman merge edelim.

Türkçe karakter düzeltmeleri yapıldı.
@fatmabadri

Copy link
Copy Markdown
Contributor Author

Merhaba @sahinyanlik,
Farkında olduğum kadarıyla, düzeltmeleri yaptım. Lütfen kısaca bir gözden geçirin. Umarım, bu kez tamdır. Hazır ise, merge edebilirsiniz.

@sahinyanlik
sahinyanlik merged commit ba434d9 into javascript-tutorial:master Jan 15, 2021
@sahinyanlik

Copy link
Copy Markdown
Contributor

Yardımlarınız için teşekkür ederim.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants