search

ozanyerli / turkeyvisited

Mark the cities you have visited in Turkey and share the map!
https://ozanyerli.github.io/turkeyvisited/
MIT License
197 stars 41 forks source link

Bölgelere Göre Şehirler #7

Closed samilboyalikli closed 4 months ago

samilboyalikli commented 5 months ago

Bölgelere göre ziyaret edilen şehirleri gösteren basit bir buton eklemeyi düşündüm:

Ekran Görüntüsü (222)

Sayfada çok yer kaplamayacak olan bu küçük buton için sayfanın genel temasıyla uygun CSS özellikleri seçmeye çalıştım. Bölgelere göre şehirler normal sayfada yer almıyor. Sadece butona tıklandığı zaman küçük bir kutucuk olarak alta çıkıyorlar:

Ekran Görüntüsü (221)

Aynı şekilde sayfa yenilendiği zaman bölge sayaçlarındaki sayıların kaybolmaması için ayrıca lokalde de bu dataların güncel olarak işlenmesini sağladım:

Ekran Görüntüsü (223)

ozanyerli commented 4 months ago

Elinize sağlık.

Öncelikle koda herhangi bir yorum eklemeniz gerektiğini düşünmüyorum. Kodun kendisi zaten yeterince açıklayıcı. Aynı şeyi tekrar yazmanın pek bir anlamı yok. Aksine, bu kadar çok yorum olunca kodu okumayı zorlaştırıyor.

Branch'inizi master'a aldığımız takdirde halihazırda sayfayı daha önce ziyaret etmiş olan kullanıcılarda harita problemli çalışacaktır. Eklediğiniz localStorage objeleri marmaraSehirleri, egeSehirleri vs. daha önce ziyaret etmiş kullanıcılarda boş gelecek, şehirler citiesSelected objesinde tutuluyor olacaktır. Bu sebeple daha önce eklenen şehirler haritadan çıkarılamıyor ve seçilen il sayısı sayacı hatalı çalışıyor.

Sorunu localhost'unuzda önce master branchi çalıştırıp bazı şehirleri seçip daha sonra aynı adreste kendi branchinizi çalıştırıp seçili olan şehirleri çıkarmaya çalışarak reproduce edebilirsiniz.

Sanırım halihazırda citiesSelected objesinde tutulan şehirlerin bölgesine göre yeni oluşturulan objelere yerleştirilmesi ile çözülecektir.

Bu özelliği master branch'e almayı planlamıyorum. Bölgelere göre sayıları görmek gibi bir taleple karşılaşmadım ve olabildiğince sade tutmaya çalışıyorum.

ozanyerli commented 4 months ago

Branchinizi test ederken fark ettim, haritada seçili olan şehirleri temizlemek için bir buton olması kullanışlı olabilirdi, bir issue açacağım.

samilboyalikli commented 4 months ago

Bu özelliği yazdıktan sonra önceki maillerinizi gördüm kusura bakmayın. Yeni yolladığım req'te açıklama yapmamıştım zaten şanslıymışım ama elimdeki bölgeler butonunu bırakmıştım. İssue'da yazdım zaten. En kötü ihtimalle "Bölgeler" butonunun olmadığı "Sıfırla" butonunun olduğu bir versiyonu gece 1 gibi yollarım. Yapıcı eleştiriniz için teşekkürler.

İyi çalışmalar.

24 May 2024 Cum 17:54 tarihinde Ozan Yerli @.***> şunu yazdı:

Branchinizi test ederken fark ettim, haritada seçili olan şehirleri temizlemek için bir buton olması kullanışlı olabilirdi, bir issue açacağım.

— Reply to this email directly, view it on GitHub https://github.com/ozanyerli/turkeyvisited/pull/7#issuecomment-2129735157, or unsubscribe https://github.com/notifications/unsubscribe-auth/BBN63F7NITCIBXRJD5IIJYLZD5ICXAVCNFSM6AAAAABH3CZZB2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMRZG4ZTKMJVG4 . You are receiving this because you authored the thread.Message ID: @.***>