vektor-inc / lightning

Lightning is powerful them for WordPress.
https://lightning.vektor-inc.co.jp
GNU General Public License v2.0
103 stars 28 forks source link

[ css微修正 ] タクソノミーウィジェットの select の上にmarginを追加 #1165

Closed goutetsuguma closed 1 month ago

goutetsuguma commented 2 months ago

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

https://github.com/vektor-inc/vk-all-in-one-expansion-unit/pull/1098#pullrequestreview-2171016667

どういう変更をしたか?

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

変更前 Before

スクリーンショット 2024-07-11 15 27 28

変更後 After

スクリーンショット 2024-07-11 17 19 15

実装者の確認事項

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

cssの微修正のためreadme.txt に書くほどでもないのかなと思い書いておりませんが、必要そうでしたら書きます🙇

プログラムの変更の場合

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

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

*ExUnitのブランチは https://github.com/vektor-inc/vk-all-in-one-expansion-unit/pull/1098 にして、タクソノミーウィジェットを配置して、プルダウンにしてタイトルとselectボックスの間に余白があることを確認してください。

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

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

*ExUnitのブランチは https://github.com/vektor-inc/vk-all-in-one-expansion-unit/pull/1098 にして、タクソノミーウィジェットを配置して、プルダウンにしてタイトルとselectボックスの間に余白があることを確認してください。


レビュワー向け

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

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

mtdkei commented 1 month ago

コンクリフトを調整しbuildしなおしたらテストが通りました。 また、レビュアーではありませんが、G3やG2において、タクソノミーウィジェットの select の上にmarginが追加されていることを確認しました。

kurudrive commented 1 month ago

@goutetsuguma Lightning Pro の方もよろしくお願いいたします ( ̄人 ̄)