vektor-inc / vk-all-in-one-expansion-unit

wordpress plugin of powerful support
https://ja.wordpress.org/plugins/vk-all-in-one-expansion-unit/
GNU General Public License v2.0
7 stars 1 forks source link

タクソノミーウィジェットにプルダウンを追加 #1098

Closed drill-lancer closed 3 months ago

drill-lancer commented 4 months ago

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

https://github.com/vektor-inc/vk-all-in-one-expansion-unit/issues/1097

どういう変更をしたか?

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

ソースコードについて

デザイン・UI

プログラムの変更の場合

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

その他

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

確認URL

ローカル環境

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

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


レビュワー向け

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

sysbird commented 3 months ago

確認しました 実装ありがとうございます

私も Pull Down / Drop down のばらつきが気になります もとからあるほうに合わせると Drop down でしょうけね

また設定するさいの UI が アーカイブリストはトグルボタンであるのに対して、 custom

こちらはプルダウンから選ぶのは、 これから増える予定とか…でなけれは合わせたほうがよいかもしれないです archive

sysbird commented 3 months ago

@kurudrive @mthaichi 石川さんのご意見と、開発チームのレビューをお願いしたいです

kurudrive commented 3 months ago

@drill-lancer @sysbird @goutetsuguma

変更しました。トグルは話がややこしくなるし、もう古いウィジェットなので select のままで良いと思います。

■ Before スクリーンショット 2024-07-29 22 38 44

■ After スクリーンショット 2024-07-29 22 35 21