Conversation
|
|
Visit the preview URL for this PR (updated for commit b8d523d): https://ameba-spindle--pr1726-codex-create-design-1f6dq62p.web.app (expires Thu, 30 Apr 2026 09:47:26 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: e7521619a2dd5c653490c8246e81ec2a5c8f1435 |
|
reg-suit detected visual differences. Check this report, and review them.
How can I change the check status?If reviewers approve this PR, the reg context status will be green automatically. |
|
|
||
| ## テスト方針 | ||
|
|
||
| ### ユニットテスト |
There was a problem hiding this comment.
自動で生成されていなくてあれれ?なんですがテスト項目記載されているので、テストファイル作成したいです!
|
|
||
| - `href`、`target`、`rel`などアンカー属性が正しく受け渡される | ||
| - `ref`が転送される | ||
| - `icon`有無と`iconPosition`の組み合わせでアイコンが正しく配置される |
|
|
||
| LinkButtonは、リンクとして機能しつつボタンと同等の視覚的強調を与えるためのコンポーネントです。フォーム送信ではなくページ遷移や別画面への導線を設定したいが、通常のリンクよりも強い注目を集めたい場面で使用します。ボタンと同様のバリエーション・サイズを備えており、ページの主要動線から補助動線まで一貫したトーンで表現できます。 | ||
|
|
||
| ## 使用例 |
There was a problem hiding this comment.
Buttonコンポーネントを見るとインタラクション的なセクションあってホバーやフォーカスに対して言及してるのでLinkButtonもあってもいいかなって思いました。
|
|
||
| ## 使用例 | ||
|
|
||
| ### DO |
There was a problem hiding this comment.
これもButtonと同様にサイズだったりアイコンに対する余白とかもうちょっと解像度高く記載できそうな気がしました。
ButtonのDesign Docを参考にもうちょいアップデートしてもよさそう。
|
テストファイルとは別なんですがついでにCodeConnectってなんでもかんでも追加するもんでもないんですっけ? |
対応するFigmaファイルがないので作るとこからなんですよね・・! (このPRでは関与しなくていいかも) |
b063be7 to
b8d523d
Compare
|
@herablog @yasuda-shin |
Summary
Testing
Codex Task