vektor-inc / x-t9

GNU General Public License v2.0
18 stars 1 forks source link

テンプレートのレイアウトに使っているカラムブロックの gap 設定が効かない現象に対応 #250

Closed mtdkei closed 6 months ago

mtdkei commented 6 months ago

チケットへのリンク / 変更の理由(元のissueがあればリンクを貼り付ければOK)

どういう変更をしたか?

個別投稿で使っているカラムブロックのgap設定が効かない現象に対応しました。

実装者はレビュワーに回す前に以下の事を確認してチェックをつけてください。

ソースコードについて

プログラムの変更の場合

テストを書かないのは普通ではありません。書けるテストは極力書くようにしてください。 書いていない場合は書かない理由を記載してください。

その他

変更内容について何を確認したか、どういう方法で確認をしたかなど

  1. [サイトエディター] > [テンプレート] > [個別投稿] に移動
  2. 使われているメインカラムとサイドバーのカラムで使用しているカラムブロックのgapに数値を入れて更新する (今回はわかりやすく0にしています。) image
  3. フロントエンドで見ると変更前はgapを0にしているはずが余白が開いている状態になります。 image この状態から今回のプルリクで2で入力した数値に対応できるようにしました。 image

レビュワーの確認方法・確認する内容など

  1. [サイトエディター] > [テンプレート] > [個別投稿] に移動します。
  2. 使われているメインカラムとサイドバーのカラムで使用しているカラムブロックのgapに数値を入れて更新します。
  3. フロントエンドで2で入力した数値がカラムブロックに反映されているか確認してください。

レビュワーに回す前の確認事項


レビュワー向け

確認して変更が反映されていない場合の確認事項

kurudrive commented 6 months ago

うーむ...どうしようかなぁ... これ基本的に「メインレイアウト」なのでユーザーには個別に変更してほしくないのよねー。

とする事でサイトのデザインの統一を測りたいんだけど...

まぁ...数値を変更する欄があってそれが効かないと不具合と思われるのもそうですよねぇ...

一旦マージしますん(・w・;

mtdkei commented 6 months ago

@kurudrive ご確認ありがとうございます。そういった事情があったのですね。勝手してすみません。 もし必要がなければ解除をお願いします。

kurudrive commented 6 months ago

@mtdkei いえいえ!気付いた事をこやって自分でプルリク出してくれるのはウルトラ大歓迎です! ありがとうございます!