ここの情報は古いです。ご理解頂いた上でお取り扱いください。

Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#3023 closed defect (fixed)

携帯版配色設定が正しく反映されない

Reported by: imamura623 Owned by: mori
Priority: minor Milestone: OpenPNE2.12.6
Component: pne-ktai Version: 2.12.x & 2.14.x
Keywords: OpenPNE2.13.4 Cc:

Description (last modified by kudo)

■現象

携帯版の配色設定が不正。結果としては同じ出力だが、フォントの色を指定するスタイル属性の「リスト見出し文字」 と 「リスト見出し背景」との対応がとれていない箇所がある。

■原因

HTMLコーディングのミス(「リスト見出し文字」 と 「リスト見出し背景」 の色指定の対応が取れていない箇所がある)。

■修正内容

今回の指摘のあった、「リスト見出し文字」 と 「リスト見出し背景」 (ktai_color_config.bg_08 など) に関して、2.10.x、2.12.x、trunk の全ファイルで対応を確認・修正。 デフォルトの設定では、ユーザにとっての変化はなし。

■関連情報

元のDescription

http://sns.openpne.jp/?m=pc&a=page_fh_diary&target_c_diary_id=19112より転記

OpenPNE v2.12.4

細かいですが、携帯版のマイホームのテンプレートファイルh_home.tplの168行目で「全体の最新日記」のフォントが、

<font color="#({$ktai_color_config.color_25})">
※リストA見出し文字

となっていますが、正しくは

<font color="#({$ktai_color_config.color_26})">
※リストB見出し文字

だと思います。

Change History (13)

comment:1 Changed 12 years ago by kiwa

Component: 指定しないpne-ktai
Keywords: 再現待ち removed
Version: 2.10.x & 2.12.x & 2.13.x2.12.x & 2.13.x

以下のバージョンで確認が取れました

version

  • 2.12.4
  • 2.13.3

※2.10には「全体の最新日記」が存在しない

場所

  • マイホーム(ktai_page_h_home/ktai(biz)_page_h_home)

内容

見出し「全体の最新日記」の文字色が「リストA見出し文字」になっている

※報告通り、他の見出しにあわせるならば「リストB見出し文字」にするべき


この現象については他のページについても検証を行うべきだと思います。(2.10含む)

comment:2 Changed 12 years ago by ebihara

Keywords: OpenPNE2.10.12 OpenPNE2.13.4 added
Milestone: OpenPNE2.12.6

comment:3 Changed 12 years ago by mori

Owner: changed from nobody to mori
Status: newassigned

担当します

comment:4 Changed 12 years ago by mori

Keywords: 確認中 added

調査修正方法

ktai_color_config.bg_08(リストB見出し背景)に対して ktai_color_config.color_26(リストB見出し文字)、 ktai_color_config.bg_05(リストA見出し背景)に対して ktai_color_config.color_25(リストA見出し文字)が使われているかを調査し、 対応付けがおかしいものを修正しました。

調査対象ファイルは以下

210

  • OpenPNE_DIR/webapp/modules/ktai/templates/c_home.tpl
  • OpenPNE_DIR/webapp/modules/ktai/templates/h_confirm_list.tpl
  • OpenPNE_DIR/webapp/modules/ktai/templates/h_home.tpl
  • OpenPNE_DIR/webapp/modules/ktai/templates/inc_f_home_h_prof.tpl
  • OpenPNE_DIR/webapp_biz/modules/ktai/templates/h_home.tpl
  • OpenPNE_DIR/webapp_biz/modules/ktai/templates/inc_f_home_h_prof.tpl

212

  • OpenPNE_DIR/webapp/modules/ktai/templates/c_home.tpl
  • OpenPNE_DIR/webapp/modules/ktai/templates/h_confirm_list.tpl
  • OpenPNE_DIR/webapp/modules/ktai/templates/h_home.tpl
  • OpenPNE_DIR/webapp/modules/ktai/templates/inc_f_home_h_prof.tpl
  • OpenPNE_DIR/webapp_biz/modules/ktai/templates/h_home.tpl
  • OpenPNE_DIR/webapp_biz/modules/ktai/templates/h_home.tpl
  • OpenPNE_DIR/webapp_biz/modules/ktai/templates/inc_f_home_h_prof.tpl
  • OpenPNE_DIR/webapp_biz/modules/ktai_biz/templates/fh_calendar_week.tpl

213

  • OpenPNE_DIR/webapp/modules/ktai/templates/c_home.tpl
  • OpenPNE_DIR/webapp/modules/ktai/templates/h_confirm_list.tpl
  • OpenPNE_DIR/webapp/modules/ktai/templates/h_home.tpl
  • OpenPNE_DIR/webapp/modules/ktai/templates/inc_f_home_h_prof.tpl
  • OpenPNE_DIR/webapp_biz/modules/ktai/templates/h_home.tpl
  • OpenPNE_DIR/webapp_biz/modules/ktai/templates/inc_f_home_h_prof.tpl

修正ファイルは下記リビジョンを参照して下さい。

  • 2.10.x - 該当なし
  • 2.12.x - r8763
  • trunk - r8769

comment:5 Changed 12 years ago by mori

Description: modified (diff)

comment:6 Changed 12 years ago by ebihara

Keywords: OpenPNE2.10.12 removed

以下の部分についても修正の必要があるのではないでしょうか。

  • webapp/modules/ktai/templates/h_home.tpl (2.12.x)
    167:<tr><td bgcolor="#({$ktai_color_config.bg_08})">
    168-<a accesskey="4" name="a4" href="#a4">[i:128]</a><font color="#({$ktai_color_config.color_25})">全体の最新({$WORD_DIARY_HALF})</font><br>
    169-</td></tr>
    

comment:7 Changed 12 years ago by ebihara

Keywords: 差し戻し added; 確認中 removed

comment:8 Changed 12 years ago by mori

Keywords: 確認中 added; 差し戻し removed

再修正しました。

comment:9 Changed 12 years ago by ebihara

Keywords: テスト待ち added; 確認中 removed

comment:10 Changed 12 years ago by kiwa

Keywords: 差し戻し added; テスト待ち removed

Descriptionについての指摘です。下記の修正をお願いします。

  1. 元のDescription(「■現象」の上にある内容)は、#2910のように「関連情報」に入れてください
  2. ■現象について、具体的に配色設定のどの辺が正しくなかったのかも書いておいてほしいです
    • 元のDescriptionにも現象の詳細は書いてありますが、「■現象」を読んだだけでどんなバグかを把握できるようにしたいので、お願いします
  3. ■修正内容について、ユーザ側から見てどのような対応になったのかも記載してほしいです
    • 例: #2910 「絞込みをすると1ページ目に移動するように修正。」

※ 今回#2910を例に挙げてますが、条件を満たしたものを適当に選んだだけなので「これが最良例」というわけではありません

comment:11 Changed 12 years ago by mori

Description: modified (diff)
Keywords: テスト待ち added; 差し戻し removed

comment:12 Changed 12 years ago by kiwa

Keywords: テスト待ち removed
Resolution: fixed
Status: assignedclosed

確認しました。問題ありません。

comment:13 Changed 12 years ago by kudo

Description: modified (diff)
Note: See TracTickets for help on using tickets.