admin2023: 閉じるボタンのクラスを close から btn-close に置き換える

Description

のコメントより。btn-close にすることで不自然に左に寄っていた×ボタンを右寄りに戻す。Bootstrap 5 では×を文字ではなくsvg表示するようになっているが、そのまま採用するといかにも大きすぎるので、ひとまず文字のままにしておく。

Checklists

None

Activity

Show:

Kenichi IshigakiNovember 13, 2023 at 2:39 AM

とともにQA、マージ済み。

Kenichi IshigakiNovember 9, 2023 at 2:40 AM

ありがとうございます。 の作業ブランチにマージしました。

Keita SugamaNovember 9, 2023 at 2:04 AM

をみて理解しましたので、↑の指摘は無視してください。

Keita SugamaNovember 9, 2023 at 1:53 AM

確認しました。

https://github.com/movabletype/movabletype-addons/blob/fix/btn_close/addons/Enterprise.pack/tmpl/admin2023/create_author_bulk_start.tmpl#L9

addonsのPRで、MT7とMT8とで修正ファイル数が違っているで気になったのですが、 MT8でだけ alert-dismiss が削除されたから「ひとまず無視」に該当したのですかね。問題なければよいと思います。

それ以外に気になる点はありませんでした。

Kenichi IshigakiNovember 8, 2023 at 7:55 AM

それぞれ以下で対応しました。alert-dismissible 関連以外の close はひとまず無視しています。念のためレビューおねがいします。

  • (MT7)

  • (MT8)

  • (MT7)

  • (MT8)

Implemented

Details

Assignee

Reporter

Fix versions

Priority

Smart Checklist

Created November 7, 2023 at 3:43 AM
Updated December 5, 2023 at 10:32 AM
Resolved November 13, 2023 at 2:39 AM