-
Notifications
You must be signed in to change notification settings - Fork 132
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
fix メールプラグインのメールフィールド 編集画面で、「注意書き」「説明文」の使い方がわかりにくい問題を解決 #1222
base: dev-4
Are you sure you want to change the base?
Conversation
・後方互換も考慮し、入力画面のUIの調整のみで完結
@katokaisya この件、やりたい事はわかりますが、後に置くのか、前に置くのかは、デフォルトのテンプレートに依存するだけで、組み方によってはどちらにも置けるんですよね。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@katokaisya レビュー入れてみました。一度確認お願いします。
<tr id="RowDescription"> | ||
<th class="col-head"> | ||
<?php echo $this->BcForm->label('MailField.description', __d('baser', '説明文')) ?> | ||
<br><small>※ 送信しない前見出し</small> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@katokaisya 説明文を前に置くか後ろに置くかはテンプレートの組み方次第なので記述するのは難しいかと思います。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@katokaisya 送信しないかどうかもテンプレートの組み方次第なんですよね
<th class="col-head"> | ||
<?php echo $this->BcForm->label('MailField.description', __d('baser', '説明文')) ?> | ||
<br><small>※ 送信しない前見出し</small> | ||
<?php echo $this->BcHtml->image('admin/icn_help.png', array('id' => 'helpDescription', 'class' => 'btn help', 'alt' => __d('baser', 'ヘルプ'))) ?> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@katokaisya ヘルプは、 th
ではなく、 td
に配置するように統一されていますので td
側に置くようにしてください。
<?php echo $this->BcForm->label('MailField.description', __d('baser', '説明文')) ?> | ||
<br><small>※ 送信しない前見出し</small> | ||
<?php echo $this->BcHtml->image('admin/icn_help.png', array('id' => 'helpDescription', 'class' => 'btn help', 'alt' => __d('baser', 'ヘルプ'))) ?> | ||
<div id="helptextDescription" class="helptext"> <?php echo __d('baser', '姓・名などメール本文に入れない内容を入力してください。')?> </div> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@katokaisya 「姓・名など」では、ちょっと意味が伝わりづらいかもしれません。
「この項目について、標準では送信メールには表示しません。」といった感じでしょうか。
@katokaisya こちらの件、一度、一緒に時間作って考えたいですね。一旦マイルストーンを更新します。 |
<tr id="RowAttention"> | ||
<th class="col-head"> | ||
<?php echo $this->BcForm->label('MailField.attention', __d('baser', '注意書き')) ?> | ||
<br><small>※ 送信しない後見出し</small> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
「※ 送信メールには表示されません」に変更
フィールド管理の項目表示順を下記に変更し、helpでメールに含まれない説明を入れるよう変更おねがいできますでしょうか。 |
admin-third にも変更が必要 |
@katokaisya だいぶ前のプルリクではありますが、一旦、レビューは入れてますので、ご確認ください。 |
・後方互換も考慮し、入力画面のUIの調整のみで完結
View側の出力順と同じになるように、入力側の「説明文」「注意書き」の順番を入れ替え、
明示的に項目の目的を追記
詳細については、helpを使用