Skip to content

fix #4276 キャッシュクリアのキャンセル時にツールバーの背景が白くなる問題を修正#4330

Open
teratai3 wants to merge 1 commit intobaserproject:5.2.xfrom
teratai3:dev-#4276
Open

fix #4276 キャッシュクリアのキャンセル時にツールバーの背景が白くなる問題を修正#4330
teratai3 wants to merge 1 commit intobaserproject:5.2.xfrom
teratai3:dev-#4276

Conversation

@teratai3
Copy link
Contributor

@teratai3 teratai3 commented Mar 18, 2026

#4276 の対応です。

修正内容

fixedMenu.js のクリックハンドラーに e.isDefaultPrevented() チェックを追加。

  • キャンセル時: inline onclick の return false により defaultPrevented = trueactive 付与をスキップ(白背景が出なくなる)
  • OK時: defaultPrevented = falseactive を付与してからページ遷移

事前準備

  • /baser/admin/baser-core/sites/indexにアクセスし新規追加からサイトを登録しておく
    (ToolMenuの動作にも影響するため)

MRで確認してほしいこと

#ToolMenu の動作

  • ヘッダーメニューのToolMenuをクリックしボタンの背景色が白くなることを確認する
  • メニューが開いた状態でどこかをクリックし、背景色の白が戻りメニューが折り畳まれることを確認する
  • 既存の動作から変わっていないことを確認する
マルチサイト

#UserMenu の動作

ヘッダーメニューのキャッシュクリアボタンを押しダイアログの以下の挙動を確認してください

OKの場合

  • キャッシュクリアが機能し背景が白の状態でリロードされることを確認する

キャンセルの場合

  • 背景色がキャンセル前と変わらないことを確認する

※ キャンセル時のフォーカスの背景色に関しては解除することもできますが、現状が自然な挙動と感じたため、本タスクでは修正していません。

備考

isDefaultPreventedメソッドのドキュメント

https://api.jquery.com/event.isDefaultPrevented/

onclick の return false により defaultPrevented が true になるケース(confirm ダイアログでキャンセル)を
e.isDefaultPrevented() で検出し、active クラスの付与をスキップするよう修正
@teratai3
Copy link
Contributor Author

4系も同じ問題があるため、こちらの修正内容が問題なくマージされたら4系も対応予定です。

@teratai3 teratai3 marked this pull request as ready for review March 18, 2026 13:05
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