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

fix メールプラグインのメールフィールド 編集画面で、「注意書き」「説明文」の使い方がわかりにくい問題を解決 #1222

Open
wants to merge 1 commit into
base: dev-4
Choose a base branch
from

Conversation

katokaisya
Copy link
Collaborator

・後方互換も考慮し、入力画面のUIの調整のみで完結

View側の出力順と同じになるように、入力側の「説明文」「注意書き」の順番を入れ替え、
明示的に項目の目的を追記

説明文
※ 送信しない前見出し

注意書き
※ 送信しない後見出し

詳細については、helpを使用

@ryuring
Copy link
Collaborator

ryuring commented Aug 18, 2019

@katokaisya この件、やりたい事はわかりますが、後に置くのか、前に置くのかは、デフォルトのテンプレートに依存するだけで、組み方によってはどちらにも置けるんですよね。
組み方が逆になった場合に今度また矛盾が発生してしまいます。
少し話し合って決めなければなりませんが、一旦、その前提にてコードをレビューしますね。

Copy link
Collaborator

@ryuring ryuring left a 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>
Copy link
Collaborator

Choose a reason for hiding this comment

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

@katokaisya 説明文を前に置くか後ろに置くかはテンプレートの組み方次第なので記述するのは難しいかと思います。

Copy link
Collaborator

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', 'ヘルプ'))) ?>
Copy link
Collaborator

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>
Copy link
Collaborator

Choose a reason for hiding this comment

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

@katokaisya 「姓・名など」では、ちょっと意味が伝わりづらいかもしれません。
「この項目について、標準では送信メールには表示しません。」といった感じでしょうか。

@ryuring ryuring added the Review レビュー要 label Sep 10, 2019
@gondoh gondoh added this to the 4.2.3 milestone Sep 10, 2019
@gondoh gondoh modified the milestones: 4.2.3, 4.2.4 Sep 20, 2019
@gondoh gondoh modified the milestones: 4.2.4, 4.2.5 Oct 8, 2019
@ryuring ryuring self-assigned this Oct 31, 2019
@ryuring ryuring modified the milestones: 4.2.5, 4.3.0 Nov 20, 2019
@ryuring
Copy link
Collaborator

ryuring commented Nov 20, 2019

@katokaisya こちらの件、一度、一緒に時間作って考えたいですね。一旦マイルストーンを更新します。

<tr id="RowAttention">
<th class="col-head">
<?php echo $this->BcForm->label('MailField.attention', __d('baser', '注意書き')) ?>
<br><small>※ 送信しない後見出し</small>
Copy link
Collaborator

Choose a reason for hiding this comment

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

「※ 送信メールには表示されません」に変更

@gondoh
Copy link
Collaborator

gondoh commented Dec 10, 2019

フィールド管理の項目表示順を下記に変更し、helpでメールに含まれない説明を入れるよう変更おねがいできますでしょうか。
前見出し
後見出し
説明文
注意書き

@ryuring
Copy link
Collaborator

ryuring commented Dec 10, 2019

admin-third にも変更が必要

@gondoh gondoh modified the milestones: 4.3.0, 4.3.1 Jan 7, 2020
@ryuring ryuring modified the milestones: 4.3.1, 4.3.2 Jan 29, 2020
@ryuring ryuring modified the milestones: 4.3.2, 4.3.3 Feb 21, 2020
@ryuring ryuring modified the milestones: 4.3.3, 4.3.4 Mar 27, 2020
@ryuring ryuring modified the milestones: 4.3.4, 4.3.5 Apr 24, 2020
@ryuring ryuring modified the milestones: 4.3.5, 4.3.6 May 22, 2020
@ryuring ryuring removed this from the 4.3.6 milestone Jul 9, 2020
@ryuring ryuring added the Adjusting 調整中 label Jul 10, 2020
@gondoh gondoh added this to the Adjusting milestone Aug 6, 2020
@ryuring ryuring added the Ver4 バージョン4 label Apr 14, 2023
@ryuring ryuring assigned katokaisya and unassigned ryuring Oct 9, 2023
@ryuring
Copy link
Collaborator

ryuring commented Oct 9, 2023

@katokaisya だいぶ前のプルリクではありますが、一旦、レビューは入れてますので、ご確認ください。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Adjusting 調整中 Review レビュー要 Ver4 バージョン4
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants