プロジェクト

全般

プロフィール

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

完了

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

Fumie Toyooka さんがほぼ13年前に追加. 12年以上前に更新.

ステータス:
Fixed(完了)
優先度:
Urgent(急いで)
担当者:
対象バージョン:
開始日:
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) submit_bottom.png Fumie Toyooka, 2012-03-26 18:08
input_submit.png (9.1 KB) input_submit.png Fumie Toyooka, 2012-03-26 18:08

関連するチケット 1 (0件未完了1件完了)

関連している OpenPNE 3 - Backport(バックポート) #3191: pc_backend の「画像投稿」ボタンの input 要素に不要な class 属性が付加されているFixed(完了)Yuya Watanabe2011-12-08

操作

Fumie Toyooka さんがほぼ13年前に更新

  • ステータスNew(新規) から Accepted(着手) に変更

すぐやります。

Fumie Toyooka さんがほぼ13年前に更新

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

https://github.com/openpne/OpenPNE3/pull/27
5a94a5fb36cc3a9b0c6d324b920672fe00cfa555

修正をプルリクエストしました。ご確認お願いいたします。

Fumie Toyooka さんがほぼ13年前に更新

Fumie Toyooka さんがほぼ13年前に更新

コア側だけでなく、プラグイン側にも同様の適切でない class 属性値が含まれています。
(2011/12/12現在の調査結果)
  • opDiaryPlugin
  • opAlbumPlugin

Shouta Kashiwagi さんが12年以上前に更新

  • 対象バージョンOpenPNE 3.7.0 から 252 に変更

Youichi Kimura さんが12年以上前に更新

  • 優先度Normal(通常) から Urgent(急いで) に変更

Yuya Watanabe さんが12年以上前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更

class 属性の値が typo であるについては一切触れられていませんが,これは考慮済みということで大丈夫なんでしょうか?

input_submit が正しく記述されていた場合に「このボタンのデザインが適切でない」という内容のチケットであるということは読み取れましたが,問題として挙げられている「適切でない」という点について正しい修正かどうかに不安を感じたためいったん差し戻します.

Fumie Toyooka さんが12年以上前に更新

  • 題名pc_backend の input 要素に付いている class 属性に適切でないものがある から pc_backend の「画像投稿」ボタンの input 要素に付いている class 属性に適切でないものがあり、デザインも不適切なので修正 に変更

Fumie Toyooka さんが12年以上前に更新

  • 題名pc_backend の「画像投稿」ボタンの input 要素に付いている class 属性に適切でないものがあり、デザインも不適切なので修正 から pc_backend の「画像投稿」ボタンの input 要素に付いている class 属性に適切でないものがあり、デザインも統一されていないので class 自体も削除する に変更

Fumie Toyooka さんが12年以上前に更新

  • 題名pc_backend の「画像投稿」ボタンの input 要素に付いている class 属性に適切でないものがあり、デザインも統一されていないので class 自体も削除する から pc_backend の「画像投稿」ボタンの input 要素に付いている class 属性に適切でないものがあり、ボタンの CSS がここだけ他の submit ボタンと異なるので class 自体も削除する に変更

Fumie Toyooka さんが12年以上前に更新

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

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

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

Fumie Toyooka さんが12年以上前に更新

Fumie Toyooka さんが12年以上前に更新

  • 題名pc_backend の「画像投稿」ボタンの input 要素に付いている class 属性に適切でないものがあり、ボタンの CSS がここだけ他の submit ボタンと異なるので class 自体も削除する から pc_backend の「画像投稿」ボタンの input 要素に不要な class 属性が付加されている に変更
  • 説明 を更新 (差分)

Fumie Toyooka さんが12年以上前に更新

Fumie Toyooka さんが12年以上前に更新

Fumie Toyooka さんが12年以上前に更新

Fumie Toyooka さんが12年以上前に更新

Fumie Toyooka さんが12年以上前に更新

Fumie Toyooka さんが12年以上前に更新

  • ステータスRejected(差し戻し) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

Mutsumi Imamura さんが12年以上前に更新

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

Yuya Watanabe さんが12年以上前に更新

「画像投稿」という文言がハードコーディングされているため使用言語を変えた場合でも日本語で表示されるという点が気になります.この件については別チケットで対応としておきましょう.

修正内容について問題ないと思います.
ただ 3.7.1 リリースに対する修正の取り込みの完了が確認できるまでステータスを「Pending Review」としておきます.

Shouta Kashiwagi さんが12年以上前に更新

  • 対象バージョン252 から OpenPNE 3.8beta1 に変更

Fumie Toyooka さんが12年以上前に更新

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

Fumie Toyooka さんが12年以上前に更新

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

Yuya Watanabe さんが12年以上前に更新

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

Yuma Sakata さんが12年以上前に更新

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

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

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