Skip to content

LinkButton の Design Doc を追加#1726

Merged
tatz012 merged 6 commits intomainfrom
codex/create-design-document-for-linkbutton
Apr 1, 2026
Merged

LinkButton の Design Doc を追加#1726
tatz012 merged 6 commits intomainfrom
codex/create-design-document-for-linkbutton

Conversation

@tatz012
Copy link
Copy Markdown
Contributor

@tatz012 tatz012 commented Dec 19, 2025

Summary

  • LinkButton コンポーネントの概要・使用例・要素を整理した Design Doc を追加
  • CSSカスタムプロパティとテスト方針を明文化し、関連ドキュメントへのリンクを掲載

Testing

  • pnpm textlint packages/spindle-ui/src/LinkButton/design-doc.md

Codex Task

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Dec 19, 2025

⚠️ No Changeset found

Latest commit: b8d523d

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 19, 2025

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
Copy link
Copy Markdown

reg-suit bot commented Dec 19, 2025

reg-suit detected visual differences.

Check this report, and review them.

⚪ New
258
How can I change the check status? If reviewers approve this PR, the reg context status will be green automatically.


## テスト方針

### ユニットテスト
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

自動で生成されていなくてあれれ?なんですがテスト項目記載されているので、テストファイル作成したいです!


- `href`、`target`、`rel`などアンカー属性が正しく受け渡される
- `ref`が転送される
- `icon`有無と`iconPosition`の組み合わせでアイコンが正しく配置される
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

こっちはstorybookで担保してもいいかも


LinkButtonは、リンクとして機能しつつボタンと同等の視覚的強調を与えるためのコンポーネントです。フォーム送信ではなくページ遷移や別画面への導線を設定したいが、通常のリンクよりも強い注目を集めたい場面で使用します。ボタンと同様のバリエーション・サイズを備えており、ページの主要動線から補助動線まで一貫したトーンで表現できます。

## 使用例
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Buttonコンポーネントを見るとインタラクション的なセクションあってホバーやフォーカスに対して言及してるのでLinkButtonもあってもいいかなって思いました。

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Buttonと合わせてっていう指示でもいいかもですね


## 使用例

### DO
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

これもButtonと同様にサイズだったりアイコンに対する余白とかもうちょっと解像度高く記載できそうな気がしました。
ButtonのDesign Docを参考にもうちょいアップデートしてもよさそう。

@yasuda-shin
Copy link
Copy Markdown
Contributor

テストファイルとは別なんですがついでにCodeConnectってなんでもかんでも追加するもんでもないんですっけ?
が、ちょっと気になりました。

@herablog
Copy link
Copy Markdown
Member

テストファイルとは別なんですがついでにCodeConnectってなんでもかんでも追加するもんでもないんですっけ? が、ちょっと気になりました。

対応するFigmaファイルがないので作るとこからなんですよね・・! (このPRでは関与しなくていいかも)

@tatz012 tatz012 force-pushed the codex/create-design-document-for-linkbutton branch from b063be7 to b8d523d Compare March 31, 2026 09:45
@tatz012
Copy link
Copy Markdown
Contributor Author

tatz012 commented Mar 31, 2026

@herablog @yasuda-shin
だいぶ時間が空いてしまったのですが、ご指摘いただいた点codexに修正してもらいました🙇

@tatz012 tatz012 merged commit c05e833 into main Apr 1, 2026
11 checks passed
@tatz012 tatz012 deleted the codex/create-design-document-for-linkbutton branch April 1, 2026 02:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants