Skip to content

fix(StepFormDialog): onPressEscape時にも状態をリセットするように修正#6182

Open
Qs-F wants to merge 1 commit into
masterfrom
fix/stepformdialog/reset-on-press-escape
Open

fix(StepFormDialog): onPressEscape時にも状態をリセットするように修正#6182
Qs-F wants to merge 1 commit into
masterfrom
fix/stepformdialog/reset-on-press-escape

Conversation

@Qs-F
Copy link
Copy Markdown
Contributor

@Qs-F Qs-F commented Mar 12, 2026

関連URL

概要

  • onPressEscape時にダイアログがリセットされず、2ステップ目から開かれるなどのバグがあったためリセットされるように修正

変更内容

  • ダイアログがアニメーション終了して閉じた後にリセットするような処理が入っていたので、共通部分に切り出して修正

プロダクト側で対応が必要な事項

ない

確認方法

https://63d0ccabb5d2dd29825524ab-vzhllwcokq.chromatic.com/?path=/story/components-dialog-stepformdialog--async-submit-success

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Mar 12, 2026

Open in StackBlitz

npm i https://pkg.pr.new/@smarthr/smarthr-ui-charts@6182
npm i https://pkg.pr.new/smarthr-ui@6182

commit: 094b9c5

@Qs-F Qs-F marked this pull request as ready for review March 12, 2026 05:21
@Qs-F Qs-F requested a review from a team as a code owner March 12, 2026 05:21
@Qs-F Qs-F requested review from oti and uknmr and removed request for a team March 12, 2026 05:21
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.

1 participant