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

Opened 11 years ago

Closed 11 years ago

Last modified 10 years ago

#2668 closed defect (fixed)

BIZ:ToDoリストの重要度を表す左側の線がなくなっている

Reported by: imamura623 Owned by: ebihara
Priority: minor Milestone: OpenPNE2.12.3
Component: pne-xhtmlcss Version: 2.12.x & 2.14.x
Keywords: OpenPNE2.13.2 Cc:

Description (last modified by kudo)

■現象

2.10で表示されていた、BIZ:ToDoリストで重要度を表している左側の赤い縦線が、2.12、2.13でなくなっている。

■原因

テンプレート内で、2.10では重要度に応じてstyle属性の装飾が切り替わっていたが、2.12、2.13では重要度による分岐がなくなっていたため。

■修正内容

CSSに新たにクラスを定義し、重要度に応じて使用するクラスを切り替え、リストの左側に2.10と同様の赤い縦線を表示するよう修正する。

重要度:高の場合は赤い縦線が左側に表示される。

重要度:中の場合は薄い赤線が表示される。

重要度:低の場合は何も表示されない。

■関連情報

元のDescription

http://sns.openpne.jp/?m=pc&a=page_c_topic_detail&target_c_commu_topic_id=3654より転記

お世話になります。

BizモードにてToDoを入れるところで
重要度を「高」にしても、赤い帯がつかなくなりました。
2.10では、赤い帯がついて目立つようになっていましたが

バグでしょうか?

Attachments (1)

20080814_2668.png (47.5 KB) - added by kiwa 11 years ago.
先に再現画像です

Download all attachments as: .zip

Change History (32)

comment:1 Changed 11 years ago by imamura623

Type: enhancementdefect

comment:2 Changed 11 years ago by nakasone

Keywords: 再現待ち removed

以下のような確認を行ったところ再現できました。

■確認ブラウザ

  • FireFox2

■確認バージョン

  • 2.10.x
  • 2.12.x
  • trunk

■手順

  1. マイホーム→Todo作成
  2. Todo編集→重要度を「高」に設定
  3. マイホーム→Todo確認
  4. 2.10.xではTodoの左に赤い線が入っている、2.11.xとtrunkでは変化がない。

  

comment:3 Changed 11 years ago by ebihara

Priority: minormajor
Summary: 2.12.0にてBizモードBIZモードでToDoの重要度を変更しても、左側の線の色が変化しない

comment:4 Changed 11 years ago by ebihara

Summary: BIZモードでToDoの重要度を変更しても、左側の線の色が変化しないBIZ:ToDoリストの重要度を表す左側の線がなくなっている

comment:5 Changed 11 years ago by kiwa

Keywords: OpenPNE2.13.2 added
Milestone: OpenPNE2.12.2
Priority: majorminor

2.13.2に入れてほしいです

comment:6 Changed 11 years ago by ebihara

Milestone: OpenPNE2.12.2OpenPNE2.12.3

comment:7 Changed 11 years ago by masabon

Owner: changed from nobody to masabon
Status: newassigned

comment:8 Changed 11 years ago by masabon

Description: modified (diff)
Keywords: 確認中 added

以下のリビジョンで修正しました。

comment:9 Changed 11 years ago by kiwa

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

CSSに新たにクラスを定義し、重要度に応じて使用するクラスを切り替えるよう修正する。

この修正を行ったことでこのバグがどう改善されたのかがわかりづらいです。descriptionの修正をお願いします。

comment:10 Changed 11 years ago by masabon

Description: modified (diff)
Keywords: 確認中 added; 差し戻し removed

修正しました。 確認お願いします。

comment:11 Changed 11 years ago by ogawa

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

チェック画像の横位置が揃うようにしてください。

comment:12 Changed 11 years ago by masabon

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

チェック画像の横位置について以下のリビジョンで修正しました。

確認お願いします。

comment:13 Changed 11 years ago by ebihara

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

comment:14 Changed 11 years ago by kiwa

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

完了したタスクに重要度のラインが表示されません。確認お願いします。

comment:15 Changed 11 years ago by kiwa

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

すいません、早とちりでした。2.10でも完了したタスクには線が表示されていませんでした。

Changed 11 years ago by kiwa

Attachment: 20080814_2668.png added

先に再現画像です

comment:16 Changed 11 years ago by kiwa

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

先に再現画像です

ブラウザ毎に表示が違います。対応お願いします。いずれもWindowsXP(SP2)で動作確認しました。

!InternetExplorer6

  • 優先度低・未完了のタスクに黒い線が入る
  • 完了したチケットに灰色の線が入る

Safari3

  • 線の入らないタスクのチェックマークの位置が若干左にずれている

Firefox3, Opera9

  • タスク毎の区切り線(横線)の左端のほうが切れている

!InternetExplorer7, Firefox1.5

  • 特に問題なし。

