Project

General

Profile

Bug(バグ) #2459

携帯版のメッセージ機能のページに罫線の配色が指定されていない部分がある

Added by Kiwa Sakai about 10 years ago. Updated over 9 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Target version:
Start date:
2011-10-03
Due date:
% Done:

100%

3.6 で発生するか:
Yes (はい)
[QA]バグ通知済:
No
3.8 で発生するか:
Unknown (未調査)

Description

Overview (現象)

携帯版のメッセージ機能のページに配色が指定されていない罫線がある。
携帯版配色設定「区切り罫線」の配色を反映してほしい。

Causes (原因)

Way to fix (修正内容)

Associated revisions

Revision 5b047e67 (diff)
Added by Maki Takahashi almost 10 years ago

(refs #2459) fixed configurations for mobile coloring (font color)

Revision d2289452 (diff)
Added by Maki Takahashi almost 10 years ago

(fixes #2459) fixed for cording standard

History

#1 Updated by Maki Takahashi almost 10 years ago

  • Status changed from New(新規) to Accepted(着手)
  • Assignee set to Maki Takahashi
  • Target version set to 1.0.0
  • 3.6 で発生するか set to Yes (はい)
  • 3.4 で発生するか set to Yes (はい)
  • 一覧画面(XX箱へのリンク上部の区切り罫線)
  • メッセージ詳細画面(メッセージ本文上下、XX箱へのリンク上部の区切り罫線)

#2 Updated by Maki Takahashi almost 10 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

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

#3 Updated by Yuya Watanabe almost 10 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

レビューを行いました.

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

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行目 など

#4 Updated by Maki Takahashi almost 10 years ago

  • Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)

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

#5 Updated by Yuya Watanabe almost 10 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

レビューしました.

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

#6 Updated by Yuma Sakata over 9 years ago

  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 70 to 100

テストOKです。

Also available in: Atom PDF