OpenPNE3 Plugins » opMessagePlugin

• 一覧画面（XX箱へのリンク上部の区切り罫線）
• メッセージ詳細画面（メッセージ本文上下、XX箱へのリンク上部の区切り罫線）

e1d37133 にて対応いたしました。

レビューを行いました．

本チケット対象となる「罫線」とは異なりますが，下記エラー部分も「エラーメッセージ」の配色設定を反映したほうがいいのではないでしょうか？

apps/mobile_frontend/modules/message/templates/listSuccess.php 5行目

  4 <?php if ($form->hasGlobalErrors()): ?>
  5 <font color="#FF0000"><?php echo $form->renderGlobalErrors() ?></font>
  6 <?php endif; ?>

また，本チケットの直接の修正箇所とは違いますが，修正対象のファイル内下記部分でコーディング規約違反があります．ついでに直しておいたほうがいいかもしれません．

apps/mobile_frontend/modules/message/templates/listSuccess.php
15行目

apps/mobile_frontend/modules/message/templates/showSuccess.php
39行目 58行目 60行目 など

更新履歴 d2289452d8c5b6b57c06b0a80cc6efa0ce91f8e7 で適用されました。

レビューしました．

下記部分の変更は前回のレビュー時にも気がついていましたが，「返信」，「未開封」の色はデフォルトの配色設定では「リンク文字」や「エラーメッセージ」に対応するため修正は不要ではないかと思い，指摘内容には含めていませんでした．そのため，この修正が正しいかどうか判断に迷いましたが，この修正はデザインレベルの話であると判断したため実装者の判断を尊重してレビューはOKとします．

apps/mobile_frontend/modules/message/templates/_receiveListRecord.php

diff --git a/apps/mobile_frontend/modules/message/templates/_receiveListRecord.php b/apps/mobile_frontend/modules/message/templates/_receiveListRecord.php
index 29085d3..50d347a 100644
--- a/apps/mobile_frontend/modules/message/templates/_receiveListRecord.php
+++ b/apps/mobile_frontend/modules/message/templates/_receiveListRecord.php
@@ -1,11 +1,11 @@
 <?php use_helper('opMessage') ?>
 <?php echo op_format_date($message->getCreatedAt(), 'XDateTime') ?> 
 <?php if ($message->getIsHensin()): ?>
-<font color="#0000FF">(<?php echo __('Replied') ?>)</font>
+<font color="<?php echo $op_color["core_color_15"] ?>">(<?php echo __('Replied') ?>)</font>
 <?php elseif ($message->getIsRead()): ?>
 (<?php echo __('Open') ?>)
 <?php else: ?>
-<font color="#FF0000">(<?php echo __('Unopened') ?>)</font>
+<font color="<?php echo $op_color["core_color_22"] ?>">(<?php echo __('Unopened') ?>)</font>
 <?php endif; ?><br>
 <?php echo sprintf('%s (%s)',
   link_to(op_truncate($message->getSubject(), 28), '@readReceiveMessage?id='. $message->getMessageId()),

テストOKです。