vektor-inc / vektor-wp-libraries

5 stars 0 forks source link

[ Widget PR Content ] 画像の枠線を指定出来るようにする #11

Closed kurudrive closed 6 years ago

kurudrive commented 6 years ago

カラーピッカーで色を選択 色選択がない場合は枠線を表示しない 色が選択されている場合は画像に対して1pxで線を出力する

kurudrive commented 6 years ago

UIは後からまた修正するのであまり気にしなくて良いです。

kurudrive commented 6 years ago

ちなみに背景画像を指定した場合はボタンが色付きだと不自然だよねー。 ボタンの色選択と、ゴーストボタンにするかどうかの選択機能付けたいですね。 まぁこれをしないで一旦リリースしちゃうか、インパクトがあるように組み込んでからリリースするかは判断に迷う所。 と言うか背景パララックス化も実装したいよね。 むーん、悩ましい...。

catherine87 commented 6 years ago

@kurudrive

カラーピッカーでカラーを選択する場合に sassで「.pr_content_media_imgage」クラスを対象に変数とかに変則的なカラーを代入することって可能ですか??

kurudrive commented 6 years ago

@catherine87 おや、なかなか高度な事をしようとしておるな(・w・ 出来るけどややこしくなるだけなので、この場合は img タグに対して直接 style 属性で出力すればOKです。

catherine87 commented 6 years ago

@kurudrive

かしこまりました。 石川さんに付属してもらったクラスは、何も指定しないけど そのまま残しておくってことでいいですか?

kurudrive commented 6 years ago

@catherine87 あれ、っていうか今直書きなんだね。そういえば僕が書いたような気がしてきたわ...。

<img src="<?php echo esc_url( $image[0] ); ?>" style="width:100%; height:auto; border: 1px solid #ccc; margin: 0 0 15px;" />

この style を色指定がなかったら出力しないで、色指定があったら border:1px solid 指定された色 で出力。 width:100%; height:auto; は この img タグにclass名つけて CSS ファイルから指定 が妥当かな。

catherine87 commented 6 years ago

@kurudrive

画像のボーダーを設定できるように変更しました。 一回指定すると、クリアができなくなってしまうので「ボーダーなし」にすることができない状態です。 どうやって書き換えたらいいでしょうか??

catherine87 commented 6 years ago

@kurudrive

とりあえず、リンクボタンのカラーピッカー設置しちゃいます。

catherine87 commented 6 years ago

@kurudrive

リンクボタンのカラーピッカーを追加しました。 仕様としては、 ・ボタンは塗りつぶしタイプとゴーストタイプを選択できます。(デフォルトは塗りつぶしタイプ) ・ボタン用のカラーピッカーとボタンテキスト用のカラーピッカーを設置してそれぞれ色の設定をできるようにしました。

hover の CSS については明日対応します。 コードがカオスだと思いますので、明日午前中ちょっと見直してみますがたぶんそんなに修正できないので、また余裕あるときにでも確認していただきたいです。

よろしくお願いしますm( )m

catherine87 commented 6 years ago

【メモ】「画像の境界線の色 :」のmargin-topを確認・調整 →調整済み

catherine87 commented 6 years ago

@kurudrive

ボタンセクションの仕様についてなのですが、 ボタンテキストカラーが選択できない?ため白塗りボタンの時にリンクボタンテキストが見えなくなる現象を確認しました。

こちらは何か設定の方法などありますでしょうか? 紹介記事などで CSS を記載して設定方法を案内する感じでよいでしょうか?

fireshot capture 275 - _ lightning x exunit i_ - http___vccw test_wp-admin_customize php

catherine87 commented 6 years ago

@kurudrive

ボタンテキストカラーの分岐ですが一応カラーは #fff と #000 で変わるようになりました。 プッシュしてあるので確認はできるかと思います。 コード微妙かもなので明日もう一回見直します。

catherine87 commented 6 years ago

@kurudrive

数値など調整してプッシュしました。 確認お願いします。

kurudrive commented 6 years ago

@catherine87

あれー、かわんないよー。 と思ったらプルしてないだけだった(・w・;

いいじゃんいいじゃん、良いではないですか! おっけーです。ありがとうござます!