OpenPNE 3

問題点¶

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 のような修正を行いました。