Skip to content

Conversation

@uguratar
Copy link
Member

No description provided.

Uğur Atar added 2 commits September 24, 2017 01:14
Namespaces kısımındaki çeviri hatalarının düzeltilmesi.
@uguratar uguratar added this to the Versiyon 2 milestone Sep 23, 2017
@uguratar uguratar requested review from msusur and ozgunbal September 23, 2017 23:19
@@ -1,5 +1,5 @@
## Namespaces
Namespaces provide you with a convenient syntax around a common pattern used in JavaScript:
## Aduzayları (Namespaces)
Copy link
Member

Choose a reason for hiding this comment

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

Uğur, aduzayları çevirini glossary.md'ye de ekler misin?

Copy link
Member Author

Choose a reason for hiding this comment

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

Ekledim @onuar, atlamışım.

One thing to note is that namespaces can be nested so you can do stuff like `namespace Utility.Messaging` to nest a `Messaging` namespace under `Utility`.

For most projects we recommend using external modules and using `namespace` for quick demos and porting old JavaScript code.
Örnek proje oluştururken veya eski JavaScript kodunu port ederken harici modül ve `namespace` kullanımı tavsiye edilmektedir.
Copy link
Member

Choose a reason for hiding this comment

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

"ederken harici" arası fazla boşluk.

Copy link
Member Author

Choose a reason for hiding this comment

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

Eye for detail :)

Uğur Atar added 2 commits September 24, 2017 11:14
Namespace -> Aduzayı eklendi
Whitespace temizliği.
@uguratar
Copy link
Member Author

@onuar hallolmuş olmalı şimdi, tekrar review alabilir miyim?

@onuar
Copy link
Member

onuar commented Sep 26, 2017

Eline sağlık abi.

@onuar onuar merged commit a5b2250 into CodeFiction:master Sep 26, 2017
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.

3 participants