Closed goktugs closed 1 year ago
vercel[bot]
commented
1 year ago The latest updates on your projects. Learn more about Vercel for Git ↗︎
| Name | Status | Preview | Comments | Updated (UTC) |
|---|---|---|---|---|
| lugat | ✅ Ready (Inspect) | Visit Preview | ✅ 1 resolved | Apr 22, 2023 8:47am |
cemtatli
commented
1 year ago Selam, kategori olarak filtreleme özelliği çok iyi olmuş fakat örneğin bir daha onClick olunduğunda filtreleme işlemini iptal etse UX anlamında daha iyi olur diye düşünüyorum.
goktugs
commented
1 year ago Selam. Haklısın ux açısından daha iyi oldu. Bu filtreleme querylerini state'de tutuyoruz ya onu url'e taşıma olayına bişe söylememişsin ama o olay baya iyi. İmplamente edilebilir. Ekstra olarak kullanıcının filtreleri localstorage'da tutulabilir. Bunları issue olarak açıcam. Aslında bu pr'a ekleyebilirim ama başlık farklı bu işler farklı doğru olmaz.
Birde burda ben ing başladım başta ama türkçe cevap verdiğin için türkçe cevap verdim. Bundan sonra full ingilizce yazıcam paso çünkü localization eklicem. Projeye başkaları girdiğinde anlasınlar ne yaptığımızı konuştuğumuzu.
When user click category button, filter should work. I think its an expected behaviour.
Beside i think filtering queries (like search or category selection etc.) should be kept in url. Because maybe users wants to share their search. However i didn't add because its changes alot of top level code.
If you open issue i think i can handle but now its working at state level like search.