Skip to content

chore: loại bỏ emoji khỏi giao diện, thay bằng icon nhất quán - #34

Merged
Phongxuan123 merged 1 commit into
mainfrom
chore/remove-emoji
Jul 25, 2026
Merged

chore: loại bỏ emoji khỏi giao diện, thay bằng icon nhất quán#34
Phongxuan123 merged 1 commit into
mainfrom
chore/remove-emoji

Conversation

@Phongxuan123

Copy link
Copy Markdown
Collaborator

Closes #33

Rà soát

Quét toàn bộ codebase (frontend, admin, backend, data) tìm ký tự emoji — phát hiện 26 điểm, xử lý 23.

Thay đổi

  • Nhãn tab thuần text (Khu luyện tập học viên + admin, Góc tranh luận, Nodes admin): bỏ emoji prefix, giữ text.
  • Trạng thái flashcard Khó/Dễ: bỏ emoji.
  • Icon hiển thị chính (cúp chiến thắng ở MatchingQuiz/WarmupSection/FinalQuiz, mặt buồn khi thất bại): thay bằng Material Symbols (emoji_events, cancel, sentiment_very_dissatisfied) đã dùng sẵn trong toàn app, thay vì bỏ trống mất tín hiệu thị giác.

Giữ nguyên (không phải emoji trang trí UI)

  • backend/README.md: emoji: 🧠 là field bắt buộc trong frontmatter cấu hình Hugging Face Space (icon hiển thị trên card Space), không phải nội dung sản phẩm.
  • BannerSlider.jsx: / là ký tự mũi tên chức năng cho nút chuyển slide, không phải emoji.

Kiểm thử

  • Frontend: vite build ✅ · vitest run 11/11 ✅
  • Admin: vite build ✅ · vitest run 1/1 ✅

Rà soát toàn bộ codebase (frontend, admin, backend, data) tìm ký tự emoji.
Xử lý 23/26 điểm phát hiện:

- Nhãn tab (Khu luyện tập, Góc tranh luận, Nodes admin): bỏ emoji prefix,
  giữ text thuần.
- Trạng thái flashcard Khó/Dễ: bỏ emoji, giữ text.
- Icon cúp/thất bại mang tính hiển thị chính (MatchingQuiz, WarmupSection,
  FinalQuiz): thay bằng Material Symbols (emoji_events, cancel,
  sentiment_very_dissatisfied) đã dùng sẵn trong toàn app, thay vì bỏ trống.

Giữ nguyên 3 điểm không phải emoji trang trí UI:
- backend/README.md: `emoji: 🧠` là field bắt buộc trong frontmatter cấu
  hình Hugging Face Space, không phải nội dung sản phẩm.
- BannerSlider.jsx: ❮/❯ là ký tự mũi tên chức năng cho nút chuyển slide.
Copilot AI review requested due to automatic review settings July 25, 2026 01:35
@Phongxuan123
Phongxuan123 merged commit 7923d9f into main Jul 25, 2026
3 checks passed
@vercel

vercel Bot commented Jul 25, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
philo-mind Ready Ready Preview, Comment Jul 25, 2026 1:35am
philo-mind-admin Ready Ready Preview, Comment Jul 25, 2026 1:35am

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

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.

Loại bỏ emoji trong giao diện (Khu luyện tập, Góc tranh luận, admin...)

2 participants