comment:17 Changed 11 years ago by masabon

Owner: changed from masabon to takai
Status: assignednew

以下のリビジョンで修正しました。

Firefox3とOpera9のタスク毎の区切り線(横線)の左端のほうが切れているのは修正できませんでした。

→takaiさん どうしたらいいでしょう?

comment:18 in reply to:  16 Changed 11 years ago by ebihara

Replying to kiwa: 補足です。

Safari3

  • 線の入らないタスクのチェックマークの位置が若干左にずれている

については、テストに使用していたブラウザのキャッシュが残っていたことが原因のようです。

適切に最新の CSS を読み込んでいる状態では、 Windows 版、 Mac 版ともにこの現象は発生しませんでした。

comment:19 Changed 11 years ago by ogawa

Component: 指定しないpne-xhtmlcss

comment:20 Changed 11 years ago by takai

本件の問題の解消と併せて、カスタマイズ性を考慮したマークアップ及びスタイル記述を提示します。

【マークアップ】r7887(inc_biz_todo.tpl) のと変更点を示します。

22: ({if $value.memo})
23: <tr({if $value.writer_id != $target_id}) class="someone"({/if})>
24: <td class="({if $value.priority == 1})state1({elseif $value.priority == 2})state2({else})state({/if})"><a href="({t_url m=biz a=do_`$cmd`_home_check_biz_todo})&amp;sessid=({$PHPSESSID})&amp;chid=({$value.biz_todo_id})&amp;is_check=({$value.is_check})"><img src="./skin/default/img/biz/checkbox_nochecked_({if $value.c_member_id == 0})share_({/if})2.gif" alt="" /></a></td>
25: <td class="text">({strip})

40: ({if $value.memo})
41: <tr class="checked({if $value.writer_id != $target_id}) someone({/if})">
42: <td class="state">
43: <a href="({t_url m=biz a=do_`$cmd`_home_check_biz_todo})&amp;sessid=({$PHPSESSID})&amp;chid=({$value.biz_todo_id})&amp;is_check=({$value.is_check})"><img src="./skin/default/img/biz/checkbox_checked_({if !$value.member_id == 0})share_({/if})2.gif" alt="" /></a>
44: </td>
45: <td class="text">({strip})
  • 上記23~24行目を、下記23~26行目に変更
  • 上記41~44行目を、下記43~46行目に変更
22: ({if $value.memo})
23: <tr class="({if $value.priority == 1})priHigh({elseif $value.priority == 2})priMiddle({else})priLow({/if})({if $value.writer_id != $target_id}) someone({/if})">
24: <td class="state">
25: <p><a href="({t_url m=biz a=do_`$cmd`_home_check_biz_todo})&amp;sessid=({$PHPSESSID})&amp;chid=({$value.biz_todo_id})&amp;is_check=({$value.is_check})"><img src="./skin/default/img/biz/checkbox_nochecked_({if $value.c_member_id == 0})share_({/if})2.gif" alt="" /></a></p>
26: </td>
27: <td class="text">({strip})

42: ({if $value.memo})
43: <tr class="checked({if $value.priority == 1}) priHigh({elseif $value.priority == 2}) priMiddle({else}) priLow({/if})({if $value.writer_id != $target_id}) someone({/if})">
44: <td class="state">
45: <p><a href="({t_url m=biz a=do_`$cmd`_home_check_biz_todo})&amp;sessid=({$PHPSESSID})&amp;chid=({$value.biz_todo_id})&amp;is_check=({$value.is_check})"><img src="./skin/default/img/biz/checkbox_checked_({if !$value.member_id == 0})share_({/if})2.gif" alt="" /></a></p>
46: </td>
47: <td class="text">({strip})
  • 23行目:tr要素にpriorityのクラス名を振りました
  • 25行目:p要素で包含し、このp要素のborder-leftで本件の『左側の線』を表示しています
  • 24~26行目:1行に書かれていたため改行しました
  • 43行目:todoリストに合わせて、checkedリストも23行目と同様にしました
  • 45行目:25行目と同様です

【スタイル】xhtml_style.php

/*==============================================================================
 * 32. bizSideTodoList(BIZ用Todoリスト)
 *----------------------------------------------------------------------------*/
#Body .bizSideTodoList {
	width: 220px;
	margin: 10px 5px;
	border: 1px solid #<?php echo $colors[1]; ?>;
}
#Body .bizSideTodoList td {
	border-width: 1px 0 0;
}
.bizSideTodoList .block {
	padding: 5px;
}
.bizSideTodoList textarea {
	width: 98%;
}
.bizSideTodoList tr.someone {
	background: #<?php echo $colors[6]; ?>;
}
.bizSideTodoList tr.checked {
	color: #999999;
}
.bizSideTodoList td.state {
	width: 38px;
	vertical-align: top;
}
.bizSideTodoList tr.priLow td.state p {
	padding-left: 4px;
	border-left: 0;
}
.bizSideTodoList tr.priMiddle td.state p {
	padding-left: 0;
	border-left: 4px solid #ff8080;
}
.bizSideTodoList tr.priHigh td.state p {
	padding-left: 0;
	border-left: 4px solid #ff0000;
}
.bizSideTodoList tr.checked td.state p {
	padding-left: 4px;
	border-left: 0;
}
.bizSideTodoList td.operation {
	width: 16px;
	vertical-align: top;
}
.bizSideTodoList td.operation a {
	display: block;
	width: 14px;
	margin-top: 2px;
}

