Enhancement（機能追加・改善） #1456
完了
Add a link to backend-top on the pc_backend. （管理画面に「管理画面トップ」へのリンクを追加）
Minoru Takai さんが14年以上前に追加.
13年以上前に更新.
説明
管理画面においてセッションタイムアウトでセッション切れした場合に操作を続けると、ログイン画面に遷移するか、
ログインに失敗しました。
<a onclick="history.back(); return false;" href="#">前のページに戻る</a>
といった内容のコンテンツしかないページが表示される。このページでは画面遷移ができず行き止まりになってしまう。
また、行き止まりに関わらず、ユーザビリティ的にもヘッダ部の見出し部分は（トップページへの）リンクになっていると好ましい。
修正内容¶
管理画面のヘッダ部の見出し部分（h1要素）の内容をリンクにする。
一般的にリンク部分は、リンクであることが分かるようになっていることが好ましいが、ロゴ部分など特定箇所に於いてはその必要性は弱いと思われる。リンクであることが分かるようになっていることが好ましい理由についてはウェブコンテンツ・アクセシビリティ・ガイドライン（WCAG）等に示されているが、本件はこれを考慮すべきものでもない。
見出し部分について、リンクにしたついでに色などの見栄えを変更することも自由ではあるが、特に変更する理由がなく、また（リンクにしたことにより）下線が引かれると現状のデザイン（見栄え）に対して不自然に感じるため、この下線は表示されないようにしておく。
関連するチケット
2 (0件未完了 — 2件完了)
- ステータス を New（新規） から Accepted（着手） に変更
- 担当者 を Itsuro Tajima にセット
- ステータス を Accepted（着手） から Pending Review（レビュー待ち） に変更
- 進捗率 を 0 から 50 に変更
- ステータス を Pending Review（レビュー待ち） から 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;
}
- ステータス を Rejected（差し戻し） から Accepted（着手） に変更
了解しました。
#header h1 a {
color: #FFFFFF;
}
に変更してコミットしました。
- ステータス を Accepted（着手） から Pending Review（レビュー待ち） に変更
- ステータス を Pending Review（レビュー待ち） から Rejected（差し戻し） に変更
説明に修正内容の記述をお願いします。
また、他バージョンでこの問題が発生しないかの確認をお願いします。
- ステータス を Rejected（差し戻し） から Accepted（着手） に変更
- 担当者 を Itsuro Tajima から Minoru Takai に変更
- ステータス を Accepted（着手） から Pending Review（レビュー待ち） に変更
- ステータス を Pending Review（レビュー待ち） から Pending Testing（テスト待ち） に変更
- 進捗率 を 50 から 70 に変更
- 題名 を Add a link to backend-top on the pc_backend. (管理画面に「管理画面トップ」へのリンクを追加） から Add a link to backend-top on the pc_backend. （管理画面に「管理画面トップ」へのリンクを追加） に変更
- ステータス を Pending Testing（テスト待ち） から Fixed（完了） に変更
- 進捗率 を 70 から 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の修正（スタイル指定）についても特に問題はありません。また、修正全体について、コーディング規約の観点からの問題も特に見当たりません。
動作テストについて¶
管理画面の見出し部分がリンクになっており、想定通りの動作となっています。この修正は小規模なためこれ以上のテストも特に必要はないと判断しています。
チケットを完了にします。
他の形式にエクスポート: Atom
PDF
