プロジェクト

全般

プロフィール

Backport(バックポート) #3191

pc_backend の「画像投稿」ボタンの input 要素に不要な class 属性が付加されている

Yuya Watanabe6年以上前に追加. 6年以上前に更新.

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

100%


説明

問題点

pc_backend の input 要素に不要な class 属性値が付与されています。

/pc_backend.php/monitoring/editImage
の「画像投稿」ボタンの class 属性値が不要です。

apps/pc_backend/modules/monitoring/templates/editImageSuccess.php の 16行目

14-    <tr>
15-      <td colspan="2">
16:        <input class="input_sbumit" type="submit" value="<?php echo __('画像投稿') ?>" />
17-      </td>
18-    </tr>

問題となる理由

  • typo でもともと機能していない
  • typo を修正しても、ボタンのデザインに統一感が無くなる

加えて、class 属性値 input_submit の CSS は、/pc_backend.php/design/mobileColorConfig の「この設定を呼び出す」というボタンと同様のデザインになるものです。

しかし、他の管理画面の submit ボタン (角丸のボタン) 同様に、この class 属性値を適用させない方が統一感のある見た目になると思います。

修正方針

  • 他の submit ボタン (角丸のボタン)と同じデザインになるようにすることを前提とする
  • class 属性が付加されているが typo でもともと機能していないので、class を削除する

また、この class 属性値をソースコードで探したところ、先に記述した箇所以外は特に見つからなかったので、この部分のみ修正することをこのチケットで扱います。

2012/03/26 追記

このtypoは偶然見つけました。
修正したところ、管理画面の別のsubmitボタンと形が違いました。
では、他のボタンが何と違うのかを探った所、
今回問題になっている「画像投稿」ボタンだけ、.input_submit を適用させていることがわかりました。

因みに、pc_backend (プラグイン除く※note-4参照) の submit ボタンの中に .input_submit を適用させている箇所はありません。
(これは本チケットの"概要"でも「この class 属性値をソースコードで探したところ、先に記述した箇所以外は特に見つからなかったので、この部分のみ修正することをこのチケットで扱います。」と述べている通りです。)

当該箇所も、他の submit ボタンと同じく class 属性値を指定しないこと(class="input_sbumit"を記述しない)が適切であると判断したため、note-2 のような修正を行いました。

submit_bottom.png 表示 (8.34 KB) Fumie Toyooka, 2012-03-26 18:08

input_submit.png 表示 (9.1 KB) Fumie Toyooka, 2012-03-26 18:08


関連するチケット

関連している OpenPNE 3 - Enhancement(機能追加・改善) #2676: pc_backend の「画像投稿」ボタンの input 要素に不要な class 属性が付加されている Fixed(完了) 2011-12-08

関係しているリビジョン

リビジョン aeb1f2b0 (差分)
Fumie Toyooka6年以上前に追加

(fixes #3191, BP from #2676) remove class attribute of template of "/monitoring/editImage".

履歴

#1 Yuma Sakata6年以上前に更新

  • 対象バージョンOpenPNE 3.6.x から OpenPNE 3.6.6 に変更

#2 Yuya Watanabe6年以上前に更新

  • ステータスNew(新規) から Accepted(着手) に変更
  • 担当者Yuya Watanabe にセット

#3 Fumie Toyooka6年以上前に更新

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

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

#4 Kousuke Ebihara6年以上前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

#5 Yuma Sakata6年以上前に更新

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

テスト不要のため、クローズします。

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