Project

General

Profile

Backport(バックポート) #3191

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

Added by Yuya Watanabe almost 7 years ago. Updated almost 7 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Assignee:
Target version:
Start date:
2011-12-08
Due date:
% Done:

100%


Description

問題点

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 View (8.34 KB) Fumie Toyooka, 2012-03-26 18:08

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


Related issues

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

Associated revisions

Revision aeb1f2b0 (diff)
Added by Fumie Toyooka almost 7 years ago

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

History

#1 Updated by Yuma Sakata almost 7 years ago

  • Target version changed from OpenPNE 3.6.x to OpenPNE 3.6.6

#2 Updated by Yuya Watanabe almost 7 years ago

  • Status changed from New(新規) to Accepted(着手)
  • Assignee set to Yuya Watanabe

#3 Updated by Fumie Toyooka almost 7 years ago

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

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

#4 Updated by Kousuke Ebihara almost 7 years ago

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

#5 Updated by Yuma Sakata almost 7 years ago

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

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

Also available in: Atom PDF