vektor-inc / x-t9

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

[CSS微修正]gridcolcard_item_body_inner 内のimageのalignが効かないのでcss追加 #202

Closed goutetsuguma closed 1 year ago

goutetsuguma commented 1 year ago

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

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

どういう変更をしたか?

VK Grid Column Card body inner内にimageブロックを設置してalignを中央・左・右に設定しても、VK Grid Column Card body innerのdivにis-layout-flowのクラス名が付与されないので、画像の配置が効かないようでしたので、wp-blocks-imageのcssを追加しました。

実装者の確認事項

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

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

実装者が確認した手順を箇条書きで記載してください。

確認URL

( どこかのデモサイトかテストサーバーにデプロイ済みなどで確認できる場合はそのURL )

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

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

レビュワーがどういう手順で何を確認して欲しいかを記載してください。


レビュワー向け

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

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

kurudrive commented 1 year ago

@goutetsuguma あ、えっと、これは x-t9 の問題になるかな? と、言うのは、特定のブロック(特にコアではないサードパーティ)用の補正クラスをテーマ追加するというのは、それを使わないユーザーにとって不要なコードが増える事になるので、 テーマ固有のCSSが原因で該当の処理を阻害するなら、テーマ側でそのCSSの出力方法を調整する形になりますが、

VK Grid Column Card body innerのdivにis-layout-flowのクラス名が付与されないので、画像の配置が効かない

とあるので GridColmunCard ブロック側を修正する方が正しいのかなと思ったのですがどうでしょう?

goutetsuguma commented 1 year ago

VK Grid Column Card body innerのdivにis-layout-flowのクラス名が付与されないので、画像の配置が効かない

とあるので GridColmunCard ブロック側を修正する方が正しいのかなと思ったのですがどうでしょう?

そうですね、GridColmunCard ブロック側の修正が正しいとおもいました!

このプルリクをCloseして、issueをvk blocks proに移動しました🙇‍♂️ https://github.com/vektor-inc/vk-blocks-pro/issues/1798