Sorunlar

[cagdasdag]

👎

• Neden iki tane JS dosyası var? Denemelerini localinde tutabilirsin. Git'e sadece projenin ihtiyacı olan kodları atmalıyız. Temiz tutmalıyız burayı
• sascss diye bir kelime ilk defa görüyorum. Sass veya scss kullanalım.
• Değişken kullanımına biraz önem verebiliriz sass'da. Küçük bir proje olduğu için ve tasarım olmadığı için önemi yok ama bu bilinci oturtalım.
• <a href="#" sıkıntılı bir kullanım. Bu linklere basıldığında sayfanın en üstüne gider. Bunun yerine <a href="javascript:void(0)" kullanalım.
• JS'de değişken isimleri yanlış. Hepsini camelCase yapalım.
• //Comment 'ları ingilizce olsun. Türkçe değişken ismi vermiyoruz türkçe kod yazmıyoruz.
• https://github.com/katirci/to-do/blob/ac6fe6803c0958699744edba7e86220d7590089f/js/index.js#L67 Bu ne?
• Boşluklar tablar falan tamamen yanlış. Otomatik düzeltme yok mu? İstersen damladan al böyle olmaması lazım.
• Adderinput nedir? Adder diye bir kelime yok. Kodu çalıştırmadan baktığımda anlayabilmem gerekiyor. TÜM isimlendirmeleri buna göre düzenleyelim.

👍

• Sass kullanmış olman güzel
• Class isimleri genel olarak iyi.

[katirci]

Bunların üzerine çalışıp düzeltilmesi halinde hemen size dönüş yapıyorum