feat: redesign Hợp Nhất sidebar, intro/social videos (autoplay+loop+f… - #1
Merged
Merged
Conversation
…aststart), next-lesson button - Bảng Nguồn Gốc Triết Học (sidebar phải): thu thập 2 mảnh ghép → ghép thành đúc kết hoàn chỉnh - Chèn + faststart video mở đầu & xã hội; tất cả video autoplay + loop - Chuyển câu hỏi nhận thức sang dạng chấm điểm; tách khối GHI NHỚ thành 3 ý - Thêm nút 'Bài học tiếp theo' ở màn hoàn thành (helper findNextLesson) - Redesign JourneyHeader thành stepper trực quan; nút Quay lại lùi từng chặng Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
Pull request overview
This PR updates the “Hành trình Khai Sáng” interactive lesson UI/flow by introducing autoplay+loop intro/social videos, a right-side “Nguồn Gốc Triết Học” merge panel, a redesigned stepper header with back navigation, and a “Bài học tiếp theo” CTA after completion.
Changes:
- Replace intro/social scene banners with video-based headers and update question flow (graded cognitive question; “GHI NHỚ” rendered as a 3-item list).
- Add “Bảng Hợp Nhất” sidebar to visualize collected pieces and enable merging into a final consolidated statement.
- Add next-lesson navigation (
findNextLesson) and completion screen CTA, plus a redesigned stepper header with per-stage back navigation.
Reviewed changes
Copilot reviewed 4 out of 6 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| src/pages/PhilosophyJourney.jsx | Adds video scene headers, merge sidebar + merged state, redesigned JourneyHeader stepper/back, and completion next-lesson CTA. |
| src/pages/Lesson.jsx | Wires nextLesson + onNextLesson into PhilosophyJourney. |
| src/data/mindmapData.js | Adds findNextLesson helper to compute the next lesson in flattened order. |
| src/data/journeyContent.js | Updates lesson script/content to match new graded flow, warning list format, and merged final statement. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
155
to
166
| <video | ||
| // React khong phai luc nao cung set dung DOM property `muted` qua JSX attribute, | ||
| // nen set truc tiep qua ref de dam bao video co tieng khi muted=false. | ||
| ref={(el) => { | ||
| if (el) el.muted = muted; | ||
| }} | ||
| src={src} | ||
| controls | ||
| autoPlay | ||
| muted | ||
| autoPlay={autoPlay} | ||
| loop | ||
| playsInline | ||
| preload="metadata" |
Comment on lines
499
to
510
| {phase === 0 && ( | ||
| <div className="bg-gray-50 rounded-2xl border border-gray-200 p-5 md:p-6"> | ||
| <DialogueSequence lines={R.setup} onComplete={() => setPhase(1)} ctaLabel="Vào vai trải nghiệm" /> | ||
| <div className="flex justify-end"> | ||
| <button | ||
| type="button" | ||
| onClick={() => setPhase(1)} | ||
| className="inline-flex items-center gap-1.5 bg-red-800 text-white px-5 py-2.5 rounded-lg font-bold hover:bg-red-900 transition-colors" | ||
| > | ||
| Vào vai trải nghiệm | ||
| <span className="material-symbols-outlined text-base">arrow_forward</span> | ||
| </button> | ||
| </div> | ||
| )} |
Phongxuan123
added a commit
that referenced
this pull request
Jul 3, 2026
feat: major FE upgrade v1.1.0 -> v1.3.0
Phongxuan123
added a commit
that referenced
this pull request
Jul 3, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
…aststart), next-lesson button