タクソノミーウィジェットにプルダウンを追加 #1098
タクソノミーウィジェットにプルダウンを追加 #1098
Conversation
drill-lancer
requested review from
mthaichi,
kurudrive,
sysbird,
goutetsuguma and
mtdkei
goutetsuguma
changed the title
There was a problem hiding this comment.
@drill-lancer
コメントには少し気になるところを書きました。（Pull Down と Drop downの名称について）
もう一つはタイトル下に余白がないかもしれません。widget-archives の方のタイトル下と同じくらいの余白があると良いと思いました。
あ、よくよく見たらタイトル下の余白はテーマ側の修正になるっぽい雰囲気を感じました、、
余白の件はスルーしてくださって大丈夫です
プルリクを出したのですが、 Waiting for status to be reported になってしまっている、、
vektor-inc/lightning#1165
| <label for="<?php echo $this->get_field_id( 'form_design' ); ?>"><?php _e( 'Form Design', 'vk-all-in-one-expansion-unit' ); ?></label> | ||
| <select name="<?php echo $this->get_field_name( 'form_design' ); ?>" > | ||
| <option value="list" <?php selected( $instance['form_design'], 'list' ); ?>><?php _e( 'List', 'vk-all-in-one-expansion-unit' ); ?></option> | ||
| <option value="pulldown" <?php selected( $instance['form_design'], 'pulldown' ); ?>><?php _e( 'Pull Down', 'vk-all-in-one-expansion-unit' ); ?></option> |
There was a problem hiding this comment.
widget-archives の方は、「Pull Down」ではなく「Drop down」という名称になっていました。どちらでも良いのかもしれませんが、どちらかに合わせれたら良いのかもしれないと思いかいておきます。
goutetsuguma
changed the title
sysbird
changed the title
|
確認しました 私も Pull Down / Drop down のばらつきが気になります また設定するさいの UI が アーカイブリストはトグルボタンであるのに対して、 こちらはプルダウンから選ぶのは、 |
|
@kurudrive @mthaichi |
sysbird
changed the title
# Conflicts: # assets/js/all.min.js # readme.txt
kurudrive
changed the title
|
@drill-lancer @sysbird @goutetsuguma 変更しました。トグルは話がややこしくなるし、もう古いウィジェットなので select のままで良いと思います。 ■ Before ■ After |
チケットへのリンク / 変更の理由（元のissueがあればリンクを貼り付ければOK）
#1097
どういう変更をしたか？
実装者はレビュワーに回す前に以下の事を確認してチェックをつけてください。
ソースコードについて
デザイン・UI
プログラムの変更の場合
テストを書かないのは普通ではありません。書けるテストは極力書くようにしてください。
書いていない場合は書かない理由を記載してください。
その他
変更内容について何を確認したか、どういう方法で確認をしたかなど
確認URL
ローカル環境
レビュワーの確認方法・確認する内容など
レビュワーに回す前の確認事項
レビュワー向け
確認して変更が反映されていない場合の確認事項