Skip to:
8.1.0以降で再現する。8.0.xと7.xでは再現しない。
AdminThemeId '', AdminThemeId admin2023どちらでも再現する。
AdminThemeId ''
AdminThemeId admin2023
元々この辺り の処理が動いていなかったようだが、 の改修で動くようになり、モーダルが二重で開くようになったと思われる。
JavaScriptのモーダルを開く処理部分を削除して対応する。
ご確認ありがとうございました。
マージしたのでクローズします。
AdminThemeId admin2023 と AdminThemeId ''で、モーダルが2つ開かないこと、適用後保存ボタン1クリックでモーダルが閉じる事を確認。
QAをお願いいたします。
下記の現象を修正して、1つだけモーダルが開くようにしました。
アセットの編集画面から「画像を編集」ボタンを押下すると、モーダルが2つ開く
AdminThemeId admin2023 と AdminThemeId '' の両方で修正しています。
修正前は開いたモーダルを閉じる作業が2回あったのですが、修正後はこれが1回になっています。
よろしくお願いいたします。
ありがとうございます。よさそうです。
コードレビューをお願いいたします。
https://github.com/movabletype/movabletype/pull/3343
問題の箇所にはモーダルを開く処理が2つ設定されていて、今回はJavaScriptの方を削除しました。
もう1つはHTMLタグの属性値によるもので（class="mt-modal-open", data-mt-modal-large）、削除すると影響があるかもしれないので、残すことにしました。
class="mt-modal-open"
data-mt-modal-large
8.1.0以降で再現する。8.0.xと7.xでは再現しない。
AdminThemeId '',AdminThemeId admin2023どちらでも再現する。元々この辺り の処理が動いていなかったようだが、 の改修で動くようになり、モーダルが二重で開くようになったと思われる。
JavaScriptのモーダルを開く処理部分を削除して対応する。