プロジェクト

全般

プロフィール

Bug(バグ) #2459

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

Kiwa Sakai12年以上前に追加. 約12年前に更新.

ステータス:
Fixed(完了)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
2011-10-03
期日:
進捗率:

100%

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

説明

Overview (現象)

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

Causes (原因)

Way to fix (修正内容)

関係しているリビジョン

リビジョン 5b047e67 (差分)
Maki Takahashi約12年前に追加

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

リビジョン d2289452 (差分)
Maki Takahashi約12年前に追加

(fixes #2459) fixed for cording standard

履歴

#1 Maki Takahashi約12年前に更新

  • ステータスNew(新規) から Accepted(着手) に変更
  • 担当者Maki Takahashi にセット
  • 対象バージョン1.0.0 にセット
  • 3.6 で発生するかYes (はい) にセット
  • 3.4 で発生するかYes (はい) にセット
  • 一覧画面(XX箱へのリンク上部の区切り罫線)
  • メッセージ詳細画面(メッセージ本文上下、XX箱へのリンク上部の区切り罫線)

#2 Maki Takahashi約12年前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

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

#3 Yuya Watanabe約12年前に更新

  • ステータスPending Review(レビュー待ち) から 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 Maki Takahashi約12年前に更新

  • ステータスRejected(差し戻し) から Pending Review(レビュー待ち) に変更

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

#5 Yuya Watanabe約12年前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 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 Yuma Sakata約12年前に更新

  • ステータスPending Testing(テスト待ち) から Fixed(完了) に変更
  • 進捗率70 から 100 に変更

テストOKです。

他の形式にエクスポート: Atom PDF