Skip to content

Conversation

@NoNiiEa
Copy link

@NoNiiEa NoNiiEa commented Aug 17, 2025

GRD-103

@NoNiiEa NoNiiEa requested a review from KanonKC August 17, 2025 15:20
@KanonKC KanonKC changed the title Feature/add pdf mode to problem creaete and update Feature: add pdf mode to problem creaete and update Aug 20, 2025
@KanonKC
Copy link
Member

KanonKC commented Aug 20, 2025

สลับระหว่าง Plate ไป PDF แล้ว Title หายไปเลย

เนื่องจาก Tab ระหว่าง Plate กับ PDF เป็นที่ตัวสลับเพื่อเปลี่ยน "วิธีการแสดงคำอธิบายโจทย์" เท่านั้น ตัว Title ซึ่งเป็น "หัวข้อของโจทย์" จึงไม่ได้เกี่ยวข้องกันและไม่ควรจะหายไป เมื่อสลับจาก Plate ไปเป็น PDF

@KanonKC
Copy link
Member

KanonKC commented Aug 20, 2025

Preview PDF มีขนาดใหญ่เกินไป

image ลอง Upload File แล้วมันแสดงใหญ่เกินไป อาจจะต้องเปลี่ยนวิธีการแสดงผลตรงนี้ใหม่ (ถ้าสมมุติ PDF มีหลายหน้ามากขึ้น ก็จะเลื่อนได้ยากขึ้นไปอีก

วิธีที่คิดออกตอนนี้

  • ลองแบ่งครึ่งหน้า ซ้าย-ขวา ให้ Input ทั้งหมดอยู่ฝั่งนึงแล้วให้ตัว Preview อยู่อีกฝั่งนึงแยกกันไปเลย
  • ตัว PDF Viewer ถ้าอ้างอิงจากหน้าเว็บนี้ https://react-pdf-viewer.dev/ จะสามารถครอบไฟล์ PDF ด้วย Toolbar ของมันได้ ซึ่งอันนี้จะทำให้เราสามารถกดเปลี่ยนไปดูหน้าถัดไปได้เลย โดยไม่ต้องเลื่อนเองเยอะ ๆ

@KanonKC
Copy link
Member

KanonKC commented Aug 20, 2025

หลังจากแก้ทุกอย่างแล้วฝากทำ 2 อย่างต่อไปนี้

  • [แนะนำให้ทำอันนี้ก่อน] ตอนนี้เหมือน Branch นี้จะมีติด Conflict อยู่ ทำให้ไม่สามารถ Merge ได้ เพราะฉะนั้นเราจะต้อง Rebase จาก dev เข้ามาก่อนนะ (ถ้า Rebase ยังไม่เป็น หรือไม่รู้ว่าคืออะไร สามารถพิมพ์มาขอ How to ได้)
  • ลองรัน npm run build ก่อนทีนึง แล้วแก้ Error ที่มันแสดงขึ้นมา (Build สามารถ Error ได้จาก Warning ที่เราเจอระหว่างการ Dev ควรจะต้องแก้ให้หมด เพื่อที่จะสามารถ Build ขึ้น Production ได้)

@KanonKC KanonKC self-requested a review August 21, 2025 06:17
@KanonKC KanonKC added enhancement New feature or request and removed enhancement New feature or request labels Aug 21, 2025
@KanonKC
Copy link
Member

KanonKC commented Aug 24, 2025

น่าจะเป็น Title ของไฟล์ PDF อาจจะต้องแยกเป็นอีก State นึงออกมา
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants