vektor-inc / vk-blocks-pro

13 stars 3 forks source link

【石川さん回答待ち/確認待ち】TT4やTT5 テーマで投稿リスト,投稿リストスライダーの見出しの上に余白が開く現象の修正 #2356

Open akito-38 opened 6 hours ago

akito-38 commented 6 hours ago

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

https://github.com/vektor-inc/vk-blocks-pro/issues/2344

どういう変更をしたか?

TT4やTT5 テーマで表示タイプが「テキスト1カラム」以外のタイプですと、見出しの上に余白が空くので、それを修正しました。 また、見出しの親(黒い半透明のボックス)の高さもずれていたので修正しました。

スクリーンショットまたは動画

変更前 Before

image

変更後 After

image

実装者の確認事項

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

プログラムの変更の場合

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

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

投稿リストブロック及び,投稿リストスライダーブロックを配置します。 テーマをTT4にして、タイトル部分の表示が問題無いことを確認しました。 また、タイプを変更してそれぞれ確認しました。 テーマをTT4と念のため、lightningにも変更して同様の確認をしました。

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

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

・実装者と同じ手順で確認をお願いします。


レビュワー向け

レビュワーが確認して変更が反映されていない場合の確認事項

レビューしてみて意図した動作をしない場合は再度ビルドするなど以下の項目を確認してください。

goutetsuguma commented 4 hours ago

@kurudrive @akito-38

調整ありがとうございます!修正ファイルを見ていたのですが、修正していただいたscssのファイルで石川さんに質問があります🙇

vk-components / package / の中のファイルは vektor-wp-libraries で修正するのが正しいのでしょうか。

/ このファイルの元ファイルは https://github.com/vektor-inc/vektor-wp-libraries にあります。修正の際は上記リポジトリのデータを修正してください。 /

vk-components / package / 内のscssファイルのコメントアウトを見て質問させていただきました