Project

General

Profile

Enhancement(機能追加・改善) #1456

Add a link to backend-top on the pc_backend. (管理画面に「管理画面トップ」へのリンクを追加)

Added by Minoru Takai over 9 years ago. Updated over 8 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Assignee:
Target version:
Start date:
2010-07-30
Due date:
% Done:

100%


Description

概要

管理画面においてセッションタイムアウトでセッション切れした場合に操作を続けると、ログイン画面に遷移するか、

ログインに失敗しました。
<a onclick="history.back(); return false;" href="#">前のページに戻る</a>

といった内容のコンテンツしかないページが表示される。このページでは画面遷移ができず行き止まりになってしまう。

また、行き止まりに関わらず、ユーザビリティ的にもヘッダ部の見出し部分は(トップページへの)リンクになっていると好ましい。

修正内容

管理画面のヘッダ部の見出し部分(h1要素)の内容をリンクにする。

補足

一般的にリンク部分は、リンクであることが分かるようになっていることが好ましいが、ロゴ部分など特定箇所に於いてはその必要性は弱いと思われる。リンクであることが分かるようになっていることが好ましい理由についてはウェブコンテンツ・アクセシビリティ・ガイドライン(WCAG)等に示されているが、本件はこれを考慮すべきものでもない。

見出し部分について、リンクにしたついでに色などの見栄えを変更することも自由ではあるが、特に変更する理由がなく、また(リンクにしたことにより)下線が引かれると現状のデザイン(見栄え)に対して不自然に感じるため、この下線は表示されないようにしておく。


Related issues

Related to OpenPNE 3 - Enhancement(機能追加・改善) #1080: Add a button to view site on the pc_backend. (管理画面に「サイトを確認」ボタンを追加) Fixed(完了) 2010-05-17
Related to OpenPNE 3 - Backport(バックポート) #2258: Add a link to backend-top on the pc_backend. (管理画面に「管理画面トップ」へのリンクを追加) Fixed(完了) 2011-07-04

Associated revisions

Revision 3c854f48 (diff)
Added by Itsuro Tajima about 9 years ago

(fixes #1456) added link to pc_backend header title

Revision 08493c15 (diff)
Added by Itsuro Tajima about 9 years ago

(fixes #1456) fixed css selector

Revision 2fe23c80 (diff)
Added by Minoru Takai over 8 years ago

(fixes #1456) removed the underline for a looks

History

#1 Updated by Itsuro Tajima about 9 years ago

  • Status changed from New(新規) to Accepted(着手)
  • Assignee set to Itsuro Tajima

#2 Updated by Itsuro Tajima about 9 years ago

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

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

#3 Updated by Minoru Takai about 9 years ago

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

HTML/CSS改善チケットへの着手ありがとうございます。

#header h1 a:link, a:visited, a:hover, a:active {
  color: #FFFFFF;
}

このセレクタ部の記述は適切ではありません。カンマで区切られるそれぞれのセレクタに対して対象となる主体を指すパターンを記述しなければ意図する結果になりません。つまり次のように #header h1 をそれぞれのセレクタに記述する必要があります(改行の有無は関係ありません)。

#header h1 a:link,
#header h1 a:visited,
#header h1 a:hover,
#header h1 a:active {
  color: #FFFFFF;
}

ただし、ここで記述しているセレクタグループのセレクタは全てキーセレクタ(主体となる要素)が同一(a 要素そのもの)です。つまり、以下のように記述した方がよいかもしれません。

#header h1 a {
  color: #FFFFFF;
}

#4 Updated by Itsuro Tajima about 9 years ago

  • Status changed from Rejected(差し戻し) to Accepted(着手)

了解しました。

#header h1 a {
  color: #FFFFFF;
}

に変更してコミットしました。

#5 Updated by Itsuro Tajima about 9 years ago

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

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

#6 Updated by Masato Nagasawa over 8 years ago

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

説明に修正内容の記述をお願いします。
また、他バージョンでこの問題が発生しないかの確認をお願いします。

#7 Updated by Minoru Takai over 8 years ago

  • Status changed from Rejected(差し戻し) to Accepted(着手)
  • Assignee changed from Itsuro Tajima to Minoru Takai

引き継ぎます。

#8 Updated by Minoru Takai over 8 years ago

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

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

#9 Updated by Minoru Takai over 8 years ago

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

#10 Updated by Minoru Takai over 8 years ago

  • Subject changed from Add a link to backend-top on the pc_backend. (管理画面に「管理画面トップ」へのリンクを追加) to Add a link to backend-top on the pc_backend. (管理画面に「管理画面トップ」へのリンクを追加)

#11 Updated by Minoru Takai over 8 years ago

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

コードチェックおよび動作テストを行いました。

修正内容について

  • apps/pc_backend/templates/layout.php
     <div id="header">
    -<h1><?php echo __('%sns% Administration', array('%sns%' => $op_config['sns_name'])) ?></h1>
    +<h1><?php echo link_to(__('%sns% Administration', array('%sns%' => $op_config['sns_name'])), '@homepage') ?></h1>
     <span id="visit-site"><a href="<?php echo app_url_for('pc_frontend', '@homepage', true) ?>" target="_blank"><?php echo __('Visit Site') ?></a></span>
     </div>
    

リンク先URLをルーティング定義されている @homepage を用いて生成しており、環境に依存する不適切なURLの書き方ではないため、(この対応でもっとも重要な)リンク先については適切な修正だと判断しています。

CSSの修正(スタイル指定)についても特に問題はありません。また、修正全体について、コーディング規約の観点からの問題も特に見当たりません。

動作テストについて

管理画面の見出し部分がリンクになっており、想定通りの動作となっています。この修正は小規模なためこれ以上のテストも特に必要はないと判断しています。

チケットを完了にします。

Also available in: Atom PDF