/*==============================================================================
 * 33. bizSideScheduleList(BIZ用スケジュールリスト) 
 *----------------------------------------------------------------------------*/
...

ソースをコメントで示してしまい申し訳ありませんが、この修正をお願いします。

comment:21 Changed 11 years ago by ebihara

Owner: changed from takai to ebihara
Status: newassigned

修正引き受けます。

comment:22 Changed 11 years ago by ebihara

以下のリビジョンで comment:20 の takai さんの修正を適用しました。

comment:23 Changed 11 years ago by takai

r8038, r8039共に comment:22 の修正が不十分です。

comment:20 で示した修正すべき点

  • 【マークアップ】r7887時点での23~24行目
  • 【マークアップ】r7887時点での41~44行目
  • 【スタイル記述】32. bizSideTodoList(BIZ用Todoリスト)部

comment:22 で修正された点

  • 【スタイル記述】32. bizSideTodoList(BIZ用Todoリスト)部

不十分な点

  • 【マークアップ】r7887時点での23~24行目
    • 修正後23行目:priLow と記述すべきところが priRow と記述されています
  • 【マークアップ】r7887時点での41~44行目
    • 修正されていません

再修正、あるいは修正が不要と思われるのであればその旨の説明をお願いします。

comment:24 in reply to:  23 ; Changed 11 years ago by ebihara

指摘事項についてコメントします。

  • 【マークアップ】r7887時点での23~24行目
    • 修正後23行目:priLow と記述すべきところが priRow と記述されています

これは誤りでした。修正します。

  • 【マークアップ】r7887時点での41~44行目
    • 修正されていません

こちらについてですが、チェック済み ToDo? リストについては、2.10 以前から重要度を表す線が表示されない実装になっているため、そちらにあわせました。

本件は「ToDoリストの重要度を表す左側の線がなくなっている」という現象ですので、 2.10 においてもともと線が表示されない箇所を線が表示されるように変更する理由はなく、取り込みをおこないませんでした。

comment:25 Changed 11 years ago by ebihara

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

priLow を priRow としてしまっていた箇所については、

で修正しました。

comment:26 in reply to:  24 ; Changed 11 years ago by takai

Replying to ebihara:

本件は「ToDoリストの重要度を表す左側の線がなくなっている」という現象ですので、 2.10 においてもともと線が表示されない箇所を線が表示されるように変更する理由はなく、取り込みをおこないませんでした。

comment:20 では!ToDoリストとCheckedリストのマークアップ修正を示していますが、線の表示といった見栄えに関してはCSSで制御しています。

comment:11 で仰られているように見栄えを統一させるためにはCheckedリストのマークアップ修正も必要になります。

修正後43行目に記述しているpriorityのクラス名についてですが、これはユーザ(オーナー)に対してのカスタマイズ性を考慮したものであって、comment:14 の内容を受けた修正ではありません。

43: <tr class="checked({if $value.priority == 1}) priHigh({elseif $value.priority == 2}) priMiddle({else}) priLow({/if})({if $value.writer_id != $target_id}) someone({/if})">
44: <td class="state">
45: <p><a href="({t_url m=biz a=do_`$cmd`_home_check_biz_todo})&amp;sessid=({$PHPSESSID})&amp;chid=({$value.biz_todo_id})&amp;is_check=({$value.is_check})"><img src="./skin/default/img/biz/checkbox_checked_({if !$value.member_id == 0})share_({/if})2.gif" alt="" /></a></p>
46: </td>

改めて、43~46行目(変更行は43, 45行目)の修正をお願いします。また、私に誤解があるようでしたらご指摘お願いします。

comment:27 Changed 11 years ago by ogawa

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

comment:28 in reply to:  26 Changed 11 years ago by ebihara

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

Replying to takai:

指摘事項について了解しました。早とちりしてしまいすいません。

以下のリビジョンで修正したのでご確認ください。

comment:29 Changed 11 years ago by ogawa

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

comment:30 Changed 11 years ago by kiwa

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

以下のブラウザで確認しました。問題ありません

確認ブラウザ

WindowsXP(SP2)

  • !InternetExplorer6
  • !InternetExplorer7
  • Firefox1.5
  • Firefox3
  • Opera9
  • Safari3

comment:31 Changed 10 years ago by kudo

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