Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

コンポーネントのStoryの調整・不足分の追加 #2333

Merged
merged 17 commits into from
Oct 28, 2024

Conversation

takusea
Copy link
Contributor

@takusea takusea commented Oct 27, 2024

内容

コンポーネントのうちStoryが存在しないものへの追加、クリックが働かなかったものを動作させるための変更、グリッドサイズの調整を行います。

  • グリッド表示のボタンを押した際の背景のグリッドのサイズをBaseコンポーネント群の基底サイズである8pxに変更
    • image
  • BaseCheckboxとBaseSwitchのStoryでクリックした際にチェック・非チェックが切り替わるように修正
  • BaseSelect、BaseTooltip、AcceptDialogのStoryファイルを追加

@takusea takusea requested a review from a team as a code owner October 27, 2024 14:45
@takusea takusea requested review from Hiroshiba and removed request for a team October 27, 2024 14:45
@Hiroshiba
Copy link
Member

Hiroshiba commented Oct 27, 2024

PRありがとうございます!!

Storybookはスナップショットが行われるようになりました!
[update snapshots]というメッセージを含めてcommitすると自動的に更新されるので、もしよければ 🙏
https://github.com/VOICEVOX/voicevox#github-actions-%E3%81%A7%E6%9B%B4%E6%96%B0%E3%81%99%E3%82%8B%E5%A0%B4%E5%90%88

今回はちょっとこっちで空コミットさせていただきます!

Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

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

ほぼLGTMです!!

Storybookの使い勝手向上と追加、ありがとうございます!!

気になった点をコメントしてみました!!
renderを共通化する方はこちらで試してみたのをPR作ってみたので、もしよければ 🙏

@takusea
Copy link
Contributor Author

takusea commented Oct 28, 2024

とりあえずいただいたPRをマージして手直ししてみます!

Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

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

LGTM!!

テストが通り次第マージさせていただきます!!

Storybookはまだまだ不慣れで正直手探り状態です 🙇
どんどん改良していきたいですね・・・!!

@Hiroshiba Hiroshiba merged commit 5708236 into VOICEVOX:main Oct 28, 2024
8 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