プロジェクト

全般

プロフィール

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

Fumie Toyooka約12年前に更新

h3. 問題点

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

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

apps/pc_backend/modules/monitoring/templates/editImageSuccess.php の 16行目
<pre>
14- <tr>
15- <td colspan="2">
16: <input class="input_sbumit" type="submit" value="<?php echo __('画像投稿') ?>" />
17- </td>
18- </tr>
</pre>

h3. 問題となる理由

* typo でもともと機能していない
* typo を修正しても、ボタンのデザインに統一感が無くなる
typoでもともと機能していませんが、ここで付与されている class 属性は不要な記述であるためです。

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

しかし、他の管理画面の submit ボタン !submit_bottom.png! (角丸のボタン) 同様に、この class 属性値を適用させない方が統一感のある見た目になると思います。
(しかし、元々この class 属性値は typo されていたため、適用されていません。)


h3. 修正方針

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

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

h3. 2012/03/26 追記

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

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

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

戻る