wevisdemo / they-work-for-us

Thailand's Politician Directory website
https://theyworkforus.wevis.info
31 stars 12 forks source link

feat(/party/ชื่อพรรค): เพิ่มให้ดู ผลการลงมติทั้งหมด ของพรรคนั้น ในหน้ารวมพรรค ได้ #21

Closed thanisara-r closed 1 year ago

thanisara-r commented 2 years ago

ตอนนี้หน้าแต่ละพรรค ดูได้แค่ 6 การลงมติล่าสุด แต่อยากให้ดูผลการลงมติได้ทั้งหมด (เฉพาะผลโหวตของพรรคนั้นๆ) เหมือนหน้า https://theyworkforus.wevis.info/votelog image

Th1nkK1D commented 1 year ago

Dev starting point

หน้านี้อยู่ที่ไฟล์ src/templates/party-template.js ครับ มันมีการเรียก joinPeopleVotelog ซึ่งมี default limit = 6 น่าจะต้องปรับ function ให้รองรับเคสนี้ที่ไม่มี limit ด้วย

bothonachiz commented 1 year ago

ขอรับ issue นี้ไว้ทำครับ

Th1nkK1D commented 1 year ago

ปรึกษา @thanisara-r @namsaisup ครับ ผมลองเล่นจากที่ @bothonachiz ทำไว้ มันมี votelog เยอะมากๆ ผมเบลอเองที่เข้าใจว่า votelog จะอยู่ล่างสุดของหน้า ต่อให้ยาวก็ไม่กังวลมาก แต่จริงๆ มันมีสมาชิกในพรรคมาต่อ การที่ votelog ยาวขนาดนี้อาจจะทำให้คนไปไม่ถึงส่วนสมาชิกพรรคด้านล่างหรือเปล่านะ ซึ่งเอาจริงสมาชิกพรรคก็ยาวเหยียดพอกัน

ผมพอนึก solution ออกเร็วๆ ได้สองอัน

  1. เพิ่ม table of content เหมือนเป็นลิ้งให้กดเลื่อนไป section ต่างๆ ได้ แล้วมีลิ้งย้อนขึ้นมาข้างบน (อันนี้ในมุม dev ง่ายดี)
  2. ทำเป็น pagination อันนี้ซับซ้อนขึ้นมาหน่อย ในมุม dev ก็คงเป็น client-site pagination แหละไม่ต้องแยกหน้าจริงๆ

ปล. ถ้าประเมินว่ามีงานต้องทำประมาณนึงเดี๋ยวอาจจะเปิดเป็นอีก issue ครับ

https://user-images.githubusercontent.com/8860448/216985470-b0207f94-13f2-4e52-8d57-07ada6b1240c.mp4

namsaisup commented 1 year ago

@Th1nkK1D แบบนี้โอปะ (ดูในfigmaหน้า👥หน้า Profile กลุ่ม_feb2023)

CleanShot 2566-02-14 at 20 09 51@2x

คือให้มีแถบ sticky สารบัญ โดยเมื่ออยู่ในsectionนั้นๆ ลูกศรชี้ลงจะหายไป และเมื่ออยู่ในsectionสมาชิก ลูกศรของแถบลงมติจะเปลี่ยนเป็นชี้ขึ้นแทน (แบบงานนี้)

https://user-images.githubusercontent.com/123941533/218748563-fe1c839f-938a-4f45-8647-744d165e04a0.mp4

Th1nkK1D commented 1 year ago

@namsaisup ชอบบบ ผมว่าเป็น solution ที่โอเคนะครับ เดี๋ยวผมเอาไปเปิด issue ใหม่

Th1nkK1D commented 1 year ago

Merged #35 with 8df3436