Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

クーポン削除モーダルのメッセージ表示をEC-CUBE本体に合わせた #169

Merged
merged 2 commits into from
Aug 31, 2022

Conversation

ryo-endo
Copy link
Contributor

概要

クーポン削除時に表示されるモーダルのメッセージ内容とレイアウトが、EC-CUBE本体と統一できていなかったのを修正しました。

修正前

スクリーンショット 2022-08-27 23 50 17

修正後

image

参考:商品削除

image

@@ -95,13 +95,13 @@
<div class="modal-dialog" role="document">
<div class="modal-content">
<div class="modal-header">
<h5 class="modal-title font-weight-bold">
{{ 'plugin_coupon.admin.index.delete.confirm'|trans }}</h5>
<h5 class="modal-title fw-bold">
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bootstrap5対応漏れてたみたいです。

@@ -14,7 +14,8 @@ plugin_coupon.admin.index.col09: 有効期間
plugin_coupon.admin.index.col10: 編集
plugin_coupon.admin.index.col11: 状態
plugin_coupon.admin.index.col12: 削除
plugin_coupon.admin.index.delete.confirm: このクーポンを削除しても宜しいですか?
plugin_coupon.admin.index.delete__confirm_title: クーポンを削除します
plugin_coupon.admin.index.delete__confirm_message: クーポンを削除してよろしいですか?
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@chihiro-adachi chihiro-adachi added bugbounty2022:pr:entry バグバウンティ2022 バグ登録 PR登録 bugbounty2022:pr:recognition バグバウンティ2022 バグ登録 PR取込 labels Aug 31, 2022
@chihiro-adachi chihiro-adachi merged commit cbc03ac into EC-CUBE:4.2 Aug 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugbounty2022:pr:entry バグバウンティ2022 バグ登録 PR登録 bugbounty2022:pr:recognition バグバウンティ2022 バグ登録 PR取込
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants