Skip to content

称号バッジの絵文字をFontAwesomeアイコンに置き換え#194

Merged
kojira merged 1 commit into
mainfrom
claude/ranking-feature-bug-a8nq2k
Jul 20, 2026
Merged

称号バッジの絵文字をFontAwesomeアイコンに置き換え#194
kojira merged 1 commit into
mainfrom
claude/ranking-feature-bug-a8nq2k

Conversation

@kojira

@kojira kojira commented Jul 20, 2026

Copy link
Copy Markdown
Member

概要

#183 の最終対応。称号バッジ17種の絵文字を FontAwesome 単色アイコンに置き換えます(機種依存の見た目を排除)。対応表はユーザーレビュー済み。

実装

  • profileStatsGo: badgeDef / profileBadgeicon(FAクラス名)を追加して返す。emoji は旧クライアント・CDNキャッシュ互換のため残す
  • ProfileStats.vue: icon があればFA表示、無ければ絵文字にフォールバック
  • 全称号に icon と emoji フォールバックがあることを保証するテストを追加

対応表(抜粋: 無料版に該当が無い4種の意訳)

  • 🏮 常連さん → fa-shoe-prints(足繁く通う)
  • 🕯️ 七日詣 → fa-moon(七夜)
  • 🎋 おみくじ好き → fa-scroll(みくじ紙)
  • 🌩️ 不運の帝王 → fa-cloud-showers-heavy

他13種は素直な対応(⛩️→torii-gate、🙏→praying-hands、💯→certificate、📅→calendar-check、🌟→star、🔥→fire、🎖️→medal、⚡→bolt、🌱→seedling、🛠️→tools、👑→crown、🌈→rainbow、💀→skull)。

検証

  • Firestoreエミュレータで go test ./... 全パス(新テスト含む)
  • yarn generate 成功
  • devで見た目確認 → OK後に本番

これで #183 の残件は解消(二拍手の👏🏻は現状維持で確定済み)。マージ後、#183 をクローズ予定。

🤖 Generated with Claude Code

https://claude.ai/code/session_01DwkB6Hj2JggoKcy7FoxNcU


Generated by Claude Code

profileStatsGoが各称号に icon(FontAwesome無料版のクラス名)を返し、
フロントは icon 優先で表示・無ければ絵文字にフォールバック
(CDNキャッシュの旧レスポンス互換)。emoji フィールドは互換のため残す。

無料版に該当が無い4種は意訳:
🏮常連さん→fa-shoe-prints / 🕯️七日詣→fa-moon /
🎋おみくじ好き→fa-scroll / 🌩️不運の帝王→fa-cloud-showers-heavy

全称号にアイコンがあることを保証するテストを追加。

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01DwkB6Hj2JggoKcy7FoxNcU
@cursor

cursor Bot commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@kojira
kojira merged commit 8d765e7 into main Jul 20, 2026
3 checks passed
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