プロジェクト

全般

プロフィール

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. 問題となる理由 

 * class 属性が付加されているがtypoでもともと機能していないので、不要な class 属性が記述されているからです。 

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

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

 属性値を適用させない方が適切な動作と考えられるためです。 


 h3. 修正方針 

 * 他の submit ボタンと同じデザインになるようにすることを前提とする 
 !submit_bottom.png! 
 * 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 のような修正を行いました。

戻る