Skip to content

バリデーショントーストの表示を制御できるようにする - #16

Merged
RyoSogawa merged 1 commit into
mainfrom
feat/optional-validation-toast
Aug 9, 2026
Merged

RyoSogawa merged 1 commit into
mainfrom
feat/optional-validation-toast

Conversation

@kanchi-Matsumoto

@kanchi-Matsumoto kanchi-Matsumoto commented Aug 8, 2026 •

Copy link
Copy Markdown
Contributor

概要

利用側で独自のエラー表示を行う場合に、ライブラリ内蔵のバリデーショントーストを非表示にできるようにします。

SpreadSheetTableにshowValidationToastプロパティを追加し、falseを指定した場合は内蔵トーストだけを非表示にします。
トーストを非表示にしても、onRowChangeRejectedによる拒否理由の通知は継続します。

確認用画面名/URL

確認用画面なし(公開APIとコンポーネントテストの変更)

デザイン、仕様等の資料

  • openspec/changes/add-validation-toast-control/

動作/表示確認済み

  • Chrome
  • Safari
  • Firefox
  • iOS Safari

確認用画面がないため、ブラウザでの手動確認は実施していません。
既定表示、トーストの非表示、拒否通知の継続、非表示中のエラーが再表示されないことをコンポーネントテストで確認しています。

実装内容についての補足

  • showValidationToastの既定値はtrueとし、既存の表示動作を維持します。
  • showValidationToast={false}の場合は内蔵トーストを非表示にします。
  • 非表示中のトーストメッセージを破棄し、再度有効にした際に古いエラーが表示されないようにします。
  • 公開型とAPIリファレンスを更新します。
<SpreadSheetTable table={table} showValidationToast={false} />

このブランチで対応しないこと

  • 利用側アプリへのshowValidationToast={false}の指定
  • npmへの新バージョン公開

迷ったこと/相談したいこと

なし

その他の変更内容

  • OpenSpecを追加
  • バリデーショントーストのコンポーネントテストを追加

Test plan

  • pnpm typecheck
  • pnpm test(218 tests)
  • pnpm lint
  • pnpm build

PR作成後確認事項

  • マージ先がmainになっている
  • レビュアーを設定する
  • アサイニーを設定する
  • Development(Linked Issue)を設定する

@kanchi-Matsumoto
kanchi-Matsumoto marked this pull request as ready for review August 8, 2026 15:41

@RyoSogawa RyoSogawa left a comment

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.

LGTM!

@RyoSogawa
RyoSogawa merged commit 0c528a5 into main Aug 9, 2026
5 checks passed
@RyoSogawa
RyoSogawa deleted the feat/optional-validation-toast branch August 9, 2026 12:26
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