Skip to content

Space Groteskのセルフホスト、カードのフリップスタイル追加#13

Merged
osgsm merged 12 commits into
mainfrom
tomo/dev
Jun 26, 2026
Merged

Space Groteskのセルフホスト、カードのフリップスタイル追加#13
osgsm merged 12 commits into
mainfrom
tomo/dev

Conversation

@git-hub-tomosan

@git-hub-tomosan git-hub-tomosan commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

@osgsm
@akiya64

以下の対応が含まれます

  • importの並び替え
  • Space Groteskフォントのセルフホスト
  • カードのhoverスタイルを追加
  • カード裏のスタイルにShikiを適用

ご確認をよろしくお願いします

@git-hub-tomosan git-hub-tomosan requested review from akiya64 and osgsm June 23, 2026 08:39
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jun 24, 2026

Copy link
Copy Markdown

Deploying hyacker with  Cloudflare Pages  Cloudflare Pages

Latest commit: f776750
Status: ✅  Deploy successful!
Preview URL: https://299b4674.hyacker.pages.dev
Branch Preview URL: https://tomo-dev.hyacker.pages.dev

View logs

@osgsm

osgsm commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

@git-hub-tomosan

追加でプッシュいただいた df7ead0 分以外の箇所のレビューです。

カードについて

  • キーボードフォーカス時にカードにフォーカスリングを表示できるとうれしい
  • スクロール可能を示唆する上方向のマスクがない
  • フリップの動きアニメーション周りを調整予定なのでそのときに解決
    • フリップ中にホバーの動きが走ることがあるので、JS を使ってフリップのタイミングを制御したりが必要かも
    • 回転中にカードの角がほかの要素に隠れることがあるので、回転中は z-index を上げるとかで対応できないか
  • Copy ボタンは <button> を使う話し合いの結果 div でいくことに
    • カード自体が <a> でラップされているためマークアップも要変更

「回答」の抽出について

  • 表記揺れが起きてもいいように、回答という文字にマッチした見出しではなく、最初の見出しの内容を取得などがよさそう

@akiya64 akiya64 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

いいと思います。
一点だけ、不具合ありましたので確認願います。

Comment thread src/components/SearchResults.tsx
@git-hub-tomosan

Copy link
Copy Markdown
Contributor Author

@osgsm
@akiya64

  • カードの回転アニメーションを、hoverで拡大→回転→元のサイズに戻る、に調整しました

ご確認をよろしくお願いします

@osgsm

osgsm commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

@git-hub-tomosan cc. @akiya64
回転アニメーションいいですね!
次のことが実現できると、よりよくなりそうだと感じました。

  • アニメーションが「拡大」→「回転」→「縮小」で分離されている感じがあるのを調整
    • 拡大終了を待って回転するのではなく、拡大しながら回転する
    • 縮小のアニメーションも回転が完全に終わる前に発火できるとより自然になりそう
    • 現状の動きだと機械っぽく見えてしまうかなと思いました

@git-hub-tomosan

git-hub-tomosan commented Jun 26, 2026

Copy link
Copy Markdown
Contributor Author

@osgsm
確認ありがとうございます!

「拡大終了を待って回転するのではなく、拡大しながら回転する」
こちらですが、「拡大と回転を同時に発火」という状態は想定していますか?
最初の実装は「拡大と回転を同時に発火」だったのですが、少し微妙に感じてしまい今の実装にしました。

拡大終了の手前あたりで回転を発火させる、という理解で良いでしょうか?

@osgsm

osgsm commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

@git-hub-tomosan
仰る通り、同時スタートではなく、タイミングをずらすイメージです!
「実際に手でカードを持ち上げて裏返して置く」動きに近づけられると自然になるかなと思ってます。

@osgsm osgsm merged commit a5ee4c9 into main Jun 26, 2026
1 check 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.

3 